Skip to content

Test: Replace DsTestPlus with Forge Std Test#92

Merged
evchip merged 2 commits into
canaryfrom
tests/forge-std-test
Apr 14, 2025
Merged

Test: Replace DsTestPlus with Forge Std Test#92
evchip merged 2 commits into
canaryfrom
tests/forge-std-test

Conversation

@evchip

@evchip evchip commented Apr 11, 2025

Copy link
Copy Markdown
Contributor

Description

Replace antiquated DsTestPlus with far superior forge std library Test

Motivation and Context

forge std library's Test has more features

How Has This Been Tested?

All tests are passing

Types of changes (remove all unchecked types)

  • Refactor (code that does not add new functionality nor fixes a bug)

@evchip evchip requested a review from diego-G April 11, 2025 08:01
@evchip evchip self-assigned this Apr 11, 2025
diego-G
diego-G previously approved these changes Apr 11, 2025
Comment thread contracts/script/test/SwapERC20.s.sol
Comment thread contracts/src/test/L1GasPriceOracle.t.sol
@evchip evchip merged commit afea36a into canary Apr 14, 2025
@evchip evchip deleted the tests/forge-std-test branch April 14, 2025 08:55
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