Changeset 51869 for trunk/tests/phpunit/tests/multisite/msFilesRewriting.php

Timestamp:

09/25/2021 01:27:47 AM (3 years ago)

Author:

SergeyBiryukov

Message:

Tests: Remove unnecessary setUp() and tearDown() methods in multisite tests.

These were originally added in [26252] to suppress database errors on setUp() and restore on tearDown() for tests that call wpmu_create_blog(), blog factory, or installation code that attempts to clear transients.

As the multisite test coverage expanded, these methods ended up being unnecessarily copied into other test classes, where database error suppression is not required.

Follow-up to [26252], [29916], [30286], [33184], [34898], [34899], [34901], [37234], [37477], [37894], [49212], [49616], [51859].

See #53363.

File:

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

trunk/tests/phpunit/tests/multisite/msFilesRewriting.php

@@ -13 +13 @@
      */
     class Tests_Multisite_msFilesRewriting extends WP_UnitTestCase {
-        protected $suppress = false;
 
         function set_up() {
-            global $wpdb;
             parent::set_up();
-            $this->suppress = $wpdb->suppress_errors();
-
             update_site_option( 'ms_files_rewriting', 1 );
             ms_upload_constants();
-        }
-
-        function tear_down() {
-            global $wpdb;
-
-            $wpdb->suppress_errors( $this->suppress );
-
-            parent::tear_down();
         }