test: invalid package.json causes error when require()ing in directory

Sam Shull · addaleax · commit 6f2f02d5ad09 · 2016-12-08T02:11:08.000+01:00
Requiring a file from a directory that contains an invalid package.json file should throw an error. PR-URL: #10044 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/test/fixtures/packages/invalid/index.js b/test/fixtures/packages/invalid/index.js
@@ -0,0 +1 @@
+exports.ok = 'ok';
diff --git a/test/fixtures/packages/invalid/package.json b/test/fixtures/packages/invalid/package.json
@@ -0,0 +1 @@
+{,}
diff --git a/test/sequential/test-module-loading.js b/test/sequential/test-module-loading.js
@@ -69,6 +69,13 @@ assert.strictEqual(threeFolder, threeIndex);
 assert.notStrictEqual(threeFolder, three);
 
 console.error('test package.json require() loading');
+assert.throws(
+  function() {
+    require('../fixtures/packages/invalid');
+  },
+  /^SyntaxError: Error parsing \S+: Unexpected token , in JSON at position 1$/
+);
+
 assert.strictEqual(require('../fixtures/packages/index').ok, 'ok',
                    'Failed loading package');
 assert.strictEqual(require('../fixtures/packages/main').ok, 'ok',
