fix: use different exit codes for errored and failed tests

Author: barbaralocsi

src/cli/send/plugin/testExitCodeInterceptor.ts

@@ -12,12 +12,30 @@ export const testExitCodeInterceptor = {
 
   afterTrigger: async function bail(hookContext: HookTriggerContext<[models.ProcessorContext], boolean>) {
     const context = hookContext.args[0];
-    const hasFailedOrErroredTestResult = context.httpRegion.testResults?.some?.(
-      obj => obj.status === models.TestResultStatus.FAILED || obj.status === models.TestResultStatus.ERROR
-    );
-    if (hasFailedOrErroredTestResult) {
+
+    if (context.httpRegion.testResults === undefined) {
+      return true;
+    }
+
+    let hasErroredTestResult = false;
+    let hasFailedTestResult = false;
+
+    for (const testResult of context.httpRegion.testResults) {
+      if (testResult.status === models.TestResultStatus.ERROR) {
+        hasErroredTestResult = true;
+        break;
+      }
+      if (testResult.status === models.TestResultStatus.FAILED) {
+        hasFailedTestResult = true;
+      }
+    }
+
+    if (hasErroredTestResult) {
+      process.exitCode = 19;
+    } else if (hasFailedTestResult) {
       process.exitCode = 20;
     }
+
     return true;
   },
 };