Changeset 51968 for trunk/tests/phpunit/tests/admin/includesPost.php

Timestamp:

11/01/2021 10:22:49 PM (3 years ago)

Author:

hellofromTonya

Message:

Build/Test Tools: Ignore "null to nullable" deprecations for select tests.

Adds an expectation for PHP 8.1 "passing null to non-nullable" deprecation notice to select tests where the deprecation is generated by one of the functions in the wp-includes/formatting.php file, either via a filter hook callback or by a direct call.

Instead of haphazardly fixing these issues exposed by the tests, a more structural and all-encompassing solution for input validation should be architected and implemented as otherwise, we'll keep running into similar issues time and again with each new PHP version.

To discourage people from "fixing" these issues now anyway, this commit "hides" nearly all of these issues from the test runs.

Once a more structural solution is designed, these tests and the underlying functions causing the deprecation notices should be revisited and the structural solution put in place.

Includes a few minor other tweaks to select tests:

• Removing a stray return (twice) from assertion statements.
• Removing calls to ob_*() functions in favour of letting PHPUnit manage the output catching. This prevents warnings along the lines of Test code or tested code did not (only) close its own output buffers.

Props jrf, hellofromTonya.
See #53635.

File:

• trunk/tests/phpunit/tests/admin/includesPost.php (modified) (6 diffs)

trunk/tests/phpunit/tests/admin/includesPost.php

@@ -877 +877 @@
      */
     public function test_post_exists_should_support_post_type() {
+        if ( PHP_VERSION_ID >= 80100 ) {
+            /*
+             * For the time being, ignoring PHP 8.1 "null to non-nullable" deprecations coming in
+             * via hooked in filter functions until a more structural solution to the
+             * "missing input validation" conundrum has been architected and implemented.
+             */
+            $this->expectDeprecation();
+            $this->expectDeprecationMessageMatches( '`Passing null to parameter \#[0-9]+ \(\$[^\)]+\) of type [^ ]+ is deprecated`' );
+        }
+
         $title     = 'Foo Bar';
         $post_type = 'page';
@@ -894 +904 @@
      */
     public function test_post_exists_should_not_match_a_page_for_post() {
+        if ( PHP_VERSION_ID >= 80100 ) {
+            /*
+             * For the time being, ignoring PHP 8.1 "null to non-nullable" deprecations coming in
+             * via hooked in filter functions until a more structural solution to the
+             * "missing input validation" conundrum has been architected and implemented.
+             */
+            $this->expectDeprecation();
+            $this->expectDeprecationMessageMatches( '`Passing null to parameter \#[0-9]+ \(\$[^\)]+\) of type [^ ]+ is deprecated`' );
+        }
+
         $title     = 'Foo Bar';
         $post_type = 'page';
@@ -911 +931 @@
      */
     public function test_post_exists_should_support_post_status() {
+        if ( PHP_VERSION_ID >= 80100 ) {
+            /*
+             * For the time being, ignoring PHP 8.1 "null to non-nullable" deprecations coming in
+             * via hooked in filter functions until a more structural solution to the
+             * "missing input validation" conundrum has been architected and implemented.
+             */
+            $this->expectDeprecation();
+            $this->expectDeprecationMessageMatches( '`Passing null to parameter \#[0-9]+ \(\$[^\)]+\) of type [^ ]+ is deprecated`' );
+        }
+
         $title       = 'Foo Bar';
         $post_type   = 'post';
@@ -931 +961 @@
      */
     public function test_post_exists_should_support_post_type_status_combined() {
+        if ( PHP_VERSION_ID >= 80100 ) {
+            /*
+             * For the time being, ignoring PHP 8.1 "null to non-nullable" deprecations coming in
+             * via hooked in filter functions until a more structural solution to the
+             * "missing input validation" conundrum has been architected and implemented.
+             */
+            $this->expectDeprecation();
+            $this->expectDeprecationMessageMatches( '`Passing null to parameter \#[0-9]+ \(\$[^\)]+\) of type [^ ]+ is deprecated`' );
+        }
+
         $title       = 'Foo Bar';
         $post_type   = 'post';
@@ -950 +990 @@
      */
     public function test_post_exists_should_only_match_correct_post_status() {
+        if ( PHP_VERSION_ID >= 80100 ) {
+            /*
+             * For the time being, ignoring PHP 8.1 "null to non-nullable" deprecations coming in
+             * via hooked in filter functions until a more structural solution to the
+             * "missing input validation" conundrum has been architected and implemented.
+             */
+            $this->expectDeprecation();
+            $this->expectDeprecationMessageMatches( '`Passing null to parameter \#[0-9]+ \(\$[^\)]+\) of type [^ ]+ is deprecated`' );
+        }
+
         $title       = 'Foo Bar';
         $post_type   = 'post';
@@ -969 +1019 @@
      */
     public function test_post_exists_should_not_match_invalid_post_type_and_status_combined() {
+        if ( PHP_VERSION_ID >= 80100 ) {
+            /*
+             * For the time being, ignoring PHP 8.1 "null to non-nullable" deprecations coming in
+             * via hooked in filter functions until a more structural solution to the
+             * "missing input validation" conundrum has been architected and implemented.
+             */
+            $this->expectDeprecation();
+            $this->expectDeprecationMessageMatches( '`Passing null to parameter \#[0-9]+ \(\$[^\)]+\) of type [^ ]+ is deprecated`' );
+        }
+
         $title       = 'Foo Bar';
         $post_type   = 'post';