Skip to content

chore: Use no isolation for unit tests#16745

Merged
sean-brydon merged 7 commits intomainfrom
chore/unit-tests-no-isolate
Sep 20, 2024
Merged

chore: Use no isolation for unit tests#16745
sean-brydon merged 7 commits intomainfrom
chore/unit-tests-no-isolate

Conversation

@keithwillcode
Copy link
Copy Markdown
Contributor

@keithwillcode keithwillcode commented Sep 20, 2024

What does this PR do?

I noticed that the unit test runs take quite awhile to collect. This PR attempts to reduce that time by using --no-isolate and a 4vCPU machine.

transform 9.90s, setup 23.18s, collect 88.14s, tests 17.62s, environment 18.50s, prepare 8.70s

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have added a Docs issue here if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

@keithwillcode keithwillcode self-assigned this Sep 20, 2024
@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Sep 20, 2024 3:36pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Sep 20, 2024 3:36pm

@graphite-app graphite-app Bot requested a review from a team September 20, 2024 14:49
@keithwillcode keithwillcode added core area: core, team members only foundation labels Sep 20, 2024
@dosubot dosubot Bot added the automated-tests area: unit tests, e2e tests, playwright label Sep 20, 2024
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Sep 20, 2024

Graphite Automations

"Add foundation team as reviewer" took an action on this PR • (09/20/24)

1 reviewer was added to this PR based on Keith Williams's automation.

@keithwillcode keithwillcode changed the title chore: Use no isolation and pooling for unit tests chore: Use no isolation for unit tests Sep 20, 2024
@keithwillcode keithwillcode added this to the v4.6 milestone Sep 20, 2024
Copy link
Copy Markdown
Member

@sean-brydon sean-brydon left a comment

Choose a reason for hiding this comment

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

Took us a while to get the sweet spot!

LGTM

@github-actions
Copy link
Copy Markdown
Contributor

E2E results are ready!

@sean-brydon sean-brydon merged commit c1c4b12 into main Sep 20, 2024
@sean-brydon sean-brydon deleted the chore/unit-tests-no-isolate branch September 20, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-tests area: unit tests, e2e tests, playwright core area: core, team members only foundation ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants