fix(test): bound instead of assume

Rubilmax · Rubilmax · commit 25c4ab6137ce · 2023-11-09T15:24:30.000+01:00
diff --git a/test/ChainlinkOracleTest.sol b/test/ChainlinkOracleTest.sol
@@ -138,7 +138,7 @@ contract ChainlinkOracleTest is Test {
     }
 
     function testConstructorVaultZeroNonOneSample(uint256 vaultConversionSample) public {
-        vm.assume(vaultConversionSample > 1);
+        vaultConversionSample = bound(vaultConversionSample, 2, type(uint256).max);
 
         vm.expectRevert(bytes(ErrorsLib.VAULT_CONVERSION_SAMPLE_IS_NOT_ONE));
         new ChainlinkOracle(vaultZero, daiEthFeed, feedZero, usdcEthFeed, feedZero, vaultConversionSample, 18, 6);

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ contract ChainlinkOracleTest is Test {`
`138`	`138`	`}`
`139`	`139`
`140`	`140`	`function testConstructorVaultZeroNonOneSample(uint256 vaultConversionSample) public {`
`141`		`- vm.assume(vaultConversionSample > 1);`
	`141`	`+ vaultConversionSample = bound(vaultConversionSample, 2, type(uint256).max);`
`142`	`142`
`143`	`143`	`vm.expectRevert(bytes(ErrorsLib.VAULT_CONVERSION_SAMPLE_IS_NOT_ONE));`
`144`	`144`	`new ChainlinkOracle(vaultZero, daiEthFeed, feedZero, usdcEthFeed, feedZero, vaultConversionSample, 18, 6);`