Repository navigation
CI rehearsal only (do not merge) - #22
michael-moffett wants to merge 6 commits into
Conversation
An opt-in pytest module that builds N negative cases for `_validate_instruction_allowlist` from a fixed seed and checks that every case is refused. pay-kit item 2, in the shape the maintainer set: a pinned-seed table generator with no new dependency, kept out of default CI.
… python/tests/allowlist_generator.py, run opt-in via `just allowlist-gen`
 |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| def _drain(rng: random.Random) -> _Built: | ||
| base = _base(rng, *SPONSORED_SOL) | ||
| lamports = rng.choice((base.amount, base.amount - 1, base.amount + 1, 0, 2**64 - 1)) | ||
| return base, [_sol_transfer(base.fee_payer.pubkey(), base.recipient, lamports)] |
There was a problem hiding this comment.
_drain only builds SOL transfers, and BASES omits sponsored USDC. The new randomized suite therefore never checks a token payment authorized by the fee payer or funded from its token account. Existing fixed tests cover these checks, but the generator adds no coverage to that path. Add sponsored-USDC drain cases and a matching valid-payment control.
… python/tests/allowlist_generator.py, run opt-in via `just allowlist-gen`
There was a problem hiding this comment.
Your organization has used all 50 credits included in the free plan this billing period. To keep receiving reviews, upgrade your plan.
1 participant
Fork CI only. Do not merge.