fix(compiler): use resolveModuleNames TypeScript API to get resolved modules for test files (#1784)

Fixes #1747

Author: ahnpnl

e2e/__cases__/diagnostics/throw/main.spec.ts renamed to e2e/__cases__/diagnostics/main.spec.ts

@@ -1,6 +1,6 @@
 import { mocked } from 'ts-jest'
-import { foo } from './to-mock'
-jest.mock('./to-mock')
+import { foo } from './pass-to-mock'
+jest.mock('./pass-to-mock')
 
 test('foo', () => {
   foo()

e2e/__cases__/diagnostics/throw/main.ts renamed to e2e/__cases__/diagnostics/main.ts

@@ -0,0 +1,25 @@
+export const foo = () => 'foo'
+
+export function bar() {
+  return 'bar'
+}
+export namespace bar {
+  export function dummy() {
+    return 'dummy'
+  }
+  export namespace dummy {
+    export const deep = {
+      deeper: (one: string = '1') => `deeper ${one}`
+    }
+  }
+}
+
+export class MyClass {
+  constructor(s: string) {
+    this.myProperty = 3
+    this.myStr = s
+  }
+  somethingClassy() { return this.myStr }
+  public myProperty: number;
+  public myStr: string;
+}

e2e/__cases__/diagnostics/warn/main.spec.ts

@@ -1,6 +1,6 @@
 import { mocked } from 'ts-jest/utils'
-import { foo, bar, MyClass } from './to-mock'
-jest.mock('./to-mock')
+import { foo, bar, MyClass } from './pass-to-mock'
+jest.mock('./pass-to-mock')
 
 test('foo', () => {
   // real returns 'foo', mocked returns 'bar'

e2e/__cases__/diagnostics/warn/main.ts

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`With diagnostics throw should fail using template "default" 1`] = `
-  × jest
+  × jest --no-cache
   ↳ exit code: 1
   ===[ STDOUT ]===================================================================
   
@@ -28,7 +28,7 @@ exports[`With diagnostics throw should fail using template "default" 1`] = `
 `;
 
 exports[`With diagnostics throw should fail using template "with-babel-7" 1`] = `
-  × jest
+  × jest --no-cache
   ↳ exit code: 1
   ===[ STDOUT ]===================================================================
   
@@ -55,7 +55,7 @@ exports[`With diagnostics throw should fail using template "with-babel-7" 1`] =
 `;
 
 exports[`With diagnostics throw should fail using template "with-babel-7-string-config" 1`] = `
-  × jest
+  × jest --no-cache
   ↳ exit code: 1
   ===[ STDOUT ]===================================================================
   

e2e/__cases__/test-helpers/deprecated.spec.ts

@@ -2,7 +2,9 @@ import { allValidPackageSets } from '../__helpers__/templates'
 import { configureTestCase } from '../__helpers__/test-case'
 
 describe('With diagnostics throw', () => {
-  const testCase = configureTestCase('diagnostics/throw')
+  const testCase = configureTestCase('diagnostics', {
+    noCache: true, // warnings shown only on first compilation
+  })
 
   testCase.runWithTemplates(allValidPackageSets, 1, (runTest, { testLabel }) => {
     it(testLabel, () => {
@@ -14,7 +16,7 @@ describe('With diagnostics throw', () => {
 })
 
 describe('With diagnostics warn only', () => {
-  const testCase = configureTestCase('diagnostics/warn', {
+  const testCase = configureTestCase('diagnostics', {
     tsJestConfig: {
       diagnostics: { warnOnly: true },
     },

e2e/__cases__/test-helpers/pass-to-mock.ts

@@ -26,7 +26,7 @@ export function tsJestConfig(options?: Partial<TsJestConfig>): TsJestConfig {
   }
 }
 
-export function getJestConfig<T extends Config.ProjectConfig>(
+function getJestConfig<T extends Config.ProjectConfig>(
   options?: Partial<Config.InitialOptions | Config.ProjectConfig>,
   tsJestOptions?: TsJestGlobalOptions,
 ): T {
@@ -53,6 +53,39 @@ export function importReason(text = '[[BECAUSE]]'): ImportReasons {
   return text as any
 }
 
+export const defaultResolve = (path: string): string => `resolved:${path}`
+
+export function createConfigSet({
+  jestConfig,
+  tsJestConfig,
+  parentConfig,
+  resolve = defaultResolve,
+  ...others
+}: {
+  jestConfig?: Partial<Config.ProjectConfig>
+  tsJestConfig?: TsJestGlobalOptions
+  parentConfig?: TsJestGlobalOptions
+  resolve?: ((path: string) => string) | null
+  [key: string]: any
+} = {}): ConfigSet {
+  const defaultTestRegex = ['(/__tests__/.*|(\\\\.|/)(test|spec))\\\\.[jt]sx?$']
+  const defaultTestMatch = ['**/__tests__/**/*.[jt]s?(x)', '**/?(*.)+(spec|test).[jt]s?(x)']
+  jestConfig = {
+    ...jestConfig,
+    testMatch: jestConfig?.testMatch ? [...jestConfig.testMatch, ...defaultTestMatch] : defaultTestMatch,
+    testRegex: jestConfig?.testRegex ? [...defaultTestRegex, ...jestConfig.testRegex] : defaultTestRegex,
+  }
+  const cs = new ConfigSet(getJestConfig(jestConfig, tsJestConfig), parentConfig)
+  if (resolve) {
+    cs.resolvePath = resolve
+  }
+  Object.keys(others).forEach((key) => {
+    Object.defineProperty(cs, key, { value: others[key] })
+  })
+
+  return cs
+}
+
 // not really unit-testing here, but it's hard to mock all those values :-D
 export function makeCompiler({
   jestConfig,
@@ -68,14 +101,7 @@ export function makeCompiler({
     ...(tsJestConfig.diagnostics as any),
     pretty: false,
   }
-  const testRegex = ['^.+\\.[tj]sx?$']
-  const testMatch = ['^.+\\.tsx?$']
-  jestConfig = {
-    ...jestConfig,
-    testMatch: jestConfig?.testMatch ? [...jestConfig.testMatch, ...testMatch] : testMatch,
-    testRegex: jestConfig?.testRegex ? [...testRegex, ...jestConfig.testRegex] : testRegex,
-  }
-  const cs = new ConfigSet(getJestConfig(jestConfig, tsJestConfig), parentConfig)
+  const cs = createConfigSet({ jestConfig, tsJestConfig, parentConfig, resolve: null })
 
   return createCompilerInstance(cs)
 }