cleanup #531

lib/fork.js

@@ -9,6 +9,7 @@ var send = require('./send');
 
 var env = process.env;
 
+// ensure NODE_PATH paths are absolute
 if (process.env.NODE_PATH) {
 	var nodePath = process.env.NODE_PATH
 		.split(path.delimiter)
@@ -17,9 +18,7 @@ if (process.env.NODE_PATH) {
 		})
 		.join(path.delimiter);
 
-	env = objectAssign({
-		NODE_PATH: nodePath
-	}, env);
+	env = objectAssign({NODE_PATH: nodePath}, env);
 }
 
 module.exports = function (file, opts) {

test/cli.js

@@ -161,8 +161,9 @@ test('watcher works', function (t) {
 
 test('handles NODE_PATH', function (t) {
 	var nodePaths = 'node-paths/modules' + path.delimiter + 'node-paths/deep/nested';
+
 	execCli('fixture/node-paths.js', {env: {NODE_PATH: nodePaths}}, function (err) {
-		t.notOk(err);
+		t.ifError(err);
 		t.end();
 	});
 });

test/fixture/node-paths.js

@@ -1,9 +1,8 @@
 import test from '../../';
-
 import foo from 'nested/foo';
 import bar from 'path/bar';
 
 test('relative require', t => {
-	t.is(foo(), 'bar');
-	t.is(bar(), 'baz');
+	t.is(foo, 'bar');
+	t.is(bar, 'baz');
 });

test/fixture/node-paths/deep/nested/path/bar.js

@@ -1,4 +1 @@
-module.exports = function () {
-	return 'baz';
-};
-
+module.exports = 'baz';

test/fixture/node-paths/modules/nested/foo.js

@@ -1,4 +1 @@
-module.exports = function () {
-	return 'bar';
-};
-
+module.exports = 'bar';