fix: json writing no longer occurs on live test

Ion-Protocol · Sep 19, 2024 · 7b22afa · 7b22afa
1 parent c34facf
commit 7b22afa
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 8 deletions.
diff --git a/script/deploy/deployAll.s.sol b/script/deploy/deployAll.s.sol
@@ -51,8 +51,19 @@ contract DeployAll is BaseScript {
 
     ConfigReader.Config mainConfig;
 
+    // skips the json writing
+    function runLiveTest(string memory deployFile) public {
+        deploy(ConfigReader.toConfig(vm.readFile(string.concat(CONFIG_PATH_ROOT, deployFile)), getChainConfigFile()));
+    }
+
     function run(string memory deployFile) public {
         deploy(ConfigReader.toConfig(vm.readFile(string.concat(CONFIG_PATH_ROOT, deployFile)), getChainConfigFile()));
+        // write everything to an out file
+        mainConfig.boringVault.toHexString().write(OUTPUT_JSON_PATH, ".boringVault");
+        mainConfig.manager.toHexString().write(OUTPUT_JSON_PATH, ".manager");
+        mainConfig.accountant.toHexString().write(OUTPUT_JSON_PATH, ".accountant");
+        mainConfig.teller.toHexString().write(OUTPUT_JSON_PATH, ".teller");
+        mainConfig.rolesAuthority.toHexString().write(OUTPUT_JSON_PATH, ".rolesAuthority");
     }
 
     function deploy(ConfigReader.Config memory config) public override returns (address) {
@@ -84,13 +95,6 @@ contract DeployAll is BaseScript {
         console.log("Set Authority And Transfer Ownerships Complete");
 
         mainConfig = config;
-
-        // write everything to an out file
-        config.boringVault.toHexString().write(OUTPUT_JSON_PATH, ".boringVault");
-        config.manager.toHexString().write(OUTPUT_JSON_PATH, ".manager");
-        config.accountant.toHexString().write(OUTPUT_JSON_PATH, ".accountant");
-        config.teller.toHexString().write(OUTPUT_JSON_PATH, ".teller");
-        config.rolesAuthority.toHexString().write(OUTPUT_JSON_PATH, ".rolesAuthority");
     }
 
     function _deployTeller(ConfigReader.Config memory config) public returns (address teller) {

diff --git a/test/LiveDeploy.t.sol b/test/LiveDeploy.t.sol
@@ -71,7 +71,7 @@ contract LiveDeploy is ForkTest, DeployAll {
 
         // Run the deployment scripts
 
-        run(FILE_NAME);
+        runLiveTest(FILE_NAME);
 
         // check for if all rate providers are deployed, if not error
         for (uint256 i; i < mainConfig.assets.length; ++i) {