Skip to content

Commit

Permalink
Merge branch '234-make-sure-that-assume-excludes-the-possibility-of-i…
Browse files Browse the repository at this point in the history
…ncoming-parameters-matching-with-sentinel' into 2024-09-13-FlowERC1155PreviewDefinedFlowIOForERC20MultiElementArraysTest
  • Loading branch information
Erikd-dev committed Sep 13, 2024
2 parents e99bfa2 + a186203 commit 25bd271
Show file tree
Hide file tree
Showing 9 changed files with 20 additions and 0 deletions.
1 change: 1 addition & 0 deletions test/concrete/flowErc1155/FlowExpressionTest.sol
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,7 @@ contract FlowExpressionTest is FlowERC1155Test {
) public {
vm.assume(!alice.isContract());
vm.assume(alice != address(0));
vm.assume(sentinel != amount);

uint256[][] memory matrixCallerContext =
fuzzedcallerContext0.matrixFrom(fuzzedcallerContext1, fuzzedcallerContext0);
Expand Down
1 change: 1 addition & 0 deletions test/concrete/flowErc1155/FlowPreviewTest.sol
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ contract FlowPreviewTest is FlowERC1155Test {
function testFlowERC1155PreviewEmptyFlowIO(string memory uri, address alice, uint256 amount) public {
(IFlowERC1155V5 flow,) = deployIFlowERC1155V5({uri: uri});
assumeEtchable(alice, address(flow));
vm.assume(sentinel != amount);

ERC1155SupplyChange[] memory mints = new ERC1155SupplyChange[](2);
mints[0] = ERC1155SupplyChange({account: alice, id: 1, amount: amount});
Expand Down
6 changes: 6 additions & 0 deletions test/concrete/flowErc1155/FlowSignedContextTest.sol
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,9 @@ contract FlowSignedContextTest is FlowUtilsAbstractTest, FlowERC1155Test {
uint256 fuzzedKeyBob
) public {
vm.assume(fuzzedKeyBob != fuzzedKeyAlice);
vm.assume(sentinel != amount);
vm.assume(sentinel != id);

(IFlowERC1155V5 erc1155Flow, EvaluableV2 memory evaluable) = deployIFlowERC1155V5(uri);

// Ensure the fuzzed key is within the valid range for secp256k1
Expand Down Expand Up @@ -84,6 +87,9 @@ contract FlowSignedContextTest is FlowUtilsAbstractTest, FlowERC1155Test {
uint256 amount
) public {
vm.assume(fuzzedKeyBob != fuzzedKeyAlice);
vm.assume(sentinel != amount);
vm.assume(sentinel != id);

(IFlowERC1155V5 erc1155Flow, EvaluableV2 memory evaluable) = deployIFlowERC1155V5(uri);

// Ensure the fuzzed key is within the valid range for secp256k1
Expand Down
4 changes: 4 additions & 0 deletions test/concrete/flowErc1155/FlowTest.sol
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ contract Erc1155FlowTest is FlowERC1155Test {
vm.assume(sentinel != erc721InTokenId);
vm.assume(sentinel != erc1155OutTokenId);
vm.assume(sentinel != erc1155OutAmount);
vm.assume(sentinel != id);
vm.assume(address(0) != alice);
vm.assume(!alice.isContract());

Expand Down Expand Up @@ -106,6 +107,7 @@ contract Erc1155FlowTest is FlowERC1155Test {
) external {
vm.assume(sentinel != erc1155OutAmount);
vm.assume(sentinel != erc20InAmount);
vm.assume(sentinel != id);

// Ensure the fuzzed key is within the valid range for secp256k1
uint256 aliceKey = (fuzzedKeyAlice % (SECP256K1_ORDER - 1)) + 1;
Expand Down Expand Up @@ -165,6 +167,7 @@ contract Erc1155FlowTest is FlowERC1155Test {

vm.assume(sentinel != erc721OutTokenId);
vm.assume(sentinel != erc721InTokenId);
vm.assume(sentinel != amount);

(IFlowERC1155V5 erc1155Flow, EvaluableV2 memory evaluable) = deployIFlowERC1155V5(uri);
assumeEtchable(alice, address(erc1155Flow));
Expand Down Expand Up @@ -347,6 +350,7 @@ contract Erc1155FlowTest is FlowERC1155Test {
) external {
vm.assume(alice != address(0));
vm.assume(sentinel != amount);
vm.assume(sentinel != id);
vm.assume(expressionA != expressionB);
vm.assume(writeToStore.length != 0);
vm.assume(!alice.isContract());
Expand Down
2 changes: 2 additions & 0 deletions test/concrete/flowErc1155/FlowTimeTest.sol
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@ contract FlowTimeTest is FlowUtilsAbstractTest, FlowERC1155Test {
{
vm.assume(alice != address(0));
vm.assume(amount != 0);
vm.assume(sentinel != amount);
vm.assume(sentinel != id);
vm.assume(writeToStore.length != 0);
vm.assume(!alice.isContract());

Expand Down
1 change: 1 addition & 0 deletions test/concrete/flowErc721/FlowExpressionTest.sol
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,7 @@ contract FlowExpressionTest is FlowERC721Test {
) public {
vm.assume(alice != address(0));
vm.assume(!alice.isContract());
vm.assume(sentinel != id);

uint256[][] memory matrixCallerContext =
fuzzedcallerContext0.matrixFrom(fuzzedcallerContext1, fuzzedcallerContext0);
Expand Down
2 changes: 2 additions & 0 deletions test/concrete/flowErc721/FlowSignedContextTest.sol
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ contract FlowSignedContextTest is FlowERC721Test {
uint256 id
) public {
vm.assume(fuzzedKeyBob != fuzzedKeyAlice);
vm.assume(sentinel != id);
(IFlowERC721V5 erc721Flow, EvaluableV2 memory evaluable) =
deployFlowERC721({name: name, symbol: symbol, baseURI: baseURI});

Expand Down Expand Up @@ -91,6 +92,7 @@ contract FlowSignedContextTest is FlowERC721Test {
uint256 id
) public {
vm.assume(fuzzedKeyBob != fuzzedKeyAlice);
vm.assume(sentinel != id);
(IFlowERC721V5 erc721Flow, EvaluableV2 memory evaluable) =
deployFlowERC721({name: name, symbol: symbol, baseURI: baseURI});

Expand Down
2 changes: 2 additions & 0 deletions test/concrete/flowErc721/FlowTest.sol
Original file line number Diff line number Diff line change
Expand Up @@ -328,6 +328,7 @@ contract Erc721FlowTest is FlowERC721Test {

vm.assume(sentinel != erc721OutTokenId);
vm.assume(sentinel != erc721InTokenId);
vm.assume(sentinel != id);

(IFlowERC721V5 erc721Flow, EvaluableV2 memory evaluable) =
deployFlowERC721({name: "FlowErc721", symbol: "FErc721", baseURI: baseURI});
Expand Down Expand Up @@ -385,6 +386,7 @@ contract Erc721FlowTest is FlowERC721Test {
vm.assume(alice != address(0));
vm.assume(expressionA != expressionB);
vm.assume(writeToStore.length != 0);
vm.assume(sentinel != tokenId);

address[] memory expressions = new address[](1);
expressions[0] = expressionA;
Expand Down
1 change: 1 addition & 0 deletions test/concrete/flowErc721/FlowTimeTest.sol
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ contract FlowTimeTest is FlowERC721Test {
) public {
vm.assume(writeToStore.length != 0);
vm.assume(alice != address(0));
vm.assume(sentinel != id);
vm.assume(!alice.isContract());

(IFlowERC721V5 erc1155Flow, EvaluableV2 memory evaluable) = deployFlowERC721(name, symbol, uri);
Expand Down

0 comments on commit 25bd271

Please sign in to comment.