Skip to content

Commit 5e04bf4

Browse files
phateddanez
authored andcommitted
Use noop importer for isStatelessComponent tests
1 parent 9dfe84c commit 5e04bf4

File tree

1 file changed

+15
-16
lines changed

1 file changed

+15
-16
lines changed

src/utils/__tests__/isStatelessComponent-test.js

Lines changed: 15 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
*
77
*/
88

9-
import { parse, statement } from '../../../tests/utils';
9+
import { parse, statement, noopImporter } from '../../../tests/utils';
1010
import isStatelessComponent from '../isStatelessComponent';
1111

1212
describe('isStatelessComponent', () => {
@@ -17,7 +17,6 @@ describe('isStatelessComponent', () => {
1717
'React.cloneElement': 'React.cloneElement(children, null)',
1818
'React.Children.only()': 'React.Children.only(children, null)',
1919
};
20-
const ignoreImports = () => null;
2120

2221
const componentStyle = {
2322
ArrowExpression: [
@@ -105,7 +104,7 @@ describe('isStatelessComponent', () => {
105104
...caseSelector,
106105
...componentSelector,
107106
);
108-
expect(isStatelessComponent(def, ignoreImports)).toBe(true);
107+
expect(isStatelessComponent(def, noopImporter)).toBe(true);
109108
});
110109
});
111110

@@ -121,7 +120,7 @@ describe('isStatelessComponent', () => {
121120
...caseSelector,
122121
...componentSelector,
123122
);
124-
expect(isStatelessComponent(def, ignoreImports)).toBe(false);
123+
expect(isStatelessComponent(def, noopImporter)).toBe(false);
125124
});
126125
});
127126
});
@@ -141,7 +140,7 @@ describe('isStatelessComponent', () => {
141140
.get('declarations', [0])
142141
.get('init');
143142

144-
expect(isStatelessComponent(def, ignoreImports)).toBe(true);
143+
expect(isStatelessComponent(def, noopImporter)).toBe(true);
145144
});
146145
});
147146

@@ -167,11 +166,11 @@ describe('isStatelessComponent', () => {
167166
const render = def.get('properties', 3);
168167
const world = def.get('properties', 4);
169168

170-
expect(isStatelessComponent(bar, ignoreImports)).toBe(true);
171-
expect(isStatelessComponent(baz, ignoreImports)).toBe(true);
172-
expect(isStatelessComponent(hello, ignoreImports)).toBe(true);
173-
expect(isStatelessComponent(render, ignoreImports)).toBe(false);
174-
expect(isStatelessComponent(world, ignoreImports)).toBe(true);
169+
expect(isStatelessComponent(bar, noopImporter)).toBe(true);
170+
expect(isStatelessComponent(baz, noopImporter)).toBe(true);
171+
expect(isStatelessComponent(hello, noopImporter)).toBe(true);
172+
expect(isStatelessComponent(render, noopImporter)).toBe(false);
173+
expect(isStatelessComponent(world, noopImporter)).toBe(true);
175174
});
176175
});
177176

@@ -184,7 +183,7 @@ describe('isStatelessComponent', () => {
184183
}
185184
}
186185
`);
187-
expect(isStatelessComponent(def, ignoreImports)).toBe(false);
186+
expect(isStatelessComponent(def, noopImporter)).toBe(false);
188187
});
189188

190189
it('does not accept React.Component classes', () => {
@@ -197,7 +196,7 @@ describe('isStatelessComponent', () => {
197196
}
198197
`).get('body', 1);
199198

200-
expect(isStatelessComponent(def, ignoreImports)).toBe(false);
199+
expect(isStatelessComponent(def, noopImporter)).toBe(false);
201200
});
202201

203202
it('does not accept React.createClass calls', () => {
@@ -209,7 +208,7 @@ describe('isStatelessComponent', () => {
209208
});
210209
`);
211210

212-
expect(isStatelessComponent(def, ignoreImports)).toBe(false);
211+
expect(isStatelessComponent(def, noopImporter)).toBe(false);
213212
});
214213
});
215214

@@ -218,7 +217,7 @@ describe('isStatelessComponent', () => {
218217
it(desc, () => {
219218
const def = parse(code).get('body', 1);
220219

221-
expect(isStatelessComponent(def, ignoreImports)).toBe(true);
220+
expect(isStatelessComponent(def, noopImporter)).toBe(true);
222221
});
223222
}
224223

@@ -231,7 +230,7 @@ describe('isStatelessComponent', () => {
231230
}
232231
`);
233232

234-
expect(isStatelessComponent(def, ignoreImports)).toBe(true);
233+
expect(isStatelessComponent(def, noopImporter)).toBe(true);
235234
});
236235

237236
it('handles recursive function calls', () => {
@@ -241,7 +240,7 @@ describe('isStatelessComponent', () => {
241240
}
242241
`);
243242

244-
expect(isStatelessComponent(def, ignoreImports)).toBe(false);
243+
expect(isStatelessComponent(def, noopImporter)).toBe(false);
245244
});
246245

247246
test(

0 commit comments

Comments
 (0)