Skip to content

Commit

Permalink
Remove Jest workaround for Babel 8 e2e test (#14224)
Browse files Browse the repository at this point in the history
  • Loading branch information
nicolo-ribaudo committed Feb 21, 2022
1 parent 79d6255 commit 2de3f1c
Showing 1 changed file with 2 additions and 16 deletions.
18 changes: 2 additions & 16 deletions scripts/integration-tests/e2e-babel.sh
Original file line number Diff line number Diff line change
Expand Up @@ -32,21 +32,7 @@ startLocalRegistry "$PWD"/scripts/integration-tests/verdaccio-config.yml
# and not from npm.
node "$PWD"/scripts/integration-tests/utils/bump-babel-dependencies.js

# Update deps, build and test
if [ "$BABEL_8_BREAKING" = true ] ; then
YARN_ENABLE_IMMUTABLE_INSTALLS=false make -j build-standalone-ci

# Jest hangs forever in the Babel 8 e2e test when using multiple workers,
# but we don't know yet why. Until we figure it out (see
# https://github.com/babel/babel/pull/13618) we can use --runInBand.
# Additinally, in Babel 8 tests Jest needs to run with ESM support because
# @babel/eslint-parser/lib/worker/babel-core.cjs uses dynamic import() to
# load @babel/core.
NODE_OPTIONS="--experimental-vm-modules" BABEL_ENV=test yarn jest --ci --runInBand

make -j test-clean
else
YARN_ENABLE_IMMUTABLE_INSTALLS=false make -j test-ci
fi
# Build and test
YARN_ENABLE_IMMUTABLE_INSTALLS=false make -j test-ci

cleanup

0 comments on commit 2de3f1c

Please sign in to comment.