chore: improve process exit handling

Informatievlaanderen · Apr 4, 2024 · 58078cd · 58078cd
1 parent 91d9290
commit 58078cd
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 9 deletions.
diff --git a/src/controller.ts b/src/controller.ts
@@ -298,15 +298,19 @@ export class Controller {
 
     this._writeMetadata(test.run);
 
-    jmeter.on('close', (code) => {
+    jmeter.on('close', (code, signal) => {
       try {
-        const duration = endTimer && endTimer({ ...labels, category: run.category, name: run.name });
-        const updatedTest = { run: { ...run, status: TestRunStatus.done, code: code, duration: duration }, process: jmeter } as Test;
-        const updatedRun = this._upsertTest(updatedTest).run;
-        this._writeMetadata(updatedRun);
-        this._moveToResults(updatedRun.id);
+        if (!!signal) {
+          console.info(`[WARN] Ignoring process exit code '${code}' for test ${id} because received signal ${signal}`);
+        } else {
+          const duration = endTimer && endTimer({ ...labels, category: run.category, name: run.name });
+          const updatedTest = { run: { ...run, status: TestRunStatus.done, code: code, duration: duration }, process: jmeter } as Test;
+          const updatedRun = this._upsertTest(updatedTest).run;
+          this._writeMetadata(updatedRun);
+          this._moveToResults(updatedRun.id);
+        }
       } catch (error) {
-        console.error('Failed to write metadata because: ', error);
+        console.error('[ERROR] Failed to write metadata because: ', error);
       }
     });
 

diff --git a/src/server.ts b/src/server.ts
@@ -70,15 +70,15 @@ server.addHook('onReady', async () => {
   try {
     await controller.importTestsAndRuns();
   } catch (error) {
-    console.error('Failed to import metadata because: ', error);
+    console.error('[ERROR] Failed to import metadata because: ', error);
   }
 });
 
 server.addHook('onClose', async () => {
   try {
     await controller.exportTestRuns();
   } catch (error) {
-    console.error('Failed to export metadata because: ', error);
+    console.error('[ERROR] Failed to export metadata because: ', error);
   }
 })