Skip to content

test: add SignedFill EIP-712 conformity vectors#184

Merged
prestwich merged 1 commit intomainfrom
fill-test-vectors
Feb 3, 2026
Merged

test: add SignedFill EIP-712 conformity vectors#184
prestwich merged 1 commit intomainfrom
fill-test-vectors

Conversation

@prestwich
Copy link
Member

Summary

  • Add fill-specific test vectors for TypeScript SDK verification
  • Fill vectors represent filler authorization - the filler authorizes transfer of tokens they provide to satisfy order outputs

Test cases:

  • minimal_fill: all zeros base case
  • mainnet_eth_fill: filler providing WETH on mainnet
  • multi_token_fill: filler providing multiple tokens
  • cross_chain_fill: outputs to mainnet and rollup
  • large_amount_fill: amounts exceeding JS safe integer
  • signet_rollup_fill: fill on Signet rollup (chain 519)

Usage

Generate vectors with:

cargo t -p signet-types fill_signing_vectors -- --nocapture --ignored

Test plan

  • cargo t -p signet-types passes
  • cargo clippy -p signet-types --all-features --all-targets passes

🤖 Generated with Claude Code

@prestwich prestwich requested a review from a team as a code owner February 3, 2026 18:42
@prestwich
Copy link
Member Author

code was used to generate table tests for the TS SDK

Add fill-specific test vectors for TypeScript SDK verification.
Fill vectors represent filler authorization - the filler authorizes
transfer of tokens they provide to satisfy order outputs.

Test cases:
- minimal_fill: all zeros base case
- mainnet_eth_fill: filler providing WETH on mainnet
- multi_token_fill: filler providing multiple tokens
- cross_chain_fill: outputs to mainnet and rollup
- large_amount_fill: amounts exceeding JS safe integer
- signet_rollup_fill: fill on Signet rollup (chain 519)

Generate vectors with:
cargo t -p signet-types fill_signing_vectors -- --nocapture --ignored

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@prestwich prestwich enabled auto-merge (squash) February 3, 2026 21:45
@prestwich prestwich merged commit 5caacbe into main Feb 3, 2026
6 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.

2 participants