Add developer environment setup script and fix symlink issues.

Summary:This diff adds a script after `postinstall` that sets up the dev environment using `lerna`. Once we move jest's source itself into a package, we can drop the custom calls to `npm link`.

Linking `jest-util` pointed out a symlink unmocking bug in Jest I've been meaning to fix for a long time. After running the setup script, Jest would not be able to mock `chalk` properly if `jest-util` was linked into Jest. It turns out we are calling `fs.realpathSync`. I think this was necessary with for `node-haste1` support and is a leftover, however symlink unmocking was never properly supported then so I'm questioning why it was there in the first place.
Closes jestjs#821

Reviewed By: vjeux

Differential Revision: D3080727

fb-gh-sync-id: fd3780f9c30031eb2505c26fb5c944de3f7c948f
shipit-source-id: fd3780f9c30031eb2505c26fb5c944de3f7c948f

Author: cpojer

.npmignore

@@ -1,6 +1,7 @@
+.haste_cache
+__tests__
 docs
 examples
 packages
+setup.sh
 website
-__tests__
-.haste_cache

CHANGELOG.md

@@ -8,6 +8,7 @@
 * Fixed `moduleNameMapper` config option when used with paths.
 * Added `jest-util`, general code cleanup.
 * Fixed an error with Jasmine 2 and tests that `throw 'string errors'`.
+* Fixed issues with unmocking symlinked module names.
 
 ## jest-cli 0.9.2, babel-jest 9.0.3
 

README.md

@@ -735,3 +735,10 @@ Indicates whether each individual test should be reported during the run. All er
 changes and is used by Jest for crawling for files. Disable this if you cannot
 use watchman or use the `--no-watchman` flag.
 <generated_api_end />
+
+## Local Development
+
+For local development the `setup.sh` file is run to link all packages together.
+On most platforms this will be run automatically after `npm install`, however if
+you find that Jest does not set up the development environment correctly, this
+script can also be run manually.

VERSION

@@ -0,0 +1 @@
+9.0.3

package.json

@@ -9,8 +9,8 @@
     "diff": "^2.1.1",
     "graceful-fs": "^4.1.3",
     "istanbul": "^0.4.2",
-    "jest-mock": "^1.0.0",
-    "jest-util": "^1.0.0",
+    "jest-mock": "^9.0.3",
+    "jest-util": "^9.0.3",
     "jsdom": "^7.2.0",
     "json-stable-stringify": "^1.0.0",
     "lodash.template": "^3.6.2",
@@ -25,7 +25,8 @@
   "devDependencies": {
     "babel-eslint": "^4.1.3",
     "eslint": "^1.7.3",
-    "fbjs-scripts": "^0.5.0"
+    "fbjs-scripts": "^0.5.0",
+    "lerna": "^1.1.2"
   },
   "bin": {
     "jest": "./bin/jest.js"
@@ -38,10 +39,11 @@
     "url": "https://github.com/facebook/jest"
   },
   "scripts": {
+    "jasmine1": "node bin/jest.js --testRunner=jasmine1",
     "lint": "eslint .",
+    "postinstall": "[ \"$NODE_ENV\" != production ] && ./setup.sh",
     "prepublish": "npm test",
-    "test": "npm run lint && node bin/jest.js && npm run jasmine1",
-    "jasmine1": "node bin/jest.js --testRunner=jasmine1"
+    "test": "npm run lint && node bin/jest.js && npm run jasmine1"
   },
   "jest": {
     "rootDir": "src",

packages/babel-jest/package.json

@@ -1,6 +1,6 @@
 {
   "name": "babel-jest",
-  "version": "9.0.4",
+  "version": "9.0.3",
   "repository": {
     "type": "git",
     "url": "https://github.com/facebook/jest.git"
@@ -9,6 +9,6 @@
   "main": "src/index.js",
   "dependencies": {
     "babel-core": "^6.0.0",
-    "babel-preset-jest": "^1.0.1"
+    "babel-preset-jest": "^9.0.3"
   }
 }

packages/babel-plugin-jest-hoist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "babel-plugin-jest-hoist",
-  "version": "1.0.0",
+  "version": "9.0.3",
   "repository": {
     "type": "git",
     "url": "https://github.com/facebook/jest.git"

packages/babel-preset-jest/package.json

@@ -1,13 +1,13 @@
 {
   "name": "babel-preset-jest",
-  "version": "1.0.1",
+  "version": "9.0.3",
   "repository": {
     "type": "git",
     "url": "https://github.com/facebook/jest.git"
   },
   "license": "BSD-3-Clause",
   "main": "index.js",
   "dependencies": {
-    "babel-plugin-jest-hoist": "^1.0.0"
+    "babel-plugin-jest-hoist": "^9.0.3"
   }
 }

packages/jest-mock/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jest-mock",
-  "version": "0.0.2",
+  "version": "9.0.3",
   "repository": {
     "type": "git",
     "url": "https://github.com/facebook/jest.git"

packages/jest-util/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jest-util",
-  "version": "1.0.0",
+  "version": "9.0.3",
   "main": "index.js",
   "dependencies": {
     "chalk": "^1.1.1",

setup.sh

@@ -0,0 +1,9 @@
+#!/bin/bash
+
+echo "Setting up Jest's development environment..."
+node_modules/.bin/lerna bootstrap
+
+(cd packages/jest-mock && npm link)
+npm link jest-mock
+(cd packages/jest-util && npm link)
+npm link jest-util

src/HasteModuleLoader/HasteModuleLoader.js

@@ -534,8 +534,7 @@ class Loader {
       throw e;
     }
 
-    const realPath = fs.realpathSync(modulePath);
-    if (this._unmockList && this._unmockList.test(realPath)) {
+    if (this._unmockList && this._unmockList.test(modulePath)) {
       shouldMockModuleCache[moduleName] = false;
       return false;
     }
@@ -544,7 +543,7 @@ class Loader {
     const currentModuleID = this._getNormalizedModuleID(currPath);
     if (
       currPath.includes(constants.NODE_MODULES) &&
-      realPath.includes(constants.NODE_MODULES) &&
+      modulePath.includes(constants.NODE_MODULES) &&
       (
         (this._unmockList && this._unmockList.test(currPath)) ||
         explicitShouldMock[currentModuleID] === false ||