Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Run jest's tests in the e2e tests #12202

Merged
merged 13 commits into from
Oct 16, 2020
Merged

Run jest's tests in the e2e tests #12202

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
5361232
Run jest't tests in the e2e tests
nicolo-ribaudo Oct 16, 2020
ba64350
Fix permission
nicolo-ribaudo Oct 16, 2020
bf20a34
Use "*" in peerDependencies rather than "latest"
nicolo-ribaudo Oct 16, 2020
9495637
Move to /tmp to prevent yarn from merging jest's .yarnrc.yml with ours
nicolo-ribaudo Oct 16, 2020
f3945f7
Update
nicolo-ribaudo Oct 16, 2020
19d1979
Skip .d.ts build since it's failing
nicolo-ribaudo Oct 16, 2020
57aeb03
Dedupe @babel/types deps
nicolo-ribaudo Oct 16, 2020
d7c56d0
Don't fail for type errors
nicolo-ribaudo Oct 16, 2020
b1d9aac
Only dedupe babel deps
nicolo-ribaudo Oct 16, 2020
086a109
Run corrects tests
nicolo-ribaudo Oct 16, 2020
cc33298
Only run some tests
nicolo-ribaudo Oct 16, 2020
11f58c8
Update scripts/integration-tests/e2e-jest.sh
nicolo-ribaudo Oct 16, 2020
1e40b57
Update scripts/integration-tests/e2e-jest.sh
nicolo-ribaudo Oct 16, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
10 changes: 8 additions & 2 deletions scripts/integration-tests/e2e-jest.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,13 @@ yarn install
yarn dedupe '@babel/*'
yarn build

# Test
CI=true yarn test-ci-partial
# The full test suite take about 20mins on CircleCI. We run only a few of them
nicolo-ribaudo marked this conversation as resolved.
Show resolved Hide resolved
# to speed it up.
# The goals of this e2e test are:
# 1) Check that the typescript compilation isn't completely broken
# 2) Make sure that we don't accidentally break jest's usage or the Babel API
nicolo-ribaudo marked this conversation as resolved.
Show resolved Hide resolved
CI=true yarn test-ci-partial packages
CI=true yarn test-ci-partial e2e/__tests__/babel
CI=true yarn test-ci-partial e2e/__tests__/transform
Comment on lines +50 to +52
Copy link
Contributor

@SimenB SimenB Oct 20, 2020 •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could do yarn test-ci-partial --ci packages e2e/__tests__/babel e2e/__tests__/transform?

FWIW I just landed jestjs/jest#10653 which adds test-ci-partial:parallel (which just removes the -i argument) so if you have multiple cores available on CI you can speed it up.

yarn test-ci-partial:parallel --ci --max-workers $(nproc) packages e2e/__tests__/babel e2e/__tests__/transform or something like that


cleanup