PHP/DiscouragedPHPFunctions: reorganize tests to use empty space and update code comment for namespaced function calls tests

Author: rodrigoprimo

WordPress/Tests/PHP/DiscouragedPHPFunctionsUnitTest.inc

@@ -1,12 +1,12 @@
 <?php
 
-
-
-
-
-
+// Safeguard against false positives on namespaced function calls
+MyNamespace\serialize( $value );
+\MyNamespace\serialize( $value );
+namespace\serialize( $value ); // The sniff should start flagging this once it can resolve relative namespaces.
 
 serialize(); // Warning.
+\serialize( $value ); // Warning.
 unserialize(); // Warning.
 
 urlencode(); // Warning.
@@ -42,10 +42,3 @@ $handle = popen( '/bin/ls', 'r' );
 class System {}
 class Serialize {}
 
-/*
- * Safeguard support for PHP 8.0+ tokenization of namespaced "names".
- */
-\serialize( $value );
-MyNamespace\serialize( $value );
-\MyNamespace\serialize( $value );
-namespace\serialize( $value ); // The sniff should start flagging this once it can resolve relative namespaces.

WordPress/Tests/PHP/DiscouragedPHPFunctionsUnitTest.php

@@ -37,6 +37,7 @@ public function getErrorList() {
 	 */
 	public function getWarningList() {
 		return array(
+			8  => 1,
 			9  => 1,
 			10 => 1,
 			12 => 1,
@@ -60,7 +61,6 @@ public function getWarningList() {
 			37 => 1,
 			38 => 1,
 			39 => 1,
-			48 => 1,
 		);
 	}
 }