Fix problem with tsconfig from node_modules

  * Use Jest rootDir in parseJsonConfigFileContent as basePath
  * Add tests

Author: hgenru

src/utils.ts

@@ -37,7 +37,7 @@ function readCompilerOptions(configPath: string, rootDir: string) {
   }
 
   // Second step: Parse the config, resolving all potential references.
-  const basePath = path.dirname(configPath); // equal to "getDirectoryPath" from ts, at least in our case.
+  const basePath = path.resolve(rootDir);
   const parsedConfig = tsc.parseJsonConfigFileContent(
     loaded.config,
     tsc.sys,

tests/__tests__/use-config-from-node-modules.spec.ts

@@ -0,0 +1,22 @@
+import * as fs from 'fs-extra';
+import * as path from 'path';
+import runJest from '../__helpers__/runJest';
+
+describe('Use config from node_modules', () => {
+  it('Should run all tests', () => {
+    const testDir = path.resolve(__dirname, '../use-config-from-node-modules');
+    const configPath = path.resolve(testDir, 'tsconfig.json');
+    const targetDir = path.resolve(testDir, 'node_modules', 'common-tsconfig');
+    const targetConfigPath = path.resolve(targetDir, 'tsconfig.json');
+    fs.ensureDirSync(targetDir);
+    fs.copyFileSync(configPath, targetConfigPath);
+
+    const result = runJest('../use-config-from-node-modules', ['--no-cache']);
+    const stderr = result.stderr;
+
+    expect(result.status).toBe(1);
+    expect(stderr).toContain('1 failed, 1 total');
+    expect(stderr).toContain('Hello Class');
+    expect(stderr).toContain('should throw an error on line 18');
+  });
+});

tests/use-config-from-node-modules/Hello.ts

@@ -0,0 +1,28 @@
+interface FooInterface {
+  foo: string;
+  bar: number; //This interface should be stripped and the line numbers should still fit.
+}
+
+export class Hello {
+  constructor() {
+    const greeting = `
+      this
+      is
+      a
+      multiline
+      greeting
+    `;
+
+    this.unexcuted(() => {});
+
+    throw new Error('Hello error!');
+  }
+
+  public unexcuted(action: () => void = () => {}): void {
+    if (action) {
+      action();
+    } else {
+      console.log('unexcuted');
+    }
+  }
+}

tests/use-config-from-node-modules/__tests__/Hello.test.ts

@@ -0,0 +1,9 @@
+declare var jest, describe, it, expect;
+
+import { Hello } from '../Hello';
+
+describe('Hello Class', () => {
+  it('should throw an error on line 18', () => {
+    const hello = new Hello();
+  });
+});

tests/use-config-from-node-modules/package.json

@@ -0,0 +1,20 @@
+{
+  "jest": {
+    "globals": {
+      "ts-jest": {
+        "tsConfigFile": "./node_modules/common-tsconfig/tsconfig.json"
+      }
+    },
+    "transform": {
+      "^.+\\.tsx?$": "ts-jest"
+    },
+    "testRegex": "(/__tests__/.*|(\\.|/)(test|spec))\\.(jsx?|tsx?)$",
+    "moduleFileExtensions": [
+      "ts",
+      "tsx",
+      "js",
+      "jsx",
+      "json"
+    ]
+  }
+}

tests/use-config-from-node-modules/tsconfig.json

@@ -0,0 +1,10 @@
+{
+  "compilerOptions": {
+    "target": "ES5",
+    "module": "commonjs",
+    "moduleResolution": "node",
+    "noEmitOnError": false,
+    "jsx": "react",
+    "allowJs": true
+  }
+}