Merge pull request #781 from Automattic/3.0/fix/713-nopaging-add-extr…

…a-test
Automattic · Aug 25, 2023 · 1f910a8 · 1f910a8
2 parents 5228fdf + 6890027
commit 1f910a8
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/WordPressVIPMinimum/Tests/Performance/NoPagingUnitTest.inc b/WordPressVIPMinimum/Tests/Performance/NoPagingUnitTest.inc
@@ -7,3 +7,9 @@ $args = array(
 _query_posts( 'nopaging=true' ); // Bad.
 
 $query_args['my_posts_per_page'] = -1; // OK.
+
+// Verify handling with no trailing comma at end of array.
+$args = array(
+	'nopaging' => true // Bad.
+);
+$args = [ 'nopaging' => true ]; // Bad.
diff --git a/WordPressVIPMinimum/Tests/Performance/NoPagingUnitTest.php b/WordPressVIPMinimum/Tests/Performance/NoPagingUnitTest.php
@@ -27,8 +27,10 @@ class NoPagingUnitTest extends AbstractSniffUnitTest {
 	 */
 	public function getErrorList() {
 		return [
-			4 => 1,
-			7 => 1,
+			4  => 1,
+			7  => 1,
+			13 => 1,
+			15 => 1,
 		];
 	}
-Original file line number
+Diff line change
@@ Expand Up / @@ -27,8 +27,10 @@ class NoPagingUnitTest extends AbstractSniffUnitTest { @@
     	 */
     	public function getErrorList() {
     		return [
-=> 1,
-=> 1,
+=> 1,
+=> 1,
+=> 1,
+=> 1,
     		];
     	}
@@ Expand Down @@