Changeset 35186 for trunk/tests/phpunit/tests/user/countUserPosts.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/user/countUserPosts.php (modified) (3 diffs)

trunk/tests/phpunit/tests/user/countUserPosts.php

@@ -9 +9 @@
     static $post_ids = array();
 
-    public static function setUpBeforeClass() {
-        $factory = new WP_UnitTest_Factory();
-
+    public static function wpSetUpBeforeClass( $factory ) {
         self::$user_id = $factory->user->create( array(
             'role' => 'author',
@@ -34 +32 @@
             'post_type'   => 'wptests_pt',
         ) ) );
-
-        self::commit_transaction();
     }
 
-    public static function tearDownAfterClass() {
+    public static function wpTearDownAfterClass() {
         if ( is_multisite() ) {
             wpmu_delete_user( self::$user_id );
@@ -48 +44 @@
             wp_delete_post( $post_id, true );
         }
-
-        self::commit_transaction();
     }