From 28486bf203867ddd91ab1f9c66b10efa59f14f66 Mon Sep 17 00:00:00 2001 From: Dan Oved Date: Wed, 10 Apr 2024 10:25:52 -0400 Subject: [PATCH] fix fuzz failures on protocol rewards test (#347) ## Description ## Motivation and Context ## Does this change the ABI/API? - [ ] This changes the ABI/API ## What tests did you add/modify to account for these changes ## Types of changes - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New module / feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) ## Checklist: - [ ] My code follows the code style of this project. - [ ] My change requires a change to the documentation. - [ ] I have updated the documentation accordingly. - [ ] i added a changeset to account for this change ## Reviewer Checklist: - [ ] My review includes a symposis of the changes and potential issues - [ ] The code style is enforced - [ ] There are no risky / concerning changes / additions to the PR --- packages/protocol-rewards/test/unit/ERC1155Rewards.t.sol | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/protocol-rewards/test/unit/ERC1155Rewards.t.sol b/packages/protocol-rewards/test/unit/ERC1155Rewards.t.sol index a8e6f63a..57ac805e 100644 --- a/packages/protocol-rewards/test/unit/ERC1155Rewards.t.sol +++ b/packages/protocol-rewards/test/unit/ERC1155Rewards.t.sol @@ -170,8 +170,8 @@ contract ERC1155RewardsTest is ProtocolRewardsTest { ); } - function testRemainderSentToZoraPaidMint(uint256 rewardPrice, uint256 numTokens, uint256 tokenPrice) public { - vm.assume(rewardPrice > 0.0000002 ether && rewardPrice < 2 ether); + function testRemainderSentToZoraPaidMint(uint128 rewardPrice, uint256 numTokens, uint256 tokenPrice) public { + vm.assume(rewardPrice > 0.0000002 ether && rewardPrice < 100 ether); vm.assume(tokenPrice > 0 ether && tokenPrice < 5 ether); vm.assume(numTokens > 0 && numTokens < 100_000);