feat(jest-core): support testResultsProcessor written in ESM (#12006)

Author: chentsulin

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ### Features
 
+- `[jest-core]` Add support for `testResultsProcessor` written in ESM ([#12006](https://github.com/facebook/jest/pull/12006))
+
 ### Fixes
 
 - `[expect]` Allow again `expect.Matchers` generic with single value ([#11986](https://github.com/facebook/jest/pull/11986))

e2e/__tests__/testResultsProcessor.test.ts

@@ -6,9 +6,10 @@
  */
 
 import * as path from 'path';
+import {onNodeVersions} from '@jest/test-utils';
 import {json as runWithJson} from '../runJest';
 
-test('testNamePattern', () => {
+test('testResultsProcessor', () => {
   const processorPath = path.resolve(
     __dirname,
     '../test-results-processor/processor.js',
@@ -19,3 +20,18 @@ test('testNamePattern', () => {
   ]);
   expect(json.processed).toBe(true);
 });
+
+// The versions where vm.Module exists and commonjs with "exports" is not broken
+onNodeVersions('>=12.16.0', () => {
+  test('testResultsProcessor written in ESM', () => {
+    const processorPath = path.resolve(
+      __dirname,
+      '../test-results-processor/processor.mjs',
+    );
+    const {json} = runWithJson('test-results-processor', [
+      '--json',
+      `--testResultsProcessor=${processorPath}`,
+    ]);
+    expect(json.processed).toBe(true);
+  });
+});

e2e/test-results-processor/processor.mjs

@@ -0,0 +1,11 @@
+/**
+ * Copyright (c) Facebook, Inc. and its affiliates. All Rights Reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+
+export default function (results) {
+  results.processed = true;
+  return results;
+}

packages/jest-core/src/runJest.ts

@@ -13,6 +13,7 @@ import {CustomConsole} from '@jest/console';
 import {
   AggregatedResult,
   Test,
+  TestResultsProcessor,
   formatTestResults,
   makeEmptyAggregatedTestResult,
 } from '@jest/test-result';
@@ -96,7 +97,10 @@ const processResults = async (
   }
 
   if (testResultsProcessor) {
-    runResults = require(testResultsProcessor)(runResults);
+    const processor = await requireOrImportModule<TestResultsProcessor>(
+      testResultsProcessor,
+    );
+    runResults = processor(runResults);
   }
   if (isJSON) {
     if (outputFile) {

packages/jest-test-result/src/index.ts

@@ -28,6 +28,7 @@ export type {
   TestEvents,
   TestFileEvent,
   TestResult,
+  TestResultsProcessor,
   TestCaseResult,
   V8CoverageResult,
 } from './types';

packages/jest-test-result/src/types.ts

@@ -78,6 +78,10 @@ export type AggregatedResult = AggregatedResultWithoutCoverage & {
   coverageMap?: CoverageMap | null;
 };
 
+export type TestResultsProcessor = (
+  results: AggregatedResult,
+) => AggregatedResult;
+
 export type Suite = {
   title: string;
   suites: Array<Suite>;