Changeset 58183 for trunk/tests/phpunit/tests/editor/navigation-fallback.php

Timestamp:

05/22/2024 03:39:51 PM (22 months ago)

Author:

SergeyBiryukov

Message:

Tests: Use assertSame() in navigation fallback tests.

This ensures that not only the return values match the expected results, but also that their type is the same.

Going forward, stricter type checking by using assertSame() should generally be preferred to assertEquals() where appropriate, to make the tests more reliable.

Follow-up to [56052].

See #60705.

File:

• trunk/tests/phpunit/tests/editor/navigation-fallback.php (modified) (9 diffs)

trunk/tests/phpunit/tests/editor/navigation-fallback.php

@@ -46 +46 @@
         $this->assertInstanceOf( 'WP_Post', $data, 'Response should be of the correct type.' );
 
-        $this->assertEquals( 'wp_navigation', $data->post_type, 'Fallback menu type should be `wp_navigation`' );
-
-        $this->assertEquals( 'Navigation', $data->post_title, 'Fallback menu title should be the default fallback title' );
-
-        $this->assertEquals( 'navigation', $data->post_name, 'Fallback menu slug (post_name) should be the default slug' );
-
-        $this->assertEquals( '<!-- wp:page-list /-->', $data->post_content );
+        $this->assertSame( 'wp_navigation', $data->post_type, 'Fallback menu type should be `wp_navigation`' );
+
+        $this->assertSame( 'Navigation', $data->post_title, 'Fallback menu title should be the default fallback title' );
+
+        $this->assertSame( 'navigation', $data->post_name, 'Fallback menu slug (post_name) should be the default slug' );
+
+        $this->assertSame( '<!-- wp:page-list /-->', $data->post_content );
 
         $navs_in_db = $this->get_navigations_in_database();
@@ -93 +93 @@
         $this->assertInstanceOf( 'WP_Post', $data, 'Response should be of the correct type.' );
 
-        $this->assertNotEquals( '<!-- wp:page-list /-->', $data->post_content, 'Navigation Menu should not contain a Page List block.' );
+        $this->assertNotSame( '<!-- wp:page-list /-->', $data->post_content, 'Navigation Menu should not contain a Page List block.' );
 
         $this->assertEmpty( $data->post_content, 'Menu should be empty.' );
@@ -114 +114 @@
         $this->assertInstanceOf( 'WP_Post', $data, 'Response should be of the correct type.' );
 
-        $this->assertEquals( 'Navigation', $data->post_title, 'Fallback menu title should be the default title' );
+        $this->assertSame( 'Navigation', $data->post_title, 'Fallback menu title should be the default title' );
 
         $navs_in_db = $this->get_navigations_in_database();
@@ -147 +147 @@
         $this->assertInstanceOf( 'WP_Post', $data, 'Response should be of the correct type.' );
 
-        $this->assertEquals( $most_recently_published_nav->post_title, $data->post_title, 'Fallback menu title should be the same as the most recently created menu.' );
-
-        $this->assertEquals( $most_recently_published_nav->post_name, $data->post_name, 'Post name should be the same as the most recently created menu.' );
-
-        $this->assertEquals( $most_recently_published_nav->post_content, $data->post_content, 'Post content should be the same as the most recently created menu.' );
+        $this->assertSame( $most_recently_published_nav->post_title, $data->post_title, 'Fallback menu title should be the same as the most recently created menu.' );
+
+        $this->assertSame( $most_recently_published_nav->post_name, $data->post_name, 'Post name should be the same as the most recently created menu.' );
+
+        $this->assertSame( $most_recently_published_nav->post_content, $data->post_content, 'Post content should be the same as the most recently created menu.' );
 
         // Check that no new Navigation menu was created.
@@ -180 +180 @@
         $this->assertInstanceOf( 'WP_Post', $data, 'Response should be of the correct type.' );
 
-        $this->assertEquals( 'Existing Classic Menu', $data->post_title, 'Fallback menu title should be the same as the classic menu.' );
+        $this->assertSame( 'Existing Classic Menu', $data->post_title, 'Fallback menu title should be the same as the classic menu.' );
 
         // Assert that the fallback contains a navigation-link block.
@@ -234 +234 @@
         $this->assertInstanceOf( 'WP_Post', $data, 'Response should be of the correct type.' );
 
-        $this->assertEquals( 'Classic Menu in Primary Location', $data->post_title, 'Fallback menu title should match the menu in the "primary" location.' );
+        $this->assertSame( 'Classic Menu in Primary Location', $data->post_title, 'Fallback menu title should match the menu in the "primary" location.' );
     }
 
@@ -272 +272 @@
         $this->assertInstanceOf( 'WP_Post', $data, 'Response should be of the correct type.' );
 
-        $this->assertEquals( 'Primary', $data->post_title, 'Fallback menu title should match the menu with the slug "primary".' );
+        $this->assertSame( 'Primary', $data->post_title, 'Fallback menu title should match the menu with the slug "primary".' );
     }
 
@@ -310 +310 @@
         $this->assertInstanceOf( 'WP_Post', $data, 'Response should be of the correct type.' );
 
-        $this->assertEquals( 'Most Recent Classic Menu', $data->post_title, 'Fallback menu title should match the menu that was created most recently.' );
+        $this->assertSame( 'Most Recent Classic Menu', $data->post_title, 'Fallback menu title should match the menu that was created most recently.' );
     }
 
@@ -342 +342 @@
         $this->assertInstanceOf( 'WP_Post', $data, 'Response should be of the correct type.' );
 
-        $this->assertEquals( $existing_navigation_menu->post_title, $data->post_title, 'Fallback menu title should be the same as the existing Navigation menu.' );
-
-        $this->assertNotEquals( 'Existing Classic Menu', $data->post_title, 'Fallback menu title should not be the same as the Classic Menu.' );
+        $this->assertSame( $existing_navigation_menu->post_title, $data->post_title, 'Fallback menu title should be the same as the existing Navigation menu.' );
+
+        $this->assertNotSame( 'Existing Classic Menu', $data->post_title, 'Fallback menu title should not be the same as the Classic Menu.' );
 
         // Check that only a single Navigation fallback was created.