Changeset 54304 for trunk/tests/phpunit/includes/abstract-testcase.php

Timestamp:

09/25/2022 04:47:02 AM (4 years ago)

Author:

SergeyBiryukov

Message:

Tests: Revert removing empty directory in WP_UnitTestCase_Base::rmdir() for now.

This appears to need more investigation. Instead, delete the test-plugin and link-manager directories in REST API plugins controller tests, for which this change was initially intended.

Follow-up to [54300], [54301], [54303].

See #55652, #56629.

File:

• trunk/tests/phpunit/includes/abstract-testcase.php (modified) (2 diffs)

trunk/tests/phpunit/includes/abstract-testcase.php

@@ -1351 +1351 @@
      *
      * @since 4.0.0
-     * @since 6.1.0 Removes the empty directory if there are no files left.
      *
      * @param string $path Directory path.
      */
     public function rmdir( $path ) {
-        if ( ! is_dir( $path ) ) {
-            return;
-        }
-
         $files = $this->files_in_dir( $path );
         foreach ( $files as $file ) {
@@ -1365 +1360 @@
                 $this->unlink( $file );
             }
-        }
-
-        /*
-         * If there were no ignored files, remove the empty directory.
-         * If there are any nested empty directories, remove them too.
-         */
-        if ( ! array_intersect( $files, self::$ignore_files ) ) {
-            $this->delete_folders( $path );
         }
     }