Skip to content

Commit 357ee0b

Browse files
parserOptions for failing tests
1 parent 0eb9bc6 commit 357ee0b

2 files changed

Lines changed: 60 additions & 7 deletions

File tree

tests/src/parsers.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,9 @@ const parsers = {
111111
|| features.has('flow')
112112
|| features.has('ts')
113113
|| features.has('types')
114-
|| (features.has('fragment') && semver.satisfies(version, '< 5'));
114+
//if it has fragments use version 5 and higher // create features for export from
115+
|| (features.has('fragment') && semver.satisfies(version, '< 5'))
116+
|| features.has('');
115117

116118
const skipBabel = features.has('no-babel');
117119
const skipOldBabel = skipBabel || features.has('no-babel-old') || semver.satisfies(version, '>= 8');

tests/src/rules/default.js

Lines changed: 57 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,25 +50,57 @@ ruleTester.run('default', rule, {
5050
test({ code: 'import common from "./common";' }),
5151

5252
// es7 export syntax
53-
test({ code: 'export bar from "./bar"' }),
53+
test({
54+
code: 'export bar from "./bar"',
55+
parserOptions: {
56+
ecmaVersion: 2018,
57+
ecmaFeatures: {
58+
modules: true,
59+
},
60+
},
61+
}),
5462
test({ code: 'export { default as bar } from "./bar"' }),
5563
test({ code: 'export bar, { foo } from "./bar"' }),
5664
test({ code: 'export { default as bar, foo } from "./bar"' }),
57-
test({ code: 'export bar, * as names from "./bar"' }),
65+
test({ code: 'export bar, * as names from "./bar"',
66+
parserOptions: {
67+
ecmaVersion: 2018,
68+
ecmaFeatures: {
69+
modules: true,
70+
},
71+
} }),
5872

5973
// sanity check
6074
test({ code: 'export {a} from "./named-exports"' }),
61-
test({ code: 'import twofer from "./trampoline"' }),
75+
test({
76+
code: 'import twofer from "./trampoline"',
77+
parserOptions: {
78+
sourceType: 'module',
79+
ecmaVersion: 2015,
80+
},
81+
}),
6282

6383
// jsx
64-
test({ code: 'import MyCoolComponent from "./jsx/MyCoolComponent.jsx"' }),
84+
test({
85+
code: 'import MyCoolComponent from "./jsx/MyCoolComponent.jsx"',
86+
parserOptions: {
87+
sourceType: 'module',
88+
ecmaVersion: 6,
89+
ecmaFeatures: { jsx: true },
90+
},
91+
}),
6592

6693
// #54: import of named export default
6794
test({ code: 'import foo from "./named-default-export"' }),
6895

6996
// #94: redux export of execution result,
7097
test({ code: 'import connectedApp from "./redux"' }),
71-
test({ code: 'import App from "./jsx/App"' }),
98+
test({
99+
code: 'import App from "./jsx/App"',
100+
parserOptions: {
101+
ecmaFeatures: { jsx: true, modules: true },
102+
},
103+
}),
72104

73105
// from no-errors
74106
test({ code: "import Foo from './jsx/FooES7.js';" }),
@@ -92,7 +124,7 @@ ruleTester.run('default', rule, {
92124
'import/resolver': { 'eslint-import-resolver-typescript': true },
93125
},
94126
//feature
95-
features: ['ts'],
127+
//features: ['ts'],
96128
}),
97129
test({
98130
code: `import foobar from "./typescript-export-assign-default"`,
@@ -118,31 +150,47 @@ ruleTester.run('default', rule, {
118150
'import/resolver': { 'eslint-import-resolver-typescript': true },
119151
},
120152
}),
153+
121154
test({
122155
code: `import React from "./typescript-export-assign-default-namespace"`,
123156
settings: {
124157
'import/resolver': { 'eslint-import-resolver-typescript': true },
125158
},
159+
parserOptions: {
160+
tsconfigRootDir: path.resolve(__dirname, '../../files/typescript-export-assign-default-namespace/'),
161+
},
126162
}),
163+
127164
test({
128165
code: `import Foo from "./typescript-export-as-default-namespace"`,
129166
settings: {
130167
'import/resolver': { 'eslint-import-resolver-typescript': true },
131168
},
169+
parserOptions: {
170+
tsconfigRootDir: path.resolve(__dirname, '../../files/typescript-export-assign-default-namespace/'),
171+
},
132172
}),
173+
133174
test({
134175
code: `import Foo from "./typescript-export-react-test-renderer"`,
135176
settings: {
136177
'import/resolver': { 'eslint-import-resolver-typescript': true },
137178
},
179+
parserOptions: {
180+
tsconfigRootDir: path.resolve(__dirname, '../../files/typescript-export-react-test-renderer/'),
181+
},
138182
}),
139183
test({
140184
code: `import Foo from "./typescript-extended-config"`,
141185
settings: {
142186
'import/resolver': { 'eslint-import-resolver-typescript': true },
143187
},
188+
parserOptions: {
189+
tsconfigRootDir: path.resolve(__dirname, '../../files/typescript-extended-config/'),
190+
},
144191
}),
145192
test({
193+
146194
code: `import foobar from "./typescript-export-assign-property"`,
147195
settings: {
148196
'import/resolver': { 'eslint-import-resolver-typescript': true },
@@ -233,6 +281,9 @@ ruleTester.run('default', rule, {
233281
settings: {
234282
'import/resolver': { 'eslint-import-resolver-typescript': true },
235283
},
284+
parserOptions: {
285+
tsconfigRootDir: path.resolve(__dirname, '../../files/typescript-no-compiler-options/'),
286+
},
236287
errors: [
237288
{
238289
message: 'No default export found in imported module "./typescript-export-as-default-namespace".',

0 commit comments

Comments
 (0)