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

[CI] Split access cohort3 tests into 2 groups #6839

Merged
merged 3 commits into from
Jan 1, 2025

Conversation

peterargue
Copy link
Contributor

Access Cohort3 integration tests are flaky and take a long time to run. sometimes flakiness is caused by too much load on the test runner.

Split test into 2 to reduce runtime and see if it will make the test less flaky

@peterargue peterargue requested a review from Kay-Zee as a code owner December 27, 2024 20:41
@codecov-commenter
Copy link

codecov-commenter commented Dec 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.14%. Comparing base (219660a) to head (3953304).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6839      +/-   ##
==========================================
+ Coverage   41.12%   41.14%   +0.01%     
==========================================
  Files        2107     2107              
  Lines      185330   185330              
==========================================
+ Hits        76220    76248      +28     
+ Misses     102710   102686      -24     
+ Partials     6400     6396       -4     
Flag Coverage Δ
unittests 41.14% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@peterargue peterargue added this pull request to the merge queue Jan 1, 2025
Merged via the queue into master with commit a7656ea Jan 1, 2025
56 checks passed
@peterargue peterargue deleted the petera/split-access-cohort3-integration-tests branch January 1, 2025 23:57
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.

4 participants