test: run eslint on test file and fix errors

This removes two entries from the eslint ignore file. One file does not exist anymore and the other one could easily be fixed. PR-URL: #25009 Reviewed-By: Sakthipriyan Vairamani <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Feb 12, 2019 · bcfc1d1 · bcfc1d1
1 parent 1752798
commit bcfc1d1
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 11 deletions.
diff --git 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
@@ -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'));
 })();