internal: fixup diagnostic reporting

Author: runspired

packages/diagnostic/server/reporters/default.js

@@ -184,6 +184,13 @@ export default class CustomDotReporter {
         )
       );
     }
+    if (this.globalFailures.length) {
+      this.write(
+        chalk.red(
+          `\n\n${this.globalFailures.length} Global Failures were detected.. Complete stack traces for failures will print at the end.`
+        )
+      );
+    }
     this.write(`\n\n`);
 
     this.reportPendingTests();
@@ -214,7 +221,7 @@ export default class CustomDotReporter {
 
     this.clearState();
 
-    return this.failedTests.length ? 1 : 0;
+    return this.globalFailures.length || this.failedTests.length ? 1 : 0;
   }
 
   addLauncher(data) {

packages/diagnostic/src/internals/run.ts

@@ -67,7 +67,20 @@ export async function runTest<TC extends TestContext>(
   }
 
   for (const hook of beforeChain) {
-    await hook.call(testContext, Assert);
+    try {
+      await hook.call(testContext, Assert);
+    } catch (err) {
+      Assert.pushResult({
+        message: `Unexpected Test Failure in beforeEach: ${(err as Error).message}`,
+        stack: (err as Error).stack!,
+        passed: false,
+        actual: false,
+        expected: true,
+      });
+      if (!Config.params.tryCatch.value) {
+        throw err;
+      }
+    }
   }
 
   try {
@@ -91,7 +104,21 @@ export async function runTest<TC extends TestContext>(
     }
   } finally {
     for (const hook of afterChain) {
-      await hook.call(testContext, Assert);
+      try {
+        await hook.call(testContext, Assert);
+      } catch (e) {
+        Assert.pushResult({
+          message: `Unexpected Test Failure in afterEach: ${(e as Error).message}`,
+          stack: (e as Error).stack!,
+          passed: false,
+          actual: false,
+          expected: true,
+        });
+        if (!Config.params.tryCatch.value) {
+          // eslint-disable-next-line no-unsafe-finally
+          throw e;
+        }
+      }
     }
     Assert._finalize();