Skip to content

test: SDL parity gate as a CI test (close remaining codecov gap)#100

Merged
chrisbc merged 1 commit into
deploy-testfrom
test/coverage-gap-2
Jun 26, 2026
Merged

test: SDL parity gate as a CI test (close remaining codecov gap)#100
chrisbc merged 1 commit into
deploy-testfrom
test/coverage-gap-2

Conversation

@chrisbc

@chrisbc chrisbc commented Jun 26, 2026

Copy link
Copy Markdown
Member

Follow-up to #99. Migration codecov on #97 was still a hair under the auto threshold (patch 95.51% vs 95.73%) because tools/schema_parity.py — the SDL byte-parity gate — ran as a standalone script at 0% coverage.

Wrap it as tests/test_sdl_parity.py (happy path + drift detection). schema_parity.py 0%→100%, package coverage 96%→98%. Side benefit: the SDL parity gate is now a permanent CI test, not a manual step.

tools/schema_parity.py (the Graphene→Strawberry SDL byte-parity gate) ran as a standalone
script, so it sat at 0% coverage and dragged the migration's codecov patch/project just under
the auto threshold on promote PR #97. Wrap it in tests/test_sdl_parity.py — happy path
(diff()=='' , main()==0) and a drift case (different baseline → non-empty diff, main()==1).

Bonus: the SDL parity gate is now a permanent CI test, not a manual invocation.
schema_parity.py 0% → 100%; package coverage 96% → 98%.
@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.54%. Comparing base (aa9a0e0) to head (02c4fc2).

Additional details and impacted files
@@               Coverage Diff               @@
##           deploy-test     #100      +/-   ##
===============================================
+ Coverage        95.08%   96.54%   +1.45%     
===============================================
  Files               12       12              
  Lines             1099     1099              
  Branches            63       63              
===============================================
+ Hits              1045     1061      +16     
+ Misses              41       25      -16     
  Partials            13       13              
Flag Coverage Δ
unittests 96.54% <ø> (+1.45%) ⬆️

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.

@chrisbc chrisbc merged commit c8d6b51 into deploy-test Jun 26, 2026
3 checks passed
@chrisbc chrisbc deleted the test/coverage-gap-2 branch June 26, 2026 02:24
chrisbc added a commit that referenced this pull request Jun 26, 2026
Record the deploy-test→main promote (#97, 5ed8586), prod deploy success, the pre-staged+closed
rollback #101, the 30-min watch (6/6, 0 failures), the #99/#100 coverage epilogue, and the
non-regression KeyError note (empty fault_system in unchanged shared compute).
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