Skip to content

tests: Make Go coverage hermetic and split live contracts#9730

Open
nishantmonu51 wants to merge 8 commits into
mainfrom
nishant/test-audit-coverage-infra
Open

tests: Make Go coverage hermetic and split live contracts#9730
nishantmonu51 wants to merge 8 commits into
mainfrom
nishant/test-audit-coverage-infra

Conversation

@nishantmonu51

Copy link
Copy Markdown
Collaborator

Make the Go coverage signal reproducible on pull requests and main.

  • run the coverage and race suite in short mode against explicit product roots
  • mark credentialed connector fixtures as expensive
  • move live-provider contracts into a separate scheduled workflow
  • compact duplicate cross-package coverage blocks before upload
  • add Codecov flags and product/test-support exclusions
  • retain diagnostic artifacts while uploading only complete successful profiles

Validation:

  • go test -short -count=1 ./runtime -run '^TestDedicatedConnector$'
  • go test -short -count=1 ./runtime/drivers/druid -run '^TestScan$'
  • go test -short -count=1 ./runtime/testruntime/...
  • full baseline coverage run across admin, cli, runtime, and scripts

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. No documentation update is required.
  • Intend to cherry-pick into the release branch
  • I am proud of this work!

@nishantmonu51 nishantmonu51 changed the title Make Go coverage hermetic and split live contracts tests: Make Go coverage hermetic and split live contracts Jul 23, 2026
@nishantmonu51 nishantmonu51 added the tests Changes to tests and test infrastructure label Jul 23, 2026
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 34.52%. Comparing base (595f477) to head (e7c9c86).
⚠️ Report is 406 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (595f477) and HEAD (e7c9c86). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (595f477) HEAD (e7c9c86)
go 1 0
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #9730       +/-   ##
===========================================
- Coverage   44.98%   34.52%   -10.46%     
===========================================
  Files         726      754       +28     
  Lines       90172    93442     +3270     
===========================================
- Hits        40562    32262     -8300     
- Misses      43484    56318    +12834     
+ Partials     6126     4862     -1264     
Flag Coverage Δ
go ?
go-hermetic 34.52% <ø> (?)

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

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Changes to tests and test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant