Skip to content

Commit 21ba3e3

Browse files
bnoordhuisevanlucas
authored andcommitted
test: add more module loader test coverage
Verify that a package.json without a .main property loads index.js. PR-URL: #9196 Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 2044619 commit 21ba3e3

File tree

3 files changed

+5
-0
lines changed

3 files changed

+5
-0
lines changed

test/fixtures/packages/index/index.js

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
exports.ok = 'ok';
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
{}

test/sequential/test-module-loading.js

+3
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,8 @@ assert.equal(threeFolder, threeIndex);
6969
assert.notEqual(threeFolder, three);
7070

7171
console.error('test package.json require() loading');
72+
assert.equal(require('../fixtures/packages/index').ok, 'ok',
73+
'Failed loading package');
7274
assert.equal(require('../fixtures/packages/main').ok, 'ok',
7375
'Failed loading package');
7476
assert.equal(require('../fixtures/packages/main-index').ok, 'ok',
@@ -208,6 +210,7 @@ assert.deepStrictEqual(children, {
208210
},
209211
'fixtures/nested-index/three.js': {},
210212
'fixtures/nested-index/three/index.js': {},
213+
'fixtures/packages/index/index.js': {},
211214
'fixtures/packages/main/package-main-module.js': {},
212215
'fixtures/packages/main-index/package-main-module/index.js': {},
213216
'fixtures/cycles/root.js': {

0 commit comments

Comments
 (0)