Skip to content

Commit 8b3177b

Browse files
committed
esm: improve performance & tidy tests
nodejs/node#43784
1 parent c137633 commit 8b3177b

File tree

1 file changed

+11
-16
lines changed

1 file changed

+11
-16
lines changed

patches/node/json_parse_errors_made_user-friendly.patch

Lines changed: 11 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -19,21 +19,16 @@ index 9d0deb70a1568c93ccdecbef59327ecb2a17ae5e..2ab8f2bdcf7cca1437df33668c4177a7
1919
}
2020
{
2121
diff --git a/test/es-module/test-esm-invalid-pjson.js b/test/es-module/test-esm-invalid-pjson.js
22-
index cdbebb17b4bb34421a2f98c384650d495908885c..12247f15dbaddc0e06f1e6aff09faf7a035cf43a 100644
22+
index f3a38018637aa349ad79617ab9835e61d7058fe9..bc78b870c84c4baedecdd7ffc1157c86c307cebf 100644
2323
--- a/test/es-module/test-esm-invalid-pjson.js
2424
+++ b/test/es-module/test-esm-invalid-pjson.js
25-
@@ -17,11 +17,13 @@ child.stderr.on('data', (data) => {
26-
child.on('close', mustCall((code, signal) => {
27-
strictEqual(code, 1);
28-
strictEqual(signal, null);
29-
+ console.log('STDERR is: ', stderr);
30-
+ console.log('DONE STDERR');
31-
ok(
32-
stderr.includes(
33-
`[ERR_INVALID_PACKAGE_CONFIG]: Invalid package config ${invalidJson} ` +
34-
`while importing "invalid-pjson" from ${entry}. ` +
35-
- `Unexpected token } in JSON at position ${12 + checkoutEOL.length * 2}`
36-
+ `Expected ':' after property name in JSON at position ${12 + checkoutEOL.length * 2}`
37-
),
38-
stderr);
39-
}));
25+
@@ -18,7 +18,7 @@ describe('ESM: Package.json', { concurrency: true }, () => {
26+
stderr.includes(
27+
`[ERR_INVALID_PACKAGE_CONFIG]: Invalid package config ${invalidJson} ` +
28+
`while importing "invalid-pjson" from ${entry}. ` +
29+
- `Unexpected token } in JSON at position ${12 + checkoutEOL.length * 2}`
30+
+ `Expected ':' after property name in JSON at position ${12 + checkoutEOL.length * 2}`
31+
),
32+
stderr
33+
);
34+

0 commit comments

Comments
 (0)