You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the suggested feature and problem it solves.
Right now we pass a parameter TestSettings into PoolManager tests to determine whether or not the we call take/settle or mint/burn during a swap. There are only two tests currently in PoolManager.t.sol which check the results of the mint/burn code path.
Describe the desired implementation.
Explore fuzzing of the TestSettings with conditional assertions or potentially remove the TestSettings logic and split out the PoolSwapRouter into PoolERC6909Router and PoolERC20Router, each of which inherit from PoolTestBase
Describe alternatives.
No response
Additional context.
No response
The text was updated successfully, but these errors were encountered:
Component
Tests
Describe the suggested feature and problem it solves.
Right now we pass a parameter
TestSettingsinto PoolManager tests to determine whether or not the we call take/settle or mint/burn during a swap. There are only two tests currently inPoolManager.t.solwhich check the results of the mint/burn code path.Describe the desired implementation.
Explore fuzzing of the TestSettings with conditional assertions or potentially remove the TestSettings logic and split out the
PoolSwapRouterintoPoolERC6909RouterandPoolERC20Router, each of which inherit fromPoolTestBaseDescribe alternatives.
No response
Additional context.
No response
The text was updated successfully, but these errors were encountered: