Skip to content

Commit

Permalink
[DevTools] Add --replaceBuild option to Older React Builds Download S…
Browse files Browse the repository at this point in the history
…cript (facebook#24621)

This PR adds a `--replaceBuild` option to the script that downloads older React version builds. If this flag is true, we will replace the contents of the `build` folder with the contents of the `build-regression` folder and remove the `build-regression` folder after, which was the original behavior.

However, for e2e tests, we need both the original build (for DevTools) and the new build (for the React Apps), so we need both the `build` and the `build-regression` folders. Not adding the `--replaceBuild` option will do this.

This PR also modifies the circle CI config to reflect this change.
  • Loading branch information
lunaruan authored May 31, 2022
1 parent aec5759 commit f534cc6
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ jobs:
- run:
name: Install nested packages from Yarn cache
command: yarn --frozen-lockfile --cache-folder ~/.cache/yarn
- run: ./scripts/circleci/download_devtools_regression_build.js << parameters.version >>
- run: ./scripts/circleci/download_devtools_regression_build.js << parameters.version >> --replaceBuild
- run: node ./scripts/jest/jest-cli.js --build --project devtools --release-channel=experimental --reactVersion << parameters.version >> --ci

yarn_lint_build:
Expand Down
22 changes: 20 additions & 2 deletions scripts/circleci/download_devtools_regression_build.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ const {exec} = require('child-process-promise');
const chalk = require('chalk');
const {join} = require('path');
const semver = require('semver');
const yargs = require('yargs');
const fs = require('fs');

const INSTALL_PACKAGES = ['react-dom', 'react', 'react-test-renderer'];
Expand All @@ -16,7 +17,10 @@ const ROOT_PATH = join(__dirname, '..', '..');
const buildPath = join(ROOT_PATH, `build`, 'oss-experimental');
const regressionBuildPath = join(ROOT_PATH, REGRESSION_FOLDER);

const argv = yargs(process.argv.slice(2)).argv;

const version = process.argv[2];
const shouldReplaceBuild = !!argv.replaceBuild;

async function downloadRegressionBuild() {
console.log(chalk.bold.white(`Downloading React v${version}\n`));
Expand All @@ -39,6 +43,12 @@ async function downloadRegressionBuild() {
`npm install --prefix ${REGRESSION_FOLDER} ${downloadPackagesStr}`
);

// If we shouldn't replace the build folder, we can stop here now
// before we modify anything
if (!shouldReplaceBuild) {
return;
}

// Remove all the packages that we downloaded in the original build folder
// so we can move the modules from the regression build over
const removePackagesStr = INSTALL_PACKAGES.reduce(
Expand Down Expand Up @@ -102,12 +112,20 @@ async function downloadRegressionBuild() {

async function main() {
try {
if (!version) {
console.log(chalk.red('Must specify React version to download'));
return;
}
await downloadRegressionBuild();
} catch (e) {
console.log(chalk.red(e));
} finally {
console.log(chalk.bold.white(`Removing regression build`));
await exec(`rm -r ${regressionBuildPath}`);
// We shouldn't remove the regression-build folder unless we're using
// it to replace the build folder
if (shouldReplaceBuild) {
console.log(chalk.bold.white(`Removing regression build`));
await exec(`rm -r ${regressionBuildPath}`);
}
}
}

Expand Down

0 comments on commit f534cc6

Please sign in to comment.