Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: migrate GPv2Signing recoverOrderSigner tests to Foundry #207

Merged
merged 30 commits into from
Aug 15, 2024
Merged
Changes from 8 commits
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
d0173f0
chore: migrate GPv2Signing domain separator tests to Foundry
fedgiac Aug 9, 2024
c7d4149
Fix unused import
fedgiac Aug 9, 2024
bc7f235
Remove duplicated import
fedgiac Aug 12, 2024
0721892
chore: migrate GPv2Signing set pre-signature tests to Foundry
fedgiac Aug 12, 2024
56ff598
chore: migrate GPv2Signing recoverOrderFromTrade tests to Foundry
fedgiac Aug 12, 2024
f71e28d
Merge branch 'main' into migrate-test-signing-domain-separator
fedgiac Aug 12, 2024
2816827
Add explanatory comment on domain separator struct
fedgiac Aug 12, 2024
585703d
Reuse existing PRE_SIGNED variable from library
fedgiac Aug 12, 2024
d51fced
Merge branch 'main' into migrate-test-signing-domain-separator
fedgiac Aug 12, 2024
e233612
Merge branch 'migrate-test-signing-domain-separator' into migrate-tes…
fedgiac Aug 12, 2024
84eb7fc
Fix missing setting of pre-signature at start of test
fedgiac Aug 12, 2024
6e618dc
Clean up imports
fedgiac Aug 14, 2024
a8c8b45
chore: migrate GPv2Signing calldata manipulation tests to Foundry
fedgiac Aug 14, 2024
ac6bcae
chore: migrate GPv2Signing recoverOrderSigner tests to Foundry
fedgiac Aug 14, 2024
483e8f2
Fix fuzz test case where buy and sell tokens are the same
fedgiac Aug 14, 2024
61a0da9
Merge branch 'main' into migrate-test-signing-set-pre-signature
fedgiac Aug 15, 2024
a16ab67
Merge branch 'main' into migrate-test-signing-set-pre-signature
fedgiac Aug 15, 2024
2ebcc44
Merge branch 'migrate-test-signing-set-pre-signature' into migrate-te…
fedgiac Aug 15, 2024
8dd66e7
Merge fuzzed order library into order library
fedgiac Aug 15, 2024
21e12c1
Merge branch 'migrate-test-signing-recover-order-from-trade' into mig…
fedgiac Aug 15, 2024
46fcb81
Update fuzz library usage
fedgiac Aug 15, 2024
db2a861
Merge branch 'migrate-test-signing-calldata-manipulation' into migrat…
fedgiac Aug 15, 2024
16d5d65
Fix invalid signing scheme test to involve a call
fedgiac Aug 15, 2024
6e97dce
Prefer EIP1271Verifier.isValidSignature.selector to static constant
fedgiac Aug 15, 2024
cc09214
Merge branch 'main' into migrate-test-signing-recover-order-from-trade
fedgiac Aug 15, 2024
a34bc31
Merge branch 'migrate-test-signing-recover-order-from-trade' into mig…
fedgiac Aug 15, 2024
53ef88b
Merge branch 'main' into migrate-test-signing-calldata-manipulation
fedgiac Aug 15, 2024
6d96620
Merge branch 'migrate-test-signing-calldata-manipulation' into migrat…
fedgiac Aug 15, 2024
a467ea6
Remove test of Solidity enum parsing
fedgiac Aug 15, 2024
94bf104
Remove unused import
fedgiac Aug 15, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 0 additions & 5 deletions src/ts/sign.ts
Original file line number Diff line number Diff line change
@@ -24,11 +24,6 @@ export const EIP1271_MAGICVALUE = ethers.utils.hexDataSlice(
4,
);

/**
* Marker value indicating a presignature is set.
*/
export const PRE_SIGNED = ethers.utils.id("GPv2Signing.Scheme.PreSign");

/**
* The signing scheme used to sign the order.
*/
Loading