Skip to content

Commit baa3f88

Browse files
committed
fixup! esm: refactor esm tests out of test/message
1 parent aeebeff commit baa3f88

6 files changed

+21
-20
lines changed

test/es-module/test-esm-export-not-found.mjs

+6-4
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,11 @@ const importStatementMultiline = `import {
2828
stderr += data;
2929
});
3030
child.on('close', () => {
31-
ok(stderr.includes(
32-
'SyntaxError: The requested module \'./module-named-exports.mjs\' ' +
33-
'does not provide an export named \'notfound\''
34-
));
31+
// SyntaxError: The requested module './module-named-exports.mjs'
32+
// does not provide an export named 'notfound'
33+
ok(stderr.includes('SyntaxError:'));
34+
// The quotes ensure that the path starts with ./ and not ../
35+
ok(stderr.includes('\'./module-named-exports.mjs\''));
36+
ok(stderr.includes('notfound'));
3537
});
3638
});

test/es-module/test-esm-import-json-named-export.mjs

+5-4
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,9 @@ child.stderr.on('data', (data) => {
1515
stderr += data;
1616
});
1717
child.on('close', () => {
18-
ok(stderr.includes(
19-
'SyntaxError: The requested module \'../experimental.json\' ' +
20-
'does not provide an export named \'ofLife\''
21-
));
18+
// SyntaxError: The requested module '../experimental.json'
19+
// does not provide an export named 'ofLife'
20+
ok(stderr.includes('SyntaxError:'));
21+
ok(stderr.includes('\'../experimental.json\''));
22+
ok(stderr.includes('\'ofLife\''));
2223
});

test/es-module/test-esm-loader-not-found.mjs

+5-4
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,10 @@ child.stderr.on('data', (data) => {
1616
stderr += data;
1717
});
1818
child.on('close', () => {
19-
ok(stderr.includes(
20-
'Error [ERR_MODULE_NOT_FOUND]: Cannot find package \'i-dont-exist\' ' +
21-
'imported from'
22-
));
19+
// Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'i-dont-exist'
20+
// imported from
21+
ok(stderr.includes('ERR_MODULE_NOT_FOUND'));
22+
ok(stderr.includes('\'i-dont-exist\''));
23+
2324
ok(!stderr.includes('Bad command or file name'));
2425
});

test/es-module/test-esm-loader-with-syntax-error.mjs

+1-3
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,6 @@ child.stderr.on('data', (data) => {
1616
stderr += data;
1717
});
1818
child.on('close', () => {
19-
ok(stderr.includes(
20-
'SyntaxError: Malformed arrow function parameter list'
21-
));
19+
ok(stderr.includes('SyntaxError:'));
2220
ok(!stderr.includes('Bad command or file name'));
2321
});

test/es-module/test-esm-module-not-found-commonjs-hint.mjs

+3-2
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,12 @@ import { execPath } from 'process';
77
[
88
{
99
input: 'import "./print-error-message"',
10-
expected: 'Did you mean to import ../print-error-message.js?'
10+
// Did you mean to import ../print-error-message.js?
11+
expected: ' ../print-error-message.js?'
1112
},
1213
{
1314
input: 'import obj from "some_module/obj"',
14-
expected: 'Did you mean to import some_module/obj.js?'
15+
expected: ' some_module/obj.js?'
1516
},
1617
].forEach(({ input, expected }) => {
1718
const child = spawn(execPath, [

test/es-module/test-esm-syntax-error.mjs

+1-3
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,5 @@ child.stderr.on('data', (data) => {
1414
stderr += data;
1515
});
1616
child.on('close', () => {
17-
ok(stderr.includes(
18-
'SyntaxError: Malformed arrow function parameter list'
19-
));
17+
ok(stderr.includes('SyntaxError:'));
2018
});

0 commit comments

Comments
 (0)