-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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.
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
[ci] Update ci config to reduce javascript test time and some of cypress. #6016
Conversation
npm run build | ||
npm run cypress run --record | ||
npm run cypress run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason you removed --record
? It doesn't work atm but I'm trying to get it to work, unless we think we don't want to use cypress dashboard.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was trying to make it work via npm run cypress run -- --record
then cypress complains asking for --key
, which is required for recording. Will leave this for @michellethomas to handle in future review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is huge! thanks for digging into this and optimizing + catching huge issues like running tests 2x 😮
…ess. (apache#6016) * try travis config * try more complex config * fix config * add cache and stages * separate cache * swap order * fix redis * remove env * add cypress * adjust caching * reduce cache * change order of execution * change order again * change node version * install yarn via curl * update cypress command * update cypress command * adjust cache and node version * do not install yarn * add npm cache * remove export * remove commented code * change order of jobs * fix indent
Highlight: Reduce time to run javascript test from 8-10mins to 3-5 mins
Details
yarn install --frozen-lockfile
instead ofyarn install
as recommended by travis.javascript test
language: node_js
for the javascript test instead of running it via python.npm run test
becausenpm run cover
already run the test. Basically we have been running js test twice.npm run build
as the cypress test will always need to build. Seems redundant.cypress test
others
postgres
only for the tests that need postgres.@michellethomas @graceguo-supercat @mistercrunch @john-bodley