Skip to content

feat: Add optional query parameter generate to aggregated-stats GET, update deps and fix flaky load test#90

Merged
skomski merged 4 commits into
mainfrom
kskomski/fix-flaky-load-test
Jul 23, 2025
Merged

feat: Add optional query parameter generate to aggregated-stats GET, update deps and fix flaky load test#90
skomski merged 4 commits into
mainfrom
kskomski/fix-flaky-load-test

Conversation

@skomski

@skomski skomski commented Jul 22, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@skomski
skomski requested a review from a team as a code owner July 22, 2025 12:51
@skomski
skomski force-pushed the kskomski/fix-flaky-load-test branch from 80349ed to 40f017a Compare July 22, 2025 13:12
@codecov

codecov Bot commented Jul 22, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 40.35088% with 34 lines in your changes missing coverage. Please review.

Project coverage is 50.98%. Comparing base (640e55b) to head (ffa81fc).
Report is 6 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/server.rs 43.75% 18 Missing ⚠️
src/database.rs 52.94% 8 Missing ⚠️
src/main.rs 0.00% 6 Missing ⚠️
src/lib.rs 0.00% 1 Missing ⚠️
src/settings.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #90      +/-   ##
==========================================
- Coverage   52.27%   50.98%   -1.30%     
==========================================
  Files           6        5       -1     
  Lines         329      253      -76     
==========================================
- Hits          172      129      -43     
+ Misses        157      124      -33     
Files with missing lines Coverage Δ
src/lib.rs 0.00% <0.00%> (ø)
src/settings.rs 0.00% <0.00%> (ø)
src/main.rs 0.00% <0.00%> (ø)
src/database.rs 65.62% <52.94%> (-5.69%) ⬇️
src/server.rs 61.68% <43.75%> (-7.43%) ⬇️

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 58cbea4...ffa81fc. Read the comment docs.

@skomski skomski changed the title tests: Fix flaky load test feat: Add optional query parameter generate to aggregated-stats GET, update deps and fix flaky load test Jul 22, 2025
Comment thread src/database.rs
Comment thread src/server.rs
Comment thread src/tests/mod.rs
@skomski
skomski requested a review from sirewix July 23, 2025 09:29
Comment thread config.sample.yaml
@skomski
skomski merged commit ffa81fc into main Jul 23, 2025
6 of 7 checks passed
@skomski
skomski deleted the kskomski/fix-flaky-load-test branch July 23, 2025 15:37
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