Don't local require perttier in jest-jasmine2

jestjs · Jul 29, 2018 · ba542f4 · ba542f4
1 parent 7caef39
commit ba542f4
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/e2e/__tests__/to_match_inline_snapshot.test.js b/e2e/__tests__/to_match_inline_snapshot.test.js
@@ -163,3 +163,20 @@ test('supports async tests', () => {
   expect(status).toBe(0);
   expect(fileAfter).toMatchSnapshot();
 });
+
+// issue: https://github.com/facebook/jest/issues/6702
+test('handles mocking native modules prettier relies on', () => {
+  const filename = 'mockFail.test.js';
+  const test = `
+    jest.mock('path', () => ({}));
+    jest.mock('fs', () => ({}));
+    test('inline snapshots', () => {
+      expect({}).toMatchInlineSnapshot();
+    });
+  `;
+
+  writeFiles(TESTS_DIR, {[filename]: test});
+  const {stderr, status} = runJest(DIR, ['-w=1', '--ci=false', filename]);
+  expect(stderr).toMatch('1 snapshot written from 1 test suite.');
+  expect(status).toBe(0);
+});
diff --git a/packages/jest-jasmine2/src/setup_jest_globals.js b/packages/jest-jasmine2/src/setup_jest_globals.js
@@ -102,7 +102,8 @@ export default ({
     expand,
     getBabelTraverse: () => require('babel-traverse').default,
     getPrettier: () =>
-      config.prettierPath ? localRequire(config.prettierPath) : null,
+      // $FlowFixMe dynamic require
+      config.prettierPath ? require(config.prettierPath) : null,
     updateSnapshot,
   });
   setState({snapshotState, testPath});