Changeset 51870

Timestamp:

09/26/2021 12:12:56 AM (5 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' );