Skip to content

Test malformed HMMA helper signatures - #870

Open
thehobbit85 wants to merge 1 commit into
seedds:mainfrom
thehobbit85:codex/fix-heaps-fixture-types
Open

thehobbit85 wants to merge 1 commit into
seedds:mainfrom
thehobbit85:codex/fix-heaps-fixture-types

Conversation

@thehobbit85

@thehobbit85 thehobbit85 commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

Add regression coverage for malformed HMMA helper signatures after upstream 0e8503e7 incorporated the production guards and Heaps fixture assertion. Main fixes the original lint failures but lacks these malformed ordinary/scratch parameter-name and output-reference cases. Each must raise ValueError identifying the bad parameter; the tests detect removal of either guard.

Revision 7dfd0420451bd1d90d8a441ecb093c45a3429408 includes main 65501fbbaf8526f7aab0a1edd028135888590f8e. The malformed HMMA signature fixtures are still absent from main.

Validation on 7dfd0420451b. python3 -m unittest dm.tools.tests.test_docs dm.gfx.tests.test_dmsh_hmma — 17 tests passed.

GitHub checks for this revision

@thehobbit85
thehobbit85 force-pushed the codex/fix-heaps-fixture-types branch from 495c412 to eb7211a Compare November 1, 2026 13:33
@thehobbit85 thehobbit85 changed the title tests: assert successful Heaps parsing before inspecting entries Fix shared Heaps fixture and HMMA parser validation Nov 1, 2026
@thehobbit85
thehobbit85 force-pushed the codex/fix-heaps-fixture-types branch from eb7211a to 162bc26 Compare November 1, 2026 13:54
@thehobbit85 thehobbit85 changed the title Fix shared Heaps fixture and HMMA parser validation Test malformed HMMA helper signatures Nov 1, 2026
@thehobbit85
thehobbit85 force-pushed the codex/fix-heaps-fixture-types branch 6 times, most recently from ede3d68 to 7f9e1e2 Compare November 2, 2026 15:49
@thehobbit85
thehobbit85 force-pushed the codex/fix-heaps-fixture-types branch from 7f9e1e2 to 7dfd042 Compare November 2, 2026 17:44
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