Changeset 48463

Timestamp:

07/14/2020 12:32:36 AM (5 years ago)

Author:

SergeyBiryukov

Message:

Tests: Copy themes from tests/phpunit/data to wp-content/themes, instead of creating a symlink.

This allows the theme file tests in phpunit/tests/link/themeFile.php to run on Windows without requiring administrative privileges.

Follow-up to [42812], [42819].

Props danielhuesken, christophherr, davidbaumwald, SergeyBiryukov.
See #40856, #39975.

File:

• trunk/tests/phpunit/tests/link/themeFile.php (modified) (1 diff)

trunk/tests/phpunit/tests/link/themeFile.php

@@ -6 +6 @@
 
     public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
-        if ( ! function_exists( 'symlink' ) ) {
-            self::markTestSkipped( 'symlink() is not available.' );
-        }
-        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-        if ( ! @symlink( DIR_TESTDATA . '/theme-file-parent', WP_CONTENT_DIR . '/themes/theme-file-parent' ) ) {
-            self::markTestSkipped( 'Could not create parent symlink.' );
-        }
-        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-        if ( ! @symlink( DIR_TESTDATA . '/theme-file-child', WP_CONTENT_DIR . '/themes/theme-file-child' ) ) {
-            self::markTestSkipped( 'Could not create child symlink.' );
+        $themes = array(
+            'theme-file-parent',
+            'theme-file-child',
+        );
+
+        // Copy themes from tests/phpunit/data to wp-content/themes.
+        foreach ( $themes as $theme ) {
+            $source_dir = DIR_TESTDATA . '/' . $theme;
+            $dest_dir   = WP_CONTENT_DIR . '/themes/' . $theme;
+
+            mkdir( $dest_dir );
+
+            foreach ( glob( $source_dir . '/*.*' ) as $theme_file ) {
+                copy( $theme_file, $dest_dir . '/' . basename( $theme_file ) );
+            }
         }
     }
 
     public static function wpTearDownAfterClass() {
-        unlink( WP_CONTENT_DIR . '/themes/theme-file-parent' );
-        unlink( WP_CONTENT_DIR . '/themes/theme-file-child' );
+        $themes = array(
+            'theme-file-parent',
+            'theme-file-child',
+        );
+
+        // Remove previously copied themes from wp-content/themes.
+        foreach ( $themes as $theme ) {
+            $dest_dir = WP_CONTENT_DIR . '/themes/' . $theme;
+
+            foreach ( glob( $dest_dir . '/*.*' ) as $theme_file ) {
+                unlink( $theme_file );
+            }
+
+            rmdir( $dest_dir );
+        }
     }