diff --git a/.eslintignore b/.eslintignore index 80959ce611ee58..bdfdfaeab2388d 100644 --- a/.eslintignore +++ b/.eslintignore @@ -1,9 +1,7 @@ node_modules -lib/internal/v8.js lib/internal/v8_prof_polyfill.js lib/punycode.js test/addons/??_* -test/es-module/test-esm-dynamic-import.js test/fixtures test/message/esm_display_syntax_error.mjs tools/icu diff --git a/test/es-module/test-esm-dynamic-import.js b/test/es-module/test-esm-dynamic-import.js index 0023b5f2d80808..887339977669ea 100644 --- a/test/es-module/test-esm-dynamic-import.js +++ b/test/es-module/test-esm-dynamic-import.js @@ -11,7 +11,7 @@ targetURL.pathname = absolutePath; function expectErrorProperty(result, propertyKey, value) { Promise.resolve(result) - .catch(common.mustCall(error => { + .catch(common.mustCall((error) => { assert.strictEqual(error[propertyKey], value); })); } @@ -22,15 +22,16 @@ function expectMissingModuleError(result) { function expectOkNamespace(result) { Promise.resolve(result) - .then(common.mustCall(ns => { + .then(common.mustCall((ns) => { // Can't deepStrictEqual because ns isn't a normal object + // eslint-disable-next-line no-restricted-properties assert.deepEqual(ns, { default: true }); })); } function expectFsNamespace(result) { Promise.resolve(result) - .then(common.mustCall(ns => { + .then(common.mustCall((ns) => { assert.strictEqual(typeof ns.default.writeFile, 'function'); assert.strictEqual(typeof ns.writeFile, 'function'); })); @@ -41,19 +42,19 @@ function expectFsNamespace(result) { (function testScriptOrModuleImport() { // Importing another file, both direct & via eval // expectOkNamespace(import(relativePath)); - expectOkNamespace(eval.call(null, `import("${relativePath}")`)); expectOkNamespace(eval(`import("${relativePath}")`)); - expectOkNamespace(eval.call(null, `import("${targetURL}")`)); + expectOkNamespace(eval(`import("${relativePath}")`)); + expectOkNamespace(eval(`import("${targetURL}")`)); // Importing a built-in, both direct & via eval - expectFsNamespace(import("fs")); + expectFsNamespace(import('fs')); + expectFsNamespace(eval('import("fs")')); expectFsNamespace(eval('import("fs")')); - expectFsNamespace(eval.call(null, 'import("fs")')); - expectMissingModuleError(import("./not-an-existing-module.mjs")); + expectMissingModuleError(import('./not-an-existing-module.mjs')); // TODO(jkrems): Right now this doesn't hit a protocol error because the // module resolution step already rejects it. These arguably should be // protocol errors. - expectMissingModuleError(import("node:fs")); + expectMissingModuleError(import('node:fs')); expectMissingModuleError(import('http://example.com/foo.js')); })();