fix: prevent double teardown on test environment error

Author: noomorph

e2e/__tests__/customEnvironmentTeardownError.test.ts

@@ -0,0 +1,14 @@
+import runJest from '../runJest';
+
+// This test ensures that if a custom environment's teardown throws, Jest reports the real error, not an internal error.
+describe('Custom Environment Teardown Error', () => {
+  it('reports the error thrown from teardown()', () => {
+    const {stderr, exitCode} = runJest('custom-environment-teardown-error');
+    expect(exitCode).toBe(1);
+    expect(stderr).toMatch('teardown error from custom environment');
+    // Should NOT contain the internal error that was seen in the regression
+    expect(stderr).not.toMatch(
+      'The "object" argument must be of type object. Received null',
+    );
+  });
+});

e2e/custom-environment-teardown-error/CustomEnvironment.js

@@ -0,0 +1,10 @@
+const NodeEnvironment = require('jest-environment-node').default;
+
+class CustomEnvironment extends NodeEnvironment {
+  async teardown() {
+    await super.teardown();
+    throw new Error('teardown error from custom environment');
+  }
+}
+
+module.exports = CustomEnvironment;

e2e/custom-environment-teardown-error/hello.test.js

@@ -0,0 +1,7 @@
+function hello() {
+  return 'Hello, world!';
+}
+
+test('hello returns Hello, world!', () => {
+  expect(hello()).toBe('Hello, world!');
+});

e2e/custom-environment-teardown-error/package.json

@@ -0,0 +1,5 @@
+{
+  "jest": {
+    "testEnvironment": "<rootDir>/CustomEnvironment.js"
+  }
+}

packages/jest-runner/src/runTest.ts

@@ -217,8 +217,11 @@ async function runTestInternal(
       );
       sourcemapSupport.resetRetrieveHandlers();
 
-      await environment.teardown();
-      isTornDown = true;
+      try {
+        await environment.teardown();
+      } finally {
+        isTornDown = true;
+      }
     }
   };