Changeset 40539

Timestamp:

04/23/2017 11:45:46 AM (8 years ago)

Author:

johnbillion

Message:

Build/Test Tools: Move the setExpectedException() method into the WP_Ajax_UnitTestCase class to avoid a fatal error when PHPUnit 3.6 is in use.

This needs more investigation but it gets the test suite on PHP 5.2 passing again.

See #39822

Location:

trunk/tests/phpunit/includes

Files:

• testcase-ajax.php (modified) (1 diff)
• testcase.php (modified) (1 diff)

trunk/tests/phpunit/includes/testcase-ajax.php ¶

@@ -201 +201 @@
             $this->_last_response = $buffer;
     }
+
+    /**
+     * PHPUnit 6+ compatibility shim.
+     *
+     * @param mixed      $exception
+     * @param string     $message
+     * @param int|string $code
+     */
+    public function setExpectedException( $exception, $message = '', $code = null ) {
+        if ( is_callable( 'parent::setExpectedException' ) ) {
+            parent::setExpectedException( $exception, $message, $code );
+        } else {
+            $this->expectException( $exception );
+            if ( '' !== $message ) {
+                $this->expectExceptionMessage( $message );
+            }
+            if ( null !== $code ) {
+                $this->expectExceptionCode( $code );
+            }
+        }
+    }
 }

trunk/tests/phpunit/includes/testcase.php ¶

@@ -414 +414 @@
     public function setExpectedIncorrectUsage( $doing_it_wrong ) {
         array_push( $this->expected_doing_it_wrong, $doing_it_wrong );
-    }
-
-    /**
-     * PHPUnit 6+ compatibility shim.
-     *
-     * @param mixed      $exception
-     * @param string     $message
-     * @param int|string $code
-     */
-    public function setExpectedException( $exception, $message = '', $code = null ) {
-        if ( is_callable( 'parent::setExpectedException' ) ) {
-            parent::setExpectedException( $exception, $message, $code );
-        } else {
-            $this->expectException( $exception );
-            if ( '' !== $message ) {
-                $this->expectExceptionMessage( $message );
-            }
-            if ( null !== $code ) {
-                $this->expectExceptionCode( $code );
-            }
-        }
     }