Changeset 13314 for trunk/tests/phpunit/testcases/messages/class.bp-messages-notice.php

Timestamp:

08/13/2022 08:58:51 AM (2 years ago)

Author:

imath

Message:

Fully enjoy Yoast’s PHPUnit polyfills

Using these polyfills let us use PHPUnit v9.x for our tests and add PHP 8.1 to our testing matrix. Some additional edits to our PHP unit tests suite were needed:

• Stop using PHPunit deprecated functions.
• Rename some BP_UnitTestCase methods to use Yoast's polyfills.
• Edit the PHP Unit test GH action and also run this action on pull requests.
• Update some composer dependencies, remove the one about phpunit/phpunit:^7.5 and add a new composer script to use PHPUnit v9.x.

Props renatonascalves, rafiahmedd

Closes ​https://github.com/buddypress/buddypress/pull/13
Fixes #8649

File:

• trunk/tests/phpunit/testcases/messages/class.bp-messages-notice.php (modified) (1 diff)

trunk/tests/phpunit/testcases/messages/class.bp-messages-notice.php

@@ -8 +8 @@
     protected $old_current_user = 0;
 
-    public function setUp() {
-        parent::setUp();
+    public function set_up() {
+        parent::set_up();
         $this->old_current_user = get_current_user_id();
         $this->set_current_user( self::factory()->user->create( array( 'role' => 'administrator' ) ) );
     }
 
-    public function tearDown() {
-        parent::tearDown();
+    public function tear_down() {
+        parent::tear_down();
         $this->set_current_user( $this->old_current_user );
     }