Skip to content

Fix merged coverage statistics#44

Merged
samuel-williams-shopify merged 4 commits into
mainfrom
fix-statistics-merged-coverage
Jun 20, 2026
Merged

Fix merged coverage statistics#44
samuel-williams-shopify merged 4 commits into
mainfrom
fix-statistics-merged-coverage

Conversation

@samuel-williams-shopify

@samuel-williams-shopify samuel-williams-shopify commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • keep aggregate statistics cached while updating them from merged per-path coverage deltas
  • avoid double-counting duplicate path coverage records from persisted multi-process runs
  • add a regression test for overlapping coverage from the same source path

Tests

  • bundle exec sus test/covered/statistics.rb
  • bundle exec bake test
  • git diff --check

@samuel-williams-shopify samuel-williams-shopify force-pushed the fix-statistics-merged-coverage branch from dbc1340 to f10c598 Compare June 20, 2026 05:05
@samuel-williams-shopify samuel-williams-shopify force-pushed the fix-statistics-merged-coverage branch 7 times, most recently from a10d78d to 56cf79a Compare June 20, 2026 09:34
@samuel-williams-shopify samuel-williams-shopify force-pushed the fix-statistics-merged-coverage branch from 56cf79a to 645aa38 Compare June 20, 2026 09:35
@samuel-williams-shopify samuel-williams-shopify merged commit f0a4597 into main Jun 20, 2026
22 of 28 checks passed
@samuel-williams-shopify samuel-williams-shopify deleted the fix-statistics-merged-coverage branch June 20, 2026 09:38
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.

1 participant