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

Timestamp:

09/18/2020 01:48:27 PM (4 years 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/EditComment.php (modified) (4 diffs)

trunk/tests/phpunit/tests/ajax/EditComment.php

@@ -150 +150 @@
 
         // Make the request.
-        $this->setExpectedException( 'WPAjaxDieStopException', '-1' );
+        $this->expectException( 'WPAjaxDieStopException' );
+        $this->expectExceptionMessage( '-1' );
         $this->_handleAjax( 'edit-comment' );
     }
@@ -178 +179 @@
 
         // Make the request.
-        $this->setExpectedException( 'WPAjaxDieStopException', '-1' );
+        $this->expectException( 'WPAjaxDieStopException' );
+        $this->expectExceptionMessage( '-1' );
         $this->_handleAjax( 'get-comments' );
     }
@@ -198 +200 @@
 
         // Make the request.
-        $this->setExpectedException( 'WPAjaxDieStopException', '-1' );
+        $this->expectException( 'WPAjaxDieStopException' );
+        $this->expectExceptionMessage( '-1' );
         $this->_handleAjax( 'edit-comment' );
     }
@@ -226 +229 @@
 
         // Make the request.
-        $this->setExpectedException( 'WPAjaxDieStopException', 'wp_update_comment_data filter fails for this comment.' );
+        $this->expectException( 'WPAjaxDieStopException' );
+        $this->expectExceptionMessage( 'wp_update_comment_data filter fails for this comment.' );
         $this->_handleAjax( 'edit-comment' );
     }