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

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>
nodejs · Feb 1, 2017 · e7c4dfb · e7c4dfb
1 parent 22226fa
commit e7c4dfb
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
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 ,$/
+);
+
 assert.strictEqual(require('../fixtures/packages/main').ok, 'ok',
                    'Failed loading package');
 assert.strictEqual(require('../fixtures/packages/main-index').ok, 'ok',