test(contracts): enhance LegacyMintableERC20 test coverage #19041
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.
test(contracts): enhance LegacyMintableERC20 test coverage
Summary
Enhances the
LegacyMintableERC20test file with improved coverage and quality:mint/burnsuccess and revert cases) to test with randomized addresses and amountsvm.expectEmitchecks forMintandBurneventssupportsInterfacereturningfalsefor unsupported interface IDsworks→succeeds) and added descriptive scenario namesReview & Testing Checklist for Human
vm.assume(_to != address(0))for mint,vm.assume(_caller != address(l2StandardBridge))for non-bridge testsTest plan: Run
just test-dev --match-path test/legacy/LegacyMintableERC20.t.sol -vto verify all 7 tests pass.Notes
Link to Devin run: https://app.devin.ai/sessions/079e16162e3845be98a3418bff98a28f
Requested by: Ariel Diaz (@aliersh)
Last update: 2026-02-02 02:15 ET