🐛 Bug: Error in mochaGlobalTeardown is swallowed and exits with code 0

Question

josh-cloudscape opened this issue 21 days ago · comments

I have read and agree to Mocha's Code of Conduct and Contributing Guidelines
I have searched for related issues and issues with the faq label, but none matched my issue.
I have 'smoke tested' the code to be tested by running it outside the real test suite to get a better sense of whether the problem is in the code under test, my usage of Mocha, or Mocha itself.
I want to provide a PR to resolve this

The error should be logged and the tests should exit with a non-zero exit code

The error is swallowed and the tests always exit with a 0 exit code (even if some tests failed)

// test.js
const { it } = require("mocha");

it("should work", () => {
  throw new Error("It doesn't work");
});

// setup.js
exports.mochaGlobalTeardown = async function () {
  throw new Error("Teardown problem");
};

$ npx mocha -r setup.js test.js

Mocha v10.7.3, Node v20.15.1

No response