test: add comprehensive tests for ParseBytes, CustomRevert, CurrencyD…#1011
Open
amussara wants to merge 1 commit intoUniswap:mainfrom
Open
test: add comprehensive tests for ParseBytes, CustomRevert, CurrencyD…#1011amussara wants to merge 1 commit intoUniswap:mainfrom
amussara wants to merge 1 commit intoUniswap:mainfrom
Conversation
…elta, and CurrencyReserves libraries Add test suites for four previously untested libraries: - ParseBytes: 15 tests covering parseSelector, parseReturnDelta, parseFee with boundary values and fuzz tests for all three functions - CustomRevert: 18 tests covering all revertWith overloads (no-args, address, int24, uint160, two-arg variants) and bubbleUpAndRevertWith with ERC-7751 WrappedError encoding verification - CurrencyDelta: 13 tests covering getDelta defaults, applyDelta accumulation, net-to-zero, currency/address isolation, slot determinism, and boundary values - CurrencyReserves: 9 tests covering sync/reset lifecycle, slot constant verification, overwrite behavior, and boundary values All tests include fuzz testing for robustness.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…elta, and CurrencyReserves libraries
Add test suites for four previously untested libraries:
All tests include fuzz testing for robustness.
Related Issue
Which issue does this pull request resolve?
Description of changes