Merge pull request #149 from rhinestonewtf/fix/associated-storage-tests

fix: fix unassociated storage
rhinestonewtf · Oct 23, 2024 · 3fa384d · 3fa384d
2 parents 14aac88 + 3c5342e
commit 3fa384d
Show file tree

Hide file tree

Showing 6 changed files with 46 additions and 15 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -19,6 +19,9 @@ jobs:
       foundry-fuzz-runs: 5000
       foundry-profile: "test"
       match-path: "test/**/*.sol"
+      foundry-verbosity: 3
+      foundry-gas-limit: "18446744073709551615"
+      foundry-memory-limit: 2147483648
     secrets:
       MAINNET_RPC_URL: ${{ secrets.MAINNET_RPC_URL }}
       TESTNET_RPC_URL: ${{ secrets.TESTNET_RPC_URL }}
@@ -30,6 +33,9 @@ jobs:
       foundry-fuzz-runs: 5000
       foundry-profile: "test"
       match-path: "test/**/*.sol"
+      foundry-verbosity: 3
+      foundry-gas-limit: "18446744073709551615"
+      foundry-memory-limit: 2147483648
     secrets:
       MAINNET_RPC_URL: ${{ secrets.MAINNET_RPC_URL }}
       TESTNET_RPC_URL: ${{ secrets.TESTNET_RPC_URL }}
@@ -41,6 +47,9 @@ jobs:
       foundry-fuzz-runs: 5000
       foundry-profile: "test"
       match-path: "test/**/*.sol"
+      foundry-verbosity: 3
+      foundry-gas-limit: "18446744073709551615"
+      foundry-memory-limit: 2147483648
     secrets:
       MAINNET_RPC_URL: ${{ secrets.MAINNET_RPC_URL }}
       TESTNET_RPC_URL: ${{ secrets.TESTNET_RPC_URL }}
diff --git a/foundry.toml b/foundry.toml
@@ -7,6 +7,9 @@ libs = ["node_modules"]
 fs_permissions = [{ access = "read", path = "out-optimized" }, { access = "read-write", path = "gas_calculations" }]
 allow_paths = ["*", "/"]
 ignored_warnings_from = ["node_modules", "src/integrations"]
+gas_limit = "18446744073709551615"
+memory_limit = 2147483648
+verbosity = 3
 
 [rpc_endpoints]
 mainnet = "${MAINNET_RPC_URL}" 

diff --git a/package.json b/package.json
@@ -30,8 +30,8 @@
     "@ERC4337/account-abstraction": "github:kopy-kat/account-abstraction#develop",
     "@ERC4337/account-abstraction-v0.6": "github:eth-infinitism/account-abstraction#v0.6.0",
     "@prb/math": "^4.0.2",
-    "@rhinestone/erc4337-validation": "^0.0.1-alpha.5",
-    "@rhinestone/module-bases": "github:rhinestonewtf/module-bases",
+    "@rhinestone/erc4337-validation": "^0.0.2",
+    "@rhinestone/module-bases": "github:rhinestonewtf/module-bases#d048ec28c8ea8b4155db3ce4f027bc64cd41f9a7",
     "@rhinestone/safe7579": "github:rhinestonewtf/safe7579#v1.0.0",
     "@rhinestone/sentinellist": "github:rhinestonewtf/sentinellist",
     "@rhinestone/registry": "github:rhinestonewtf/registry#v1.0",

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/test/integrations/SmartSession.t.sol b/test/integrations/SmartSession.t.sol
@@ -56,7 +56,7 @@ contract SmartSessionTest is BaseTest {
         // Deploy mock policy
         mockPolicy = new MockPolicy();
         // Set the policy to allow any action
-        mockPolicy.setValidationData(0);
+        mockPolicy.setValidationData(address(instance.account), 0);
         // Deploy mock target
         target = new MockTarget();
     }

diff --git a/test/mocks/MockValidatorFalse.sol b/test/mocks/MockValidatorFalse.sol
@@ -43,4 +43,17 @@ contract MockValidatorFalse is ERC7579ValidatorBase {
     function isInitialized(address smartAccount) external pure returns (bool) {
         return false;
     }
+
+    function validateSignatureWithData(
+        bytes32,
+        bytes calldata,
+        bytes calldata
+    )
+        external
+        view
+        override
+        returns (bool validSig)
+    {
+        return false;
+    }
 }