fix geth script, coverage test

eth-infinitism · Jan 29, 2025 · e54c2a7 · e54c2a7
1 parent bac3113
commit e54c2a7
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 3 deletions.
diff --git a/contracts/core/EntryPointSimulations.sol b/contracts/core/EntryPointSimulations.sol
@@ -192,7 +192,7 @@ contract EntryPointSimulations is EntryPoint, IEntryPointSimulations {
 
     //slightly stricter gas limit than the real EntryPoint
     function _getVerificationGasLimit(uint256 verificationGasLimit) internal pure virtual override returns (uint256) {
-        return verificationGasLimit - 350;
+        return verificationGasLimit - 300;
     }
 
 

diff --git a/scripts/geth.sh b/scripts/geth.sh
@@ -1,2 +1,3 @@
 #!/bin/sh
-docker run --rm -p 54321:54321 dtr22/geth-7702-23
+trap "echo killing docker; docker kill geth-7702-23" EXIT
+docker run --name geth-7702-23 --rm -p 8545:8545 -p 54321:54321 dtr22/geth-7702-23 $*
diff --git a/test/GethExecutable.ts b/test/GethExecutable.ts
@@ -7,7 +7,7 @@ const port = 54321
 export const gethLauncher = {
   name: 'geth',
   exec: './scripts/geth.sh',
-  args: `--http --http.api personal,eth,net,web3,debug --rpc.allow-unprotected-txs --allow-insecure-unlock --dev --http.port=${port}`
+  args: `--http --http.api personal,eth,net,web3,debug --rpc.allow-unprotected-txs --allow-insecure-unlock --dev --http.addr 0.0.0.0 --http.port=${port}`
 }
 
 // eslint-disable-next-line @typescript-eslint/no-unused-vars

diff --git a/test/entrypoint.test.ts b/test/entrypoint.test.ts
@@ -225,6 +225,10 @@ describe('EntryPoint', function () {
       })
 
       describe('test with geth', () => {
+        if (process.env.COVERAGE != null) {
+          return
+        }
+
         let geth: GethExecutable
         let prov: JsonRpcProvider
         let delegate: TestEip7702DelegateAccount