Changeset 51870

Timestamp:

09/26/2021 12:12:56 AM (3 years ago)

Author:

SergeyBiryukov

Message:

Tests: Don't skip some Ajax tests on multisite, add them to the ms-excluded group instead.

Follow-up to [46693], [49835].

See #53363.

Location:

trunk/tests/phpunit/tests/ajax

Files:

• Attachments.php (modified) (1 diff)
• DeletePlugin.php (modified) (1 diff)
• ManageThemes.php (modified) (2 diffs)
• UpdatePlugin.php (modified) (1 diff)

trunk/tests/phpunit/tests/ajax/Attachments.php

@@ -64 +64 @@
     /**
      * @ticket 36578
+     * @group ms-excluded
      */
     public function test_wp_ajax_send_attachment_to_editor_should_return_a_link() {
-        $this->skipWithMultisite();
-
         // Become an administrator.
         $post    = $_POST;

trunk/tests/phpunit/tests/ajax/DeletePlugin.php

@@ -126 +126 @@
     }
 
+    /**
+     * @group ms-excluded
+     */
     public function test_delete_plugin() {
-        $this->skipWithMultisite();
         $this->_setRole( 'administrator' );
 

trunk/tests/phpunit/tests/ajax/ManageThemes.php

@@ -102 +102 @@
     }
 
+    /**
+     * @group ms-excluded
+     */
     public function test_update_theme() {
-        $this->skipWithMultisite();
         $this->_setRole( 'administrator' );
 
@@ -140 +142 @@
     }
 
+    /**
+     * @group ms-excluded
+     */
     function test_uppercase_theme_slug() {
-        $this->skipWithMultisite();
         $this->_setRole( 'administrator' );
 

trunk/tests/phpunit/tests/ajax/UpdatePlugin.php

@@ -130 +130 @@
     }
 
+    /**
+     * @group ms-excluded
+     */
     public function test_update_plugin() {
-        $this->skipWithMultisite();
         $this->_setRole( 'administrator' );