Errors are not reporeted when thrown from before contexts

[jmcdo29]

Current Behavior

When using test suites, and a before throws an error, the only indication of the error being thrown is that the number of tests passed is 0 when there is 1 test in total.

Expected Behavior

The error being thrown should be bubbled up and reported as any other error would be

Minimum Reproduction

const { suite } = require("uvu");
const { ok } = require("uvu/assert");

const TestSuite = suite(
  "This suite will not report an error from the before context, just that no tests were ran"
);
TestSuite.before(() => {
  throw new Error("You will never see this");
});
TestSuite("I should be reported on!", () => {
  ok(true);
});
TestSuite.run();

You can find a reproduction of the error here

Reproduction steps

git clone
cd uvu-context-error
pnpm i / npm i / yarn
node index 

Other Information

I get the same outcome whether using uvu as a CLI (like the test script does) or calling the file directly

❯ node index.js
 This suite will not report an error from the before context, just that no tests were ran    (0 / 1)

  Total:     1
  Passed:    0
  Skipped:   0
  Duration:  1.06ms

❯ pnpm test

> uvu-before-fail@0.0.0 test ~/uvu-before-fail
> uvu . index\.js

index.js
 This suite will not report an error from the before context, just that no tests were ran    (0 / 1)

  Total:     1
  Passed:    0
  Skipped:   0
  Duration:  0.88ms

[jmcdo29]

I also just noticed that if there is a failure in a before then the CLI still exits with code 0 so CI doesn't register the failure.

[jmcdo29]

Looks like the for (hook of before) await hook(state); is not being caught properly, so the finally doesn't have anything to report on. Should this have a try/catch of its own and write to the errors like it does around line 81?

[haayhappen]

This still seems to be an issue - any News?