From 3400eb9cb787121053abd6bb34ff2781ca88146d Mon Sep 17 00:00:00 2001 From: Pascal Birchler Date: Mon, 13 Jan 2025 17:13:25 +0000 Subject: [PATCH] Build/Test Tools: Use Git when fetching the WordPress Importer for use in tests. This switches to using Git in the local Docker environment install script to check out a copy of the WordPress Importer plugin for use in unit tests. Previously, SVN was used and the commands were not correctly run within the Docker container. The container does not actually have SVN installed, and the script was only working when the machine running the command had SVN present. Merges [51179] to the 5.3 branch. Reviewed by desrosj. Props czapla, alexstine, jnylen0, francina, desrosj. See #52909. See #62280. git-svn-id: https://develop.svn.wordpress.org/branches/5.3@59601 602fd350-edb4-49c9-b593-d223f7449a82 --- .env | 7 ------- tools/local-env/scripts/install.js | 5 +++-- 2 files changed, 3 insertions(+), 9 deletions(-) diff --git a/.env b/.env index ba45a296f5..f159bd7002 100644 --- a/.env +++ b/.env @@ -70,10 +70,3 @@ LOCAL_SCRIPT_DEBUG=true # The URL to use when running e2e tests. WP_BASE_URL=http://localhost:${LOCAL_PORT} - -## -# The revision number of the WordPress Importer plugin to use when running unit tests. -# -# This should be an SVN revision number from the official plugin repository on wordpress.org. -## -WP_IMPORTER_REVISION=2387243 diff --git a/tools/local-env/scripts/install.js b/tools/local-env/scripts/install.js index 208c6b47fc..6d95165d21 100644 --- a/tools/local-env/scripts/install.js +++ b/tools/local-env/scripts/install.js @@ -51,7 +51,8 @@ function wp_cli( cmd ) { * Downloads the WordPress Importer plugin for use in tests. */ function install_wp_importer() { - const test_plugin_directory = 'tests/phpunit/data/plugins/wordpress-importer'; + const testPluginDirectory = 'tests/phpunit/data/plugins/wordpress-importer'; - execSync( `docker compose exec -T php rm -rf ${test_plugin_directory} && svn checkout -r ${process.env.WP_IMPORTER_REVISION} https://plugins.svn.wordpress.org/wordpress-importer/trunk/ ${test_plugin_directory}`, { stdio: 'inherit' } ); + execSync( `docker compose exec -T php rm -rf ${testPluginDirectory}`, { stdio: 'inherit' } ); + execSync( `docker compose exec -T php git clone https://github.com/WordPress/wordpress-importer.git ${testPluginDirectory} --depth=1`, { stdio: 'inherit' } ); }