Merge pull request #1677 from jmm/node_modules-robust3

Complete "Fix flawed require hook node_modules check" properly

test/core/fixtures/bin/babel-node/require/in-files/foo2.js

@@ -1,4 +1,5 @@
 import "./bar2";
+import "./not_node_modules";
 
 var foo = () => console.log("foo");
 foo();

test/core/fixtures/bin/babel-node/require/in-files/not_node_modules.jsx

@@ -0,0 +1,10 @@
+/*
+The purpose of this file is to test that the node_modules check in the require
+hook doesn't mistakenly exclude something like "not_node_modules". To pass, this
+file merely needs to be transpiled. The transpiled code won't, and doesn't need
+to, execute without error. It won't execute because React will be undefined.
+*/
+try {
+  <Some jsx="element" />;
+}
+catch (e) {}

test/core/fixtures/require-hook/not_node_modules/input.js

@@ -1 +0,0 @@
-() => {};

test/core/require-hook.js

@@ -1,12 +0,0 @@
-var chai        = require("chai");
-
-// Require-hook.
-require("../../lib/babel/api/register/node");
-
-suite("require hook", function () {
-  test("not node_modules", function () {
-    chai.expect(function () {
-      require("./fixtures/require-hook/not_node_modules/input");
-    }).to.not.throw();
-  });
-});