[Tests] named: add passing tests

Related to import-js#2199
ljharb · Aug 24, 2021 · 75170fa · 75170fa
1 parent 4aff27c
commit 75170fa
Show file tree

Hide file tree

Showing 3 changed files with 74 additions and 87 deletions.
diff --git a/tests/files/export-star-3/b.ts b/tests/files/export-star-3/b.ts
@@ -0,0 +1 @@
+export * as b from './c';
diff --git a/tests/files/export-star-3/c.ts b/tests/files/export-star-3/c.ts
diff --git a/tests/src/rules/named.js b/tests/src/rules/named.js
@@ -352,105 +352,91 @@ ruleTester.run('named (export *)', rule, {
 
 context('TypeScript', function () {
   getTSParsers().forEach((parser) => {
+    const settings = {
+      'import/parsers': { [parser]: ['.ts'] },
+      'import/resolver': { 'eslint-import-resolver-typescript': true },
+    };
+
+    const valid = [];
+    const invalid = [
+      test({
+        code: `import {a} from './export-star-3/b';`,
+        filename: testFilePath('./export-star-3/a.js'),
+        parser: parser,
+        settings: settings,
+      }),
+    ];
+
     [
       'typescript',
       'typescript-declare',
       'typescript-export-assign-namespace',
       'typescript-export-assign-namespace-merged',
     ].forEach((source) => {
-      ruleTester.run(`named`, rule, {
-        valid: [
-          test({
-            code: `import { MyType } from "./${source}"`,
-            parser: parser,
-            settings: {
-              'import/parsers': { [parser]: ['.ts'] },
-              'import/resolver': { 'eslint-import-resolver-typescript': true },
-            },
-          }),
-          test({
-            code: `import { Foo } from "./${source}"`,
-            parser: parser,
-            settings: {
-              'import/parsers': { [parser]: ['.ts'] },
-              'import/resolver': { 'eslint-import-resolver-typescript': true },
-            },
-          }),
-          test({
-            code: `import { Bar } from "./${source}"`,
-            parser: parser,
-            settings: {
-              'import/parsers': { [parser]: ['.ts'] },
-              'import/resolver': { 'eslint-import-resolver-typescript': true },
-            },
-          }),
-          test({
-            code: `import { getFoo } from "./${source}"`,
-            parser: parser,
-            settings: {
-              'import/parsers': { [parser]: ['.ts'] },
-              'import/resolver': { 'eslint-import-resolver-typescript': true },
-            },
-          }),
-          test({
-            code: `import { MyEnum } from "./${source}"`,
-            parser: parser,
-            settings: {
-              'import/parsers': { [parser]: ['.ts'] },
-              'import/resolver': { 'eslint-import-resolver-typescript': true },
-            },
-          }),
-          test({
-            code: `
+      valid.push(
+        test({
+          code: `import { MyType } from "./${source}"`,
+          parser: parser,
+          settings: settings,
+        }),
+        test({
+          code: `import { Foo } from "./${source}"`,
+          parser: parser,
+          settings: settings,
+        }),
+        test({
+          code: `import { Bar } from "./${source}"`,
+          parser: parser,
+          settings: settings,
+        }),
+        test({
+          code: `import { getFoo } from "./${source}"`,
+          parser: parser,
+          settings: settings,
+        }),
+        test({
+          code: `import { MyEnum } from "./${source}"`,
+          parser: parser,
+          settings: settings,
+        }),
+        test({
+          code: `
               import { MyModule } from "./${source}"
               MyModule.ModuleFunction()
             `,
-            parser: parser,
-            settings: {
-              'import/parsers': { [parser]: ['.ts'] },
-              'import/resolver': { 'eslint-import-resolver-typescript': true },
-            },
-          }),
-          test({
-            code: `
+          parser: parser,
+          settings: settings,
+        }),
+        test({
+          code: `
               import { MyNamespace } from "./${source}"
               MyNamespace.NSModule.NSModuleFunction()
             `,
-            parser: parser,
-            settings: {
-              'import/parsers': { [parser]: ['.ts'] },
-              'import/resolver': { 'eslint-import-resolver-typescript': true },
-            },
-          }),
-        ],
-
-        invalid: [
-          test({
-            code: `import { MissingType } from "./${source}"`,
-            parser: parser,
-            settings: {
-              'import/parsers': { [parser]: ['.ts'] },
-              'import/resolver': { 'eslint-import-resolver-typescript': true },
-            },
-            errors: [{
-              message: `MissingType not found in './${source}'`,
-              type: 'Identifier',
-            }],
-          }),
-          test({
-            code: `import { NotExported } from "./${source}"`,
-            parser: parser,
-            settings: {
-              'import/parsers': { [parser]: ['.ts'] },
-              'import/resolver': { 'eslint-import-resolver-typescript': true },
-            },
-            errors: [{
-              message: `NotExported not found in './${source}'`,
-              type: 'Identifier',
-            }],
-          }),
-        ],
-      });
+          parser: parser,
+          settings: settings,
+        }),
+      );
+
+      invalid.push(
+        test({
+          code: `import { MissingType } from "./${source}"`,
+          parser: parser,
+          settings: settings,
+          errors: [{
+            message: `MissingType not found in './${source}'`,
+            type: 'Identifier',
+          }],
+        }),
+        test({
+          code: `import { NotExported } from "./${source}"`,
+          parser: parser,
+          settings: settings,
+          errors: [{
+            message: `NotExported not found in './${source}'`,
+            type: 'Identifier',
+          }],
+        }),
+      );
     });
   });
 });