diff options
author | Guy Bedford <guybedford@gmail.com> | 2019-12-05 00:11:04 -0500 |
---|---|---|
committer | Guy Bedford <guybedford@gmail.com> | 2019-12-05 02:49:17 -0500 |
commit | 781e41df618a8d8878fb1c819855a31ef75115d1 (patch) | |
tree | d4125249f8d8fb783f89a3c9e775941cf59aab24 /test/es-module | |
parent | 1549c8e077422b79573936960daaa4c695620310 (diff) | |
download | android-node-v8-781e41df618a8d8878fb1c819855a31ef75115d1.tar.gz android-node-v8-781e41df618a8d8878fb1c819855a31ef75115d1.tar.bz2 android-node-v8-781e41df618a8d8878fb1c819855a31ef75115d1.zip |
module: fixup lint and test regressions
PR-URL: https://github.com/nodejs/node/pull/30802
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: João Reis <reis@janeasystems.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Diffstat (limited to 'test/es-module')
-rw-r--r-- | test/es-module/test-cjs-esm-warn.js | 11 | ||||
-rw-r--r-- | test/es-module/test-esm-specifiers-both-flags.mjs | 4 |
2 files changed, 5 insertions, 10 deletions
diff --git a/test/es-module/test-cjs-esm-warn.js b/test/es-module/test-cjs-esm-warn.js index b800a47d05..ddeda72fc8 100644 --- a/test/es-module/test-cjs-esm-warn.js +++ b/test/es-module/test-cjs-esm-warn.js @@ -26,7 +26,7 @@ child.on('close', common.mustCall((code, signal) => { assert.strictEqual(code, 1); assert.strictEqual(signal, null); - assert.ok(stderr.indexOf( + assert.ok(stderr.replace(/\r/g, '').includes( `Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: ${required}` + '\nrequire() of ES modules is not supported.\nrequire() of ' + `${required} from ${requiring} ` + @@ -35,10 +35,7 @@ child.on('close', common.mustCall((code, signal) => { 'files in that package scope as ES modules.\nInstead rename ' + `${basename} to end in .cjs, change the requiring code to use ` + 'import(), or remove "type": "module" from ' + - `${pjson}.\n`) !== -1); - assert.ok(stderr.indexOf( - 'Error [ERR_REQUIRE_ESM]: Must use import to load ES Module') !== -1); - - assert.strictEqual( - stderr.match(/Must use import to load ES Module/g).length, 1); + `${pjson}.\n`)); + assert.ok(stderr.includes( + 'Error [ERR_REQUIRE_ESM]: Must use import to load ES Module')); })); diff --git a/test/es-module/test-esm-specifiers-both-flags.mjs b/test/es-module/test-esm-specifiers-both-flags.mjs index fc5c7fcd0e..a434684589 100644 --- a/test/es-module/test-esm-specifiers-both-flags.mjs +++ b/test/es-module/test-esm-specifiers-both-flags.mjs @@ -9,8 +9,6 @@ const expectedError = const flags = '--es-module-specifier-resolution=node ' + '--experimental-specifier-resolution=node'; -exec(`${process.execPath} ${flags}`, { - timeout: 300 -}, mustCall((error) => { +exec(`${process.execPath} ${flags}`, mustCall((error) => { assert(error.message.includes(expectedError)); })); |