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

fix(ci): Reduce the amount of test data that's stored #60197

Merged
merged 1 commit into from
Nov 21, 2023

Conversation

evanh
Copy link
Member

@evanh evanh commented Nov 17, 2023

This test stores a lot of test data, since it gives more granular results and
makes it easier to test different combinations of filters/group by etc.

However it also means the tests run super slowly, so reduce the amount of test
data to give this a bit of a speed boost.

This test stores a lot of test data, since it gives more granular results and
makes it easier to test different combinations of filters/group by etc.

However it also means the tests run super slowly, so reduce the amount of test
data to give this a bit of a boost.
Copy link

codecov bot commented Nov 17, 2023

Codecov Report

Merging #60197 (c967c3a) into master (efefa10) will increase coverage by 0.00%.
Report is 3 commits behind head on master.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #60197   +/-   ##
=======================================
  Coverage   80.82%   80.82%           
=======================================
  Files        5182     5182           
  Lines      227694   227699    +5     
  Branches    38329    38329           
=======================================
+ Hits       184030   184045   +15     
+ Misses      38039    38031    -8     
+ Partials     5625     5623    -2     

see 10 files with indirect coverage changes

@evanh evanh merged commit 2b44ed7 into master Nov 21, 2023
48 checks passed
@evanh evanh deleted the evanh/fix/speed-up-metrics-layer-test branch November 21, 2023 19:34
@github-actions github-actions bot locked and limited conversation to collaborators Dec 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants