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.

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

Tests: Try to speed up CI tests execution #1827

Closed
wants to merge 3 commits into from

Conversation

gziolo
Copy link
Member

@gziolo gziolo commented Jul 10, 2017

From Tests are Extremely Slow on Docker and/or Continuous Integration (CI) server section from Jest Troubleshooting documentation:

Another alternative to expediting test execution time on Continuous Integration Servers such as Travis-CI is to set the max worker pool to ~4. Specifically on Travis-CI, this can reduce test execution time in half.

First try is to run all tests in the same thread using --runInBand (-i). On Travis with this change:

screen shot 2017-07-10 at 09 23 09

screen shot 2017-07-10 at 09 23 55

Second try is to run all tests with the max worker pool set to ~4. On Travis with this change:

screen shot 2017-07-10 at 09 35 12

screen shot 2017-07-10 at 09 35 26

Third try is to run all tests using unlimited number of workers as it is at the moment. Travis with this change:

screen shot 2017-07-10 at 09 47 32

screen shot 2017-07-10 at 09 46 45

@gziolo gziolo added [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. [Status] In Progress Tracking issues with work in progress labels Jul 10, 2017
@gziolo gziolo self-assigned this Jul 10, 2017
@gziolo gziolo requested review from nylen and ntwb July 10, 2017 07:22
@coveralls
Copy link

Coverage Status

Coverage remained the same at 17.816% when pulling efdb78e on update/run-jest-in-band-ci into d157b5e on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 17.816% when pulling 17f2d9b on update/run-jest-in-band-ci into d157b5e on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 17.816% when pulling 9ffd99d on update/run-jest-in-band-ci into d157b5e on master.

@gziolo
Copy link
Member Author

gziolo commented Jul 10, 2017

@nylen @ntwb It looks like we should limit number of workers used with Travis to run Jest. It looks like we should pick between 1-4 workers to improve performance. What would be your pick?

@nylen
Copy link
Member

nylen commented Jul 10, 2017

Let's go with 1 because we already have another task running during build, and Travis workers only have 2 cores. See #1808 for details.

@nylen nylen closed this Jul 10, 2017
@nylen nylen deleted the update/run-jest-in-band-ci branch July 10, 2017 09:53
@nylen
Copy link
Member

nylen commented Jul 10, 2017

Also, switching to npm 5 will get us another big speed boost (#1600).

Tug pushed a commit that referenced this pull request Feb 28, 2020
…sheet_for_media_editing_button

Fixes the Image's media button not displaying the Picker
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] In Progress Tracking issues with work in progress [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants