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

feat: Add jest shards in a11y tests #57

Merged
merged 1 commit into from
Sep 11, 2023
Merged

Conversation

taheramr
Copy link
Member

@taheramr taheramr commented Sep 7, 2023

Issue #, if available:

Description of changes:
Currently, the a11y tests takes around 30~35 minutes to be executed and it increases the waiting time for a PR to have a successful workflow action by almost 15 minutes as the highest tests that takes time after the a11y are the integ tests ( 15 mins )

This PR introduces jest shards, by which we divide the a11y into shards and each shard executes a subset of the tests in a separate machine. To create the shards mechanism we'll also have to merge the changes in the dry-run github workflow definition in this PR.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@taheramr taheramr requested a review from a team as a code owner September 7, 2023 10:19
@taheramr taheramr requested review from gethinwebster and removed request for a team September 7, 2023 10:19
@taheramr taheramr requested review from a team and connorlanigan and removed request for gethinwebster and a team September 7, 2023 10:24
@taheramr taheramr merged commit 1374982 into main Sep 11, 2023
19 checks passed
@taheramr taheramr deleted the feat/add-jest-shards-a11y branch September 11, 2023 10:32
just-boris pushed a commit that referenced this pull request Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants