fix: log hybrid module as warning instead of failing tests (#5144)

Fixes #5130

Author: ahnpnl

e2e/hybrid-module/__tests__/hybrid-module.spec.ts

@@ -0,0 +1,5 @@
+describe('hybrid-module', () => {
+  it('should work', () => {
+    expect(true).toBe(true)
+  })
+})

e2e/hybrid-module/jest-compiler-cjs.config.ts

@@ -0,0 +1,17 @@
+import type { Config } from 'jest'
+import { TS_JS_TRANSFORM_PATTERN } from 'ts-jest'
+
+export default {
+  displayName: 'hybrid-module-compiler-cjs',
+  transform: {
+    [TS_JS_TRANSFORM_PATTERN]: [
+      'ts-jest',
+      {
+        tsconfig: '<rootDir>/tsconfig.spec.json',
+        diagnostics: {
+          ignoreCodes: [151002],
+        },
+      },
+    ],
+  },
+} satisfies Config

e2e/hybrid-module/jest-compiler-esm.config.ts

@@ -0,0 +1,19 @@
+import type { Config } from 'jest'
+import { TS_JS_TRANSFORM_PATTERN } from 'ts-jest'
+
+export default {
+  displayName: 'hybrid-module-compiler-esm',
+  extensionsToTreatAsEsm: ['.ts'],
+  transform: {
+    [TS_JS_TRANSFORM_PATTERN]: [
+      'ts-jest',
+      {
+        tsconfig: '<rootDir>/tsconfig.spec.json',
+        useESM: true,
+        diagnostics: {
+          ignoreCodes: [151002],
+        },
+      },
+    ],
+  },
+} satisfies Config

e2e/hybrid-module/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "hybrid-module",
+  "private": true
+}

e2e/hybrid-module/tsconfig.spec.json

@@ -0,0 +1,3 @@
+{
+  "extends": "../../tsconfig.base.json"
+}

src/legacy/compiler/ts-compiler.spec.ts

@@ -9,7 +9,6 @@ import { createConfigSet, makeCompiler } from '../../__helpers__/fakers'
 import { logTargetMock } from '../../__helpers__/mocks'
 import { tsTranspileModule } from '../../transpilers/typescript/transpile-module'
 import type { DepGraphInfo, TsJestTransformerOptions } from '../../types'
-import { TsJestDiagnosticCodes } from '../../utils'
 import { Errors, Helps, interpolate } from '../../utils/messages'
 
 import { updateOutput } from './compiler-utils'
@@ -392,17 +391,11 @@ describe('TsCompiler', () => {
         expect(usedCompilerOptions.customConditions).toBeUndefined()
         expect(output).toEqual({
           code: updateOutput(jsOutput, fileName, sourceMap),
-          diagnostics: [
-            {
-              category: ts.DiagnosticCategory.Message,
-              code: TsJestDiagnosticCodes.ModernNodeModule,
-              messageText: Helps.UsingModernNodeResolution,
-              file: undefined,
-              start: undefined,
-              length: undefined,
-            },
-          ],
+          diagnostics: [],
         })
+        expect(logTarget.filteredLines(LogLevels.warn)).toEqual(
+          expect.arrayContaining([expect.stringContaining(Helps.UsingModernNodeResolution)]),
+        )
 
         // @ts-expect-error testing purpose
         // eslint-disable-next-line @typescript-eslint/no-non-null-assertion

src/legacy/compiler/ts-compiler.ts

@@ -211,14 +211,16 @@ export class TsCompiler implements TsCompilerInstance {
       const output: EmitOutput = this._languageService.getEmitOutput(fileName)
       const diagnostics = this.getDiagnostics(fileName)
       if (isModernNodeModuleKind(this._initialCompilerOptions.module)) {
-        diagnostics.push({
-          category: this._ts.DiagnosticCategory.Message,
-          code: TsJestDiagnosticCodes.ModernNodeModule,
-          messageText: Helps.UsingModernNodeResolution,
-          file: undefined,
-          start: undefined,
-          length: undefined,
-        })
+        this.configSet.raiseDiagnostics([
+          {
+            category: this._ts.DiagnosticCategory.Message,
+            code: TsJestDiagnosticCodes.ModernNodeModule,
+            messageText: Helps.UsingModernNodeResolution,
+            file: undefined,
+            start: undefined,
+            length: undefined,
+          },
+        ])
       }
       if (!isEsmMode && diagnostics.length) {
         this.configSet.raiseDiagnostics(diagnostics, fileName, this._logger)

src/transpilers/typescript/transpile-module.ts

@@ -49,7 +49,11 @@ type ExtendedTsTranspileModuleFn = (
 ) => ts.TranspileOutput
 
 export const isModernNodeModuleKind = (module: ts.ModuleKind | undefined): boolean => {
-  return module ? [ts.ModuleKind.Node16, /* ModuleKind.Node18 */ 101, ts.ModuleKind.NodeNext].includes(module) : false
+  return module
+    ? [ts.ModuleKind.Node16, /* ModuleKind.Node18 */ 101, /* ModuleKind.Node20 */ 102, ts.ModuleKind.NodeNext].includes(
+        module,
+      )
+    : false
 }
 
 const shouldCheckProjectPkgJsonContent = (fileName: string, moduleKind: ts.ModuleKind | undefined): boolean => {