Skip to content

tests: re-mint expired grant fixtures, guard against the expiry cliff - #18

Merged
sumerjohal merged 2 commits into
mainfrom
fix/grant-fixture-expiry
Aug 27, 2026
Merged

sumerjohal merged 2 commits into
mainfrom
fix/grant-fixture-expiry

Conversation

@sumerjohal

Copy link
Copy Markdown
Contributor

tests: re-mint expired grant fixtures, guard against the expiry cliff

Every committed grant fixture expired on 2026-08-20 (30-day window,
minted mid-July). Since then test_valid_grant and three siblings fail on
every branch -- and worse, the tampered/revoked/wrong-geoid tests kept
passing vacuously: the verifier rejected those credentials for expiry,
not for the property under test. The conftest docstring claimed "no
expiry cliff"; that was true only of the authority credentials it mints
fresh each run.

Fixtures re-minted with a 365-day window via the pancake testkit minter
(needs agstack/pancake#8 for the --days flag). conftest now refuses to
run when any non-expired-by-design fixture is within 30 days of expiry,
and the error names the exact re-mint command. The guard was demonstrated
failing against the old fixtures before the re-mint (lessons ledger:
gate-reports-green-while-blind).

Every committed grant fixture expired on 2026-08-20 (30-day window,
minted mid-July). Since then test_valid_grant and three siblings fail on
every branch -- and worse, the tampered/revoked/wrong-geoid tests kept
passing vacuously: the verifier rejected those credentials for expiry,
not for the property under test. The conftest docstring claimed "no
expiry cliff"; that was true only of the authority credentials it mints
fresh each run.

Fixtures re-minted with a 365-day window via the pancake testkit minter
(needs agstack/pancake#8 for the --days flag). conftest now refuses to
run when any non-expired-by-design fixture is within 30 days of expiry,
and the error names the exact re-mint command. The guard was demonstrated
failing against the old fixtures before the re-mint (lessons ledger:
gate-reports-green-while-blind).

Co-authored-by: Cursor <cursoragent@cursor.com>
@sumerjohal

Copy link
Copy Markdown
Contributor Author

Guard demonstration (run before the re-mint, against the expired fixtures on main): all four fixtures reported -7 days of validity and the guard raised with the re-mint command; after re-minting with --days 365 it passes. Merge order: agstack/pancake#8 provides the --days flag this re-mint used, then this, then #17 re-runs green.

…while-blind

Co-authored-by: Cursor <cursoragent@cursor.com>
@sumerjohal
sumerjohal merged commit bfcb95a into main Aug 27, 2026
9 checks passed
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