Skip to content

test: throwaway red-first run for the checkpoints signatures test, not for merging - #483

Closed
TAIPANBOX wants to merge 2 commits into
mainfrom
test/checkpoint-red-throwaway
Closed

TAIPANBOX wants to merge 2 commits into
mainfrom
test/checkpoint-red-throwaway

Conversation

@TAIPANBOX

Copy link
Copy Markdown
Owner

The new test from test/checkpoint-statement-needs-signatures, run against check_statement with the presence check on signatures disabled; the job passes only if the test fails. ci.yml is removed on this branch only. Closed and deleted once read.

🤖 Generated with Claude Code

TAIPANBOX and others added 2 commits October 26, 2026 04:53
check_statement requires a non-empty `signatures` list before it reads a
statement (the witness verifies the signatures, invariant 128), but no
test served a statement without one, so a mutant that dropped the check
survived the p256 0.14 PR's mutation run (#480). The keyryx stub gains
two modes, no `signatures` and an empty list, and both must be refused
as not of the known shape. One scenario, bound.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ver merged

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@TAIPANBOX

Copy link
Copy Markdown
Owner Author

Red first read: with the presence check disabled, a_statement_with_no_signatures_is_refused failed at checkpoints/tests.rs:568 (run 40271390648). Closing the throwaway; the test lands in #484 with the check intact.

@TAIPANBOX TAIPANBOX closed this Oct 26, 2026
@TAIPANBOX
TAIPANBOX deleted the test/checkpoint-red-throwaway branch October 26, 2026 05:05
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