Changeset 35186 for trunk/tests/phpunit/tests/adminbar.php

Timestamp:

10/15/2015 04:43:37 AM (9 years ago)

Author:

wonderboymusic

Message:

Unit Tests: implement setUpBeforeClass() and tearDownAfterClass() on WP_UnitTestCase. Use late static binding (plus a gross fallback for PHP 5.2) to check if wpSetUpBeforeClass() or wpTearDownAfterClass() exist on the called class, and then call it and pass a static WP_UnitTest_Factory instance via Dependency Injection, if it exists.

This makes it way easier to add fixtures, and tear them down, without needing to instantiate WP_UnitTest_Factory in every class - removes the need to call commit_transaction() in each individual class.

See #30017, #33968.

File:

• trunk/tests/phpunit/tests/adminbar.php (modified) (1 diff)

trunk/tests/phpunit/tests/adminbar.php

@@ -8 +8 @@
 class Tests_AdminBar extends WP_UnitTestCase {
 
-    static function setUpBeforeClass() {
-        WP_UnitTestCase::setUpBeforeClass();
-        require_once ABSPATH . WPINC . '/class-wp-admin-bar.php';
-    }
-
-    function setUp() {
-        parent::setUp();
-        $this->current_user = get_current_user_id();
-    }
-
-    function tearDown() {
-        wp_set_current_user( $this->current_user );
-        parent::tearDown();
+    public static function setUpBeforeClass() {
+        require_once( ABSPATH . WPINC . '/class-wp-admin-bar.php' );
     }