Skip to content

Commit 75170fa

Browse files
committedAug 24, 2021
[Tests] named: add passing tests
Related to #2199
1 parent 4aff27c commit 75170fa

File tree

3 files changed

+74
-87
lines changed

3 files changed

+74
-87
lines changed
 

‎tests/files/export-star-3/b.ts

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export * as b from './c';

‎tests/files/export-star-3/c.ts

Whitespace-only changes.

‎tests/src/rules/named.js

+73-87
Original file line numberDiff line numberDiff line change
@@ -352,105 +352,91 @@ ruleTester.run('named (export *)', rule, {
352352

353353
context('TypeScript', function () {
354354
getTSParsers().forEach((parser) => {
355+
const settings = {
356+
'import/parsers': { [parser]: ['.ts'] },
357+
'import/resolver': { 'eslint-import-resolver-typescript': true },
358+
};
359+
360+
const valid = [];
361+
const invalid = [
362+
test({
363+
code: `import {a} from './export-star-3/b';`,
364+
filename: testFilePath('./export-star-3/a.js'),
365+
parser: parser,
366+
settings: settings,
367+
}),
368+
];
369+
355370
[
356371
'typescript',
357372
'typescript-declare',
358373
'typescript-export-assign-namespace',
359374
'typescript-export-assign-namespace-merged',
360375
].forEach((source) => {
361-
ruleTester.run(`named`, rule, {
362-
valid: [
363-
test({
364-
code: `import { MyType } from "./${source}"`,
365-
parser: parser,
366-
settings: {
367-
'import/parsers': { [parser]: ['.ts'] },
368-
'import/resolver': { 'eslint-import-resolver-typescript': true },
369-
},
370-
}),
371-
test({
372-
code: `import { Foo } from "./${source}"`,
373-
parser: parser,
374-
settings: {
375-
'import/parsers': { [parser]: ['.ts'] },
376-
'import/resolver': { 'eslint-import-resolver-typescript': true },
377-
},
378-
}),
379-
test({
380-
code: `import { Bar } from "./${source}"`,
381-
parser: parser,
382-
settings: {
383-
'import/parsers': { [parser]: ['.ts'] },
384-
'import/resolver': { 'eslint-import-resolver-typescript': true },
385-
},
386-
}),
387-
test({
388-
code: `import { getFoo } from "./${source}"`,
389-
parser: parser,
390-
settings: {
391-
'import/parsers': { [parser]: ['.ts'] },
392-
'import/resolver': { 'eslint-import-resolver-typescript': true },
393-
},
394-
}),
395-
test({
396-
code: `import { MyEnum } from "./${source}"`,
397-
parser: parser,
398-
settings: {
399-
'import/parsers': { [parser]: ['.ts'] },
400-
'import/resolver': { 'eslint-import-resolver-typescript': true },
401-
},
402-
}),
403-
test({
404-
code: `
376+
valid.push(
377+
test({
378+
code: `import { MyType } from "./${source}"`,
379+
parser: parser,
380+
settings: settings,
381+
}),
382+
test({
383+
code: `import { Foo } from "./${source}"`,
384+
parser: parser,
385+
settings: settings,
386+
}),
387+
test({
388+
code: `import { Bar } from "./${source}"`,
389+
parser: parser,
390+
settings: settings,
391+
}),
392+
test({
393+
code: `import { getFoo } from "./${source}"`,
394+
parser: parser,
395+
settings: settings,
396+
}),
397+
test({
398+
code: `import { MyEnum } from "./${source}"`,
399+
parser: parser,
400+
settings: settings,
401+
}),
402+
test({
403+
code: `
405404
import { MyModule } from "./${source}"
406405
MyModule.ModuleFunction()
407406
`,
408-
parser: parser,
409-
settings: {
410-
'import/parsers': { [parser]: ['.ts'] },
411-
'import/resolver': { 'eslint-import-resolver-typescript': true },
412-
},
413-
}),
414-
test({
415-
code: `
407+
parser: parser,
408+
settings: settings,
409+
}),
410+
test({
411+
code: `
416412
import { MyNamespace } from "./${source}"
417413
MyNamespace.NSModule.NSModuleFunction()
418414
`,
419-
parser: parser,
420-
settings: {
421-
'import/parsers': { [parser]: ['.ts'] },
422-
'import/resolver': { 'eslint-import-resolver-typescript': true },
423-
},
424-
}),
425-
],
426-
427-
invalid: [
428-
test({
429-
code: `import { MissingType } from "./${source}"`,
430-
parser: parser,
431-
settings: {
432-
'import/parsers': { [parser]: ['.ts'] },
433-
'import/resolver': { 'eslint-import-resolver-typescript': true },
434-
},
435-
errors: [{
436-
message: `MissingType not found in './${source}'`,
437-
type: 'Identifier',
438-
}],
439-
}),
440-
test({
441-
code: `import { NotExported } from "./${source}"`,
442-
parser: parser,
443-
settings: {
444-
'import/parsers': { [parser]: ['.ts'] },
445-
'import/resolver': { 'eslint-import-resolver-typescript': true },
446-
},
447-
errors: [{
448-
message: `NotExported not found in './${source}'`,
449-
type: 'Identifier',
450-
}],
451-
}),
452-
],
453-
});
415+
parser: parser,
416+
settings: settings,
417+
}),
418+
);
419+
420+
invalid.push(
421+
test({
422+
code: `import { MissingType } from "./${source}"`,
423+
parser: parser,
424+
settings: settings,
425+
errors: [{
426+
message: `MissingType not found in './${source}'`,
427+
type: 'Identifier',
428+
}],
429+
}),
430+
test({
431+
code: `import { NotExported } from "./${source}"`,
432+
parser: parser,
433+
settings: settings,
434+
errors: [{
435+
message: `NotExported not found in './${source}'`,
436+
type: 'Identifier',
437+
}],
438+
}),
439+
);
454440
});
455441
});
456442
});

0 commit comments

Comments
 (0)
Please sign in to comment.