Changeset 53388

Timestamp:

05/10/2022 09:28:18 PM (4 years ago)

Author:

hellofromTonya

Message:

Editor: Use stricter assertions in comment template tests.

Backport of the remaining Comment Template block tests from Gutenberg:

• assertEquals() replaced with assertSameSetsWithIndex().
• assertion's argument order changed to ensure expected and then actual.

Follow-up to [53353], [53298], [53172], [53138].

Props bernhard-reiter.
Fixes #55708.

File:

• trunk/tests/phpunit/tests/blocks/renderCommentTemplate.php (modified) (4 diffs)

trunk/tests/phpunit/tests/blocks/renderCommentTemplate.php

@@ -67 +67 @@
                 );
 
-                $this->assertEquals(
+                $this->assertSameSetsWithIndex(
                         array(
                                 'orderby'       => 'comment_date_gmt',
@@ -124 +124 @@
                 $block = new WP_Block( $parsed_blocks[0] );
 
-                $this->assertEquals(
+                $this->assertSameSetsWithIndex(
                         array(
                                 'orderby'       => 'comment_date_gmt',
@@ -422 +422 @@
                 add_filter( 'wp_get_current_commenter', $commenter_filter );
 
-                $this->assertEquals(
-                        build_comment_query_vars_from_block( $block ),
+                $this->assertSameSetsWithIndex(
                         array(
                                 'orderby'            => 'comment_date_gmt',
@@ -434 +433 @@
                                 'number'             => 5,
                                 'paged'              => 1,
-                        )
+                        ),
+                        build_comment_query_vars_from_block( $block )
                 );
         }