Changeset 48997 for trunk/tests/phpunit/tests/ajax/TagSearch.php

Timestamp:

09/18/2020 01:48:27 PM (7 months ago)

Author:

SergeyBiryukov

Message:

Tests: Switch Ajax tests to use the expectException() method directly, instead of the WP_UnitTestCase_Base::setExpectedException() wrapper.

This brings some consistency with other tests.

Follow-up to [48996].

See #51344.

File:

• trunk/tests/phpunit/tests/ajax/TagSearch.php (modified) (4 diffs)

trunk/tests/phpunit/tests/ajax/TagSearch.php

@@ -75 +75 @@
         // Make the request.
         // No output, so we get a stop exception.
-        $this->setExpectedException( 'WPAjaxDieStopException', '' );
+        $this->expectException( 'WPAjaxDieStopException' );
+        $this->expectExceptionMessage( '' );
         $this->_handleAjax( 'ajax-tag-search' );
     }
@@ -115 +116 @@
 
         // Make the request.
-        $this->setExpectedException( 'WPAjaxDieStopException', '-1' );
+        $this->expectException( 'WPAjaxDieStopException' );
+        $this->expectExceptionMessage( '-1' );
         $this->_handleAjax( 'ajax-tag-search' );
     }
@@ -132 +134 @@
 
         // Make the request.
-        $this->setExpectedException( 'WPAjaxDieStopException', '0' );
+        $this->expectException( 'WPAjaxDieStopException' );
+        $this->expectExceptionMessage( '0' );
         $this->_handleAjax( 'ajax-tag-search' );
     }
@@ -149 +152 @@
 
         // Make the request.
-        $this->setExpectedException( 'WPAjaxDieStopException', '-1' );
+        $this->expectException( 'WPAjaxDieStopException' );
+        $this->expectExceptionMessage( '-1' );
         $this->_handleAjax( 'ajax-tag-search' );
     }