add catch for dumpLogs and stop network in teardown (#1878)

* add catch for dumpLogs and stop network in teardown * lint
paritytech · Sep 25, 2024 · 7e1c711 · 7e1c711
1 parent e9f0993
commit 7e1c711
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/javascript/packages/orchestrator/src/test-runner/index.ts b/javascript/packages/orchestrator/src/test-runner/index.ts
@@ -139,7 +139,14 @@ export async function run(
       debug(`\t 🕰 [Test] elapsed time: ${elapsedSecs} secs`);
       if (inCI) await registerTotalElapsedTimeSecs(elapsedSecs);
 
-      const logsPath = await network.dumpLogs(false);
+      let logsPath;
+      try {
+        logsPath = await network.dumpLogs(false);
+      } catch (e) {
+        console.log(`${decorators.red("❌ Error dumping logs!")}`);
+        console.log(`err: ${e}`);
+      }
+
       const tests = this.test?.parent?.tests;
 
       if (tests) {
@@ -154,7 +161,12 @@ export async function run(
 
         // All test passed, just remove the network
         console.log(`\n\t ${decorators.green("Deleting network")}`);
-        await network.stop();
+        try {
+          await network.stop();
+        } catch (e) {
+          console.log(`${decorators.yellow("⚠️  Error deleting network")}`);
+          console.log(`err: ${e}`);
+        }
 
         // show logs
         console.log(