Add missing vm.assume()s to setUpAddressesAndBounds function

gildlab · Oct 15, 2024 · 2ad951d · 2ad951d
1 parent 6f31029
commit 2ad951d
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/test/src/concrete/vault/OffchainAssetReceiptVault.handler.t.sol b/test/src/concrete/vault/OffchainAssetReceiptVault.handler.t.sol
@@ -25,6 +25,8 @@ contract OffchainAssetReceiptVaultHandlerTest is OffchainAssetReceiptVaultTest {
         alice = vm.addr((fuzzedKeyAlice % (SECP256K1_ORDER - 1)) + 1);
         bob = vm.addr((fuzzedKeyBob % (SECP256K1_ORDER - 1)) + 1);
         john = vm.addr((fuzzedKeyJohn % (SECP256K1_ORDER - 1)) + 1);
+        vm.assume(alice != bob && alice != john);
+        vm.assume(bob != john);
 
         balance = bound(balance, 1, type(uint256).max); // Bound from one to avoid ZeroAssets
         referenceBlockNumber = bound(referenceBlockNumber, 1, block.number);
@@ -173,10 +175,6 @@ contract OffchainAssetReceiptVaultHandlerTest is OffchainAssetReceiptVaultTest {
         // due to a handler role
         futureTimeStamp = bound(futureTimeStamp, certifyUntil + 1, type(uint32).max);
 
-        vm.assume(alice != bob);
-        vm.assume(alice != john);
-        vm.assume(bob != john);
-
         OffchainAssetReceiptVault vault;
         ReceiptContract receipt;
         (vault, receipt) = setUpVault(alice, assetName, assetName);