test: add module require tests for certain package.json errors
test for unusual error cases: verify that module require() falls back to index if package.json names a missing file and throws an error if package.json is unparseable. PR-URL: https://github.com/nodejs/node/pull/23285 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
This commit is contained in:
parent
1d96d7da73
commit
13e6e01ce5
22
test/fixtures/packages/missing-main/index.js
vendored
Normal file
22
test/fixtures/packages/missing-main/index.js
vendored
Normal file
@ -0,0 +1,22 @@
|
|||||||
|
// Copyright Joyent, Inc. and other Node contributors.
|
||||||
|
//
|
||||||
|
// Permission is hereby granted, free of charge, to any person obtaining a
|
||||||
|
// copy of this software and associated documentation files (the
|
||||||
|
// "Software"), to deal in the Software without restriction, including
|
||||||
|
// without limitation the rights to use, copy, modify, merge, publish,
|
||||||
|
// distribute, sublicense, and/or sell copies of the Software, and to permit
|
||||||
|
// persons to whom the Software is furnished to do so, subject to the
|
||||||
|
// following conditions:
|
||||||
|
//
|
||||||
|
// The above copyright notice and this permission notice shall be included
|
||||||
|
// in all copies or substantial portions of the Software.
|
||||||
|
//
|
||||||
|
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
|
||||||
|
// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
|
||||||
|
// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN
|
||||||
|
// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
|
||||||
|
// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
|
||||||
|
// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
|
||||||
|
// USE OR OTHER DEALINGS IN THE SOFTWARE.
|
||||||
|
|
||||||
|
exports.ok = 'ok';
|
4
test/fixtures/packages/missing-main/package.json
vendored
Normal file
4
test/fixtures/packages/missing-main/package.json
vendored
Normal file
@ -0,0 +1,4 @@
|
|||||||
|
{
|
||||||
|
"name": "missingmain",
|
||||||
|
"main": "doesnotexist.js"
|
||||||
|
}
|
3
test/fixtures/packages/unparseable/package.json
vendored
Normal file
3
test/fixtures/packages/unparseable/package.json
vendored
Normal file
@ -0,0 +1,3 @@
|
|||||||
|
{
|
||||||
|
"main": "therain" "inspain"
|
||||||
|
}
|
@ -104,6 +104,12 @@ const d2 = require('../fixtures/b/d');
|
|||||||
assert.strictEqual(require('../fixtures/packages/index').ok, 'ok');
|
assert.strictEqual(require('../fixtures/packages/index').ok, 'ok');
|
||||||
assert.strictEqual(require('../fixtures/packages/main').ok, 'ok');
|
assert.strictEqual(require('../fixtures/packages/main').ok, 'ok');
|
||||||
assert.strictEqual(require('../fixtures/packages/main-index').ok, 'ok');
|
assert.strictEqual(require('../fixtures/packages/main-index').ok, 'ok');
|
||||||
|
assert.strictEqual(require('../fixtures/packages/missing-main').ok, 'ok');
|
||||||
|
|
||||||
|
assert.throws(
|
||||||
|
function() { require('../fixtures/packages/unparseable'); },
|
||||||
|
/^SyntaxError: Error parsing/
|
||||||
|
);
|
||||||
|
|
||||||
{
|
{
|
||||||
console.error('test cycles containing a .. path');
|
console.error('test cycles containing a .. path');
|
||||||
@ -267,6 +273,7 @@ try {
|
|||||||
'fixtures/packages/index/index.js': {},
|
'fixtures/packages/index/index.js': {},
|
||||||
'fixtures/packages/main/package-main-module.js': {},
|
'fixtures/packages/main/package-main-module.js': {},
|
||||||
'fixtures/packages/main-index/package-main-module/index.js': {},
|
'fixtures/packages/main-index/package-main-module/index.js': {},
|
||||||
|
'fixtures/packages/missing-main/index.js': {},
|
||||||
'fixtures/cycles/root.js': {
|
'fixtures/cycles/root.js': {
|
||||||
'fixtures/cycles/folder/foo.js': {}
|
'fixtures/cycles/folder/foo.js': {}
|
||||||
},
|
},
|
||||||
|
Loading…
x
Reference in New Issue
Block a user