Changeset 53510 for trunk/tests/phpunit/tests/post/nav-menu.php

Timestamp:

06/15/2022 02:42:58 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Tests: Use assertSameSets() in some newly introduced tests.

This ensures that not only the array values being compared are equal, but also that their type is the same.

Going forward, stricter type checking by using assertSameSets() or assertSameSetsWithIndex() should generally be preferred, to make the tests more reliable.

Follow-up to [48939], [51137], [51943], [53499], [53504], [53506], [53509].

See #55652.

File:

• trunk/tests/phpunit/tests/post/nav-menu.php (modified) (2 diffs)

trunk/tests/phpunit/tests/post/nav-menu.php

@@ -231 +231 @@
         $args = $action->get_args();
         $last = end( $args );
-        $this->assertEqualSets( array( $post_id ), $last[1], '_prime_post_caches() was not executed.' );
+        $this->assertSameSets( array( $post_id ), $last[1], '_prime_post_caches() was not executed.' );
     }
 
@@ -263 +263 @@
         $args = $action->get_args();
         $last = end( $args );
-        $this->assertEqualSets( array( $term_id ), $last[1], '_prime_term_caches() was not executed.' );
+        $this->assertSameSets( array( $term_id ), $last[1], '_prime_term_caches() was not executed.' );
     }