fix: minor tweaks to e2e

Signed-off-by: Tomás Migone <[email protected]>

Author: tmigone

TESTING.md

@@ -38,7 +38,7 @@ There are several types of e2e tests which can be run separately:
 It can be useful to run E2E tests against a fresh protocol deployment on L1, this can be done with the following:
 
 ```bash
-yarn test:e2e
+L1_NETWORK=localhost yarn test:e2e
 ```
 
 The command will:
@@ -72,13 +72,13 @@ To run tests against a live testnet or even mainnet run:
 
 ```bash
 # All e2e tests
-npx hardhat e2e --network <network> --graph-config config/graph.<network>.yml
+ARBITRUM_ADDRESS_BOOK=<arbitrum-address-book> npx hardhat e2e --network <network> --l1-graph-config config/graph.<l1-network>.yml --l2-graph-config config/graph.<l2-network>.yml
 
 # Only deployment config tests
-npx hardhat e2e:config --network <network> --graph-config config/graph.<network>.yml
+ARBITRUM_ADDRESS_BOOK=<arbitrum-address-book> npx hardhat e2e:config --network <network> --l1-graph-config config/graph.<l1-network>.yml --l2-graph-config config/graph.<l2-network>.yml
 
 # Only deployment init tests
-npx hardhat e2e:init --network <network> --graph-config config/graph.<network>.yml
+ARBITRUM_ADDRESS_BOOK=<arbitrum-address-book> npx hardhat e2e:init --network <network> --l1-graph-config config/graph.<l1-network>.yml --l2-graph-config config/graph.<l2-network>.yml
 
 # Only a specific scenario
 npx hardhat e2e:scenario <scenario> --network <network> --graph-config config/graph.<network>.yml

tasks/e2e/e2e.ts

@@ -34,7 +34,6 @@ const setGraphConfig = async (args: TaskArguments, hre: HardhatRuntimeEnvironmen
 }
 
 task('e2e', 'Run all e2e tests')
-  .addFlag('disableSecureAccounts', 'Disable secure accounts on GRE')
   .addOptionalParam('graphConfig', cliOpts.graphConfig.description)
   .addOptionalParam('l1GraphConfig', cliOpts.graphConfig.description)
   .addOptionalParam('l2GraphConfig', cliOpts.graphConfig.description)
@@ -50,34 +49,43 @@ task('e2e', 'Run all e2e tests')
       testFiles = testFiles.filter((file) => !['l1', 'l2'].includes(file.split('/')[3]))
     }
 
+    // Disable secure accounts, we don't need them for this task
+    hre.config.graph.disableSecureAccounts = true
+
     setGraphConfig(args, hre)
     await hre.run(TASK_TEST, {
       testFiles: testFiles,
     })
   })
 
 task('e2e:config', 'Run deployment configuration e2e tests')
-  .addFlag('disableSecureAccounts', 'Disable secure accounts on GRE')
   .addOptionalParam('graphConfig', cliOpts.graphConfig.description)
   .addOptionalParam('l1GraphConfig', cliOpts.graphConfig.description)
   .addOptionalParam('l2GraphConfig', cliOpts.graphConfig.description)
   .addOptionalParam('addressBook', cliOpts.addressBook.description)
   .setAction(async (args, hre: HardhatRuntimeEnvironment) => {
     const files = new glob.GlobSync(CONFIG_TESTS).found
+
+    // Disable secure accounts, we don't need them for this task
+    hre.config.graph.disableSecureAccounts = true
+
     setGraphConfig(args, hre)
     await hre.run(TASK_TEST, {
       testFiles: files,
     })
   })
 
 task('e2e:init', 'Run deployment initialization e2e tests')
-  .addFlag('disableSecureAccounts', 'Disable secure accounts on GRE')
   .addOptionalParam('graphConfig', cliOpts.graphConfig.description)
   .addOptionalParam('l1GraphConfig', cliOpts.graphConfig.description)
   .addOptionalParam('l2GraphConfig', cliOpts.graphConfig.description)
   .addOptionalParam('addressBook', cliOpts.addressBook.description)
   .setAction(async (args, hre: HardhatRuntimeEnvironment) => {
     const files = new glob.GlobSync(INIT_TESTS).found
+
+    // Disable secure accounts, we don't need them for this task
+    hre.config.graph.disableSecureAccounts = true
+
     setGraphConfig(args, hre)
     await hre.run(TASK_TEST, {
       testFiles: files,