Consume ASTTransform::Testing::Assertions for static line-alignment pins - #14
Merged
Conversation
JPDuchesne
force-pushed
the
jpd/consume-testing-assertions
branch
from
July 24, 2026 15:52
7b9177a to
e3db0ff
Compare
Each dialect fixture gains an in-process assert_line_aligned check through the same annotation pipeline the install hook runs. The Then-citation subprocess test is replaced outright: its runtime half was already pinned end-to-end by BacktraceSourceMappingTest. The remaining subprocess tests stay — they pin behavior only execution can prove (failure vs error counts, test-name embedding, ensure-time Cleanup output, locals visibility). Co-authored-by: Cursor <cursoragent@cursor.com>
JPDuchesne
force-pushed
the
jpd/consume-testing-assertions
branch
from
July 24, 2026 20:26
e3db0ff to
2601f6e
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adopts
ASTTransform::Testing::Assertions(from ast-transform's newtestingnamespace) inLineAlignmentTest— the first real consumption of the public test surface.assert_line_alignedpin throughASTTransform::Transformation— the exact annotation pipeline the install hook runs — in-process, no subprocess.BacktraceSourceMappingTestalready proves end-to-end.How much simplification did we get?
An honest accounting:
MISALIGNED <render>: source line N, emitted line Mplus a numbered emitted listing) instead of a subprocess transcript to eyeball, and they cover every surviving statement in each fixture rather than the one line each runtime test happens to cite.assert_line_alignedmatches statements by unparsed render, so statements RSpock rewrites (Then expressions -> assertions, interactions -> Mocha calls) or consumes (Where rows) are exempt from the static check. Alignment of the rewritten forms is still proven at runtime by the remaining subprocess tests andBacktraceSourceMappingTest. The helpers and the acceptance layer are complements, not substitutes.assert_backtrace_linesfound no consumer here: it evals compiled source directly and expects a top-level raise, while RSpock fixtures define test classes that only fail under a Minitest run.Test plan
dev testgreen: 239 tests (was 235), 0 failures.Made with Cursor