Changeset 35311 for trunk/tests/phpunit/tests/ajax/Autosave.php

Timestamp:

10/21/2015 03:17:36 AM (10 years ago)

Author:

wonderboymusic

Message:

AJAX UNIT TESTS: Have you ever wondered why these take 600 forevers to run? They all eventually call do_action( 'admin_init' ), which has _maybe_update_core, _maybe_update_plugins, and _maybe_update_themes hooked to it. REMOVE THEM, and AJAX unit tests run like the wind. Tests_Ajax_Response is still slow.

See #30017, #33968.

File:

• trunk/tests/phpunit/tests/ajax/Autosave.php (modified) (10 diffs)

trunk/tests/phpunit/tests/ajax/Autosave.php

@@ -22 +22 @@
     protected $_post = null;
 
-    /**
-     * user_id
-     * @var int
-     */
-    protected $user_id = 0;
+    protected static $admin_id = 0;
+    protected static $editor_id = 0;
+    protected static $post;
+    protected static $post_id;
+    protected static $user_ids = array();
+
+    public static function wpSetUpBeforeClass( $factory ) {
+        self::$user_ids[] = self::$admin_id = $factory->user->create( array( 'role' => 'administrator' ) );
+        self::$user_ids[] = self::$editor_id = $factory->user->create( array( 'role' => 'editor' ) );
+
+        self::$post_id = $factory->post->create( array( 'post_status' => 'draft' ) );
+        self::$post = get_post( self::$post_id );
+    }
+
+    public static function wpTearDownAfterClass() {
+        foreach ( self::$user_ids as $user_id ) {
+            self::delete_user( $user_id );
+        }
+
+        wp_delete_post( self::$post_id, true );
+    }
 
     /**
@@ -34 +50 @@
         parent::setUp();
         // Set a user so the $post has 'post_author'
-        $this->user_id = self::factory()->user->create( array( 'role' => 'administrator' ) );
-        wp_set_current_user( $this->user_id );
-
-        $post_id = self::factory()->post->create( array( 'post_status' => 'draft' ) );
-        $this->_post = get_post( $post_id );
+        wp_set_current_user( self::$admin_id );
     }
 
@@ -47 +59 @@
     public function test_autosave_post() {
         // The original post_author
-        wp_set_current_user( $this->user_id );
+        wp_set_current_user( self::$admin_id );
 
         // Set up the $_POST request
@@ -56 +68 @@
             'data' => array(
                 'wp_autosave' => array(
-                    'post_id'       => $this->_post->ID,
-                    '_wpnonce'      => wp_create_nonce( 'update-post_' . $this->_post->ID ),
-                    'post_content'  => $this->_post->post_content . PHP_EOL . $md5,
+                    'post_id'       => self::$post_id,
+                    '_wpnonce'      => wp_create_nonce( 'update-post_' . self::$post->ID ),
+                    'post_content'  => self::$post->post_content . PHP_EOL . $md5,
                     'post_type'     => 'post',
                 ),
@@ -79 +91 @@
 
         // Check that the edit happened
-        $post = get_post( $this->_post->ID );
-        $this->assertGreaterThanOrEqual( 0, strpos( $post->post_content, $md5 ) );
+        $post = get_post( self::$post_id );
+        $this->assertGreaterThanOrEqual( 0, strpos( self::$post->post_content, $md5 ) );
     }
 
@@ -89 +101 @@
     public function test_autosave_locked_post() {
         // Lock the post to another user
-        $another_user_id = self::factory()->user->create( array( 'role' => 'editor' ) );
-        wp_set_current_user( $another_user_id );
-        wp_set_post_lock( $this->_post->ID );
+        wp_set_current_user( self::$editor_id );
+        wp_set_post_lock( self::$post_id );
 
-        wp_set_current_user( $this->user_id );
+        wp_set_current_user( self::$admin_id );
 
         // Ensure post is locked
-        $this->assertEquals( $another_user_id, wp_check_post_lock( $this->_post->ID ) );
+        $this->assertEquals( self::$editor_id, wp_check_post_lock( self::$post_id ) );
 
         // Set up the $_POST request
@@ -105 +116 @@
             'data' => array(
                 'wp_autosave' => array(
-                    'post_id'       => $this->_post->ID,
-                    '_wpnonce'      => wp_create_nonce( 'update-post_' . $this->_post->ID ),
-                    'post_content'  => $this->_post->post_content . PHP_EOL . $md5,
+                    'post_id'       => self::$post_id,
+                    '_wpnonce'      => wp_create_nonce( 'update-post_' . self::$post_id ),
+                    'post_content'  => self::$post->post_content . PHP_EOL . $md5,
                     'post_type'     => 'post',
                 ),
@@ -127 +138 @@
 
         // Check that the original post was NOT edited
-        $post = get_post( $this->_post->ID );
+        $post = get_post( self::$post_id );
         $this->assertFalse( strpos( $post->post_content, $md5 ) );
 
         // Check if the autosave post was created
-        $autosave = wp_get_post_autosave( $this->_post->ID, get_current_user_id() );
+        $autosave = wp_get_post_autosave( self::$post_id, get_current_user_id() );
         $this->assertNotEmpty( $autosave );
         $this->assertGreaterThanOrEqual( 0, strpos( $autosave->post_content, $md5 ) );
@@ -142 +153 @@
     public function test_with_invalid_nonce( ) {
 
-        wp_set_current_user( $this->user_id );
+        wp_set_current_user( self::$admin_id );
 
         // Set up the $_POST request
@@ -150 +161 @@
             'data' => array(
                 'wp_autosave' => array(
-                    'post_id'  => $this->_post->ID,
+                    'post_id'  => self::$post_id,
                     '_wpnonce' => substr( md5( uniqid() ), 0, 10 ),
                 ),