Skip to content

test(simulate): cover validate_verification_tests (EDD-07) - #5735

Closed
noahgift wants to merge 6 commits into
mainfrom
split/5588-sim
Closed

noahgift wants to merge 6 commits into
mainfrom
split/5588-sim

Conversation

@noahgift

Copy link
Copy Markdown
Contributor

One ticket: cover validate_verification_tests (EDD-07) in aprender-simulate, plus its contract (test/cov-simulate-top f2fd459). Split out of #5588 per "Do not fold tickets into PRs". Formals use declared Σ glyphs; nt, witness and README count regenerated.

Local: validate_contracts, edd::validation (53 tests) and the sigma gate pass.

keep-open: test-coverage ticket stays open until its owner confirms
ont-delta: none test-only change, no ontology delta

🤖 Generated with Claude Code

… contract

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

@noahgift noahgift left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seat 1, aprender-triage-worker @59a988e0d8d9e79c1a752d24ae36063958b42436 APPROVE
Read: PR diff (+97 -4, 5 files): three validate_verification_tests tests in aprender-simulate validation.rs (all-pass records nothing; failure is EDD-07 Critical with FAIL context; failures accumulate across runs), contract simulate-edd-verification-tests-v1.yaml, plus regenerated README/contracts.nt/witness. Same content I read inside #5588, split out. Test-only, no production change. 0 own-author trailers.
Not run: build, tests, validate_contracts, CI; regenerated derived files not re-derived.

@noahgift noahgift left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seat 1, aprender-pace @59a988e0d8d9e79c1a752d24ae36063958b42436

Read the diff (+97/-4): the only code change is three new tests and a helper (emc_with_tests) inside the existing tests module of aprender-simulate edd/validation.rs, plus the new contract simulate-edd-verification-tests-v1.yaml, whose formals carry the '∀ ... exit(t) = 0' form and whose test commands name the new tests. No production code changed. The helper uses .ok().unwrap() in test code. I did not check the asserted values (e.g. 4 accumulated violations) against validate_verification_tests. 0 aprender-pace trailers. No tests run by me, CI not read, generated files (contracts.nt, witness, README) not re-inspected.

@noahgift
noahgift enabled auto-merge October 19, 2026 06:49
@github-actions

github-actions Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

§13.11 rung 1 — quorum shadow verdict

S13-SHADOW pr=5735 head=e39cc24123773dddc30ce1ac7357e4aecbf37c62 verdict=REFUSE class=Q1 arm_rc=1

Shadow mode: this records a verdict and merges nothing. A refusal
to arm is not a block (§13 adds zero rows to §7) — the pull request is
exactly as green as it was.

# Conflicts:
#	contracts/witness/811c78184eb2609244c59534f015f0be2f90923120a3027c176ad56723d0d850.json
noahgift and others added 4 commits October 19, 2026 20:23
…rging main

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
# Conflicts:
#	README.md
#	contracts/witness/811c78184eb2609244c59534f015f0be2f90923120a3027c176ad56723d0d850.json
@noahgift
noahgift marked this pull request as draft October 20, 2026 12:45
@noahgift

Copy link
Copy Markdown
Contributor Author

Closing per operator 13:18Z (relayed by aprender-57): superseded/folded drafts close with evidence. Folded into main by #5765 (d5dedf1 'batch(manual-1615): fold 6 ready branches (#5735 #5592 #5586 #5657 #5672 #5762)'). Evidence: this head's tree is identical to main at its merge-base d1861e7 (the PR adds nothing), and validate_verification_tests first appears on main in d5dedf1.

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