Skip to content

Commit 6d476ef

Browse files
phateddanez
authored andcommitted
Switch resolveExportDeclaration tests to use noopImporter
1 parent fa878b8 commit 6d476ef

File tree

1 file changed

+14
-24
lines changed

1 file changed

+14
-24
lines changed

src/utils/__tests__/resolveExportDeclaration-test.js

Lines changed: 14 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -8,74 +8,64 @@
88

99
jest.mock('../resolveToValue');
1010

11-
import { statement } from '../../../tests/utils';
11+
import { statement, noopImporter } from '../../../tests/utils';
1212
import resolveToValue from '../resolveToValue';
1313
import resolveExportDeclaration from '../resolveExportDeclaration';
1414

1515
describe('resolveExportDeclaration', () => {
1616
const returnValue = {};
17-
const ignoreImports = () => null;
1817

1918
beforeEach(() => {
2019
resolveToValue.mockReturnValue(returnValue);
2120
});
2221

2322
it('resolves default exports', () => {
2423
const exp = statement('export default 42;');
25-
const resolved = resolveExportDeclaration(exp, ignoreImports);
24+
const resolved = resolveExportDeclaration(exp, noopImporter);
2625

2726
expect(resolved).toEqual([returnValue]);
28-
expect(resolveToValue).toBeCalledWith(
29-
exp.get('declaration'),
30-
ignoreImports,
31-
);
27+
expect(resolveToValue).toBeCalledWith(exp.get('declaration'), noopImporter);
3228
});
3329

3430
it('resolves named exports', () => {
3531
let exp = statement('export var foo = 42, bar = 21;');
36-
let resolved = resolveExportDeclaration(exp, ignoreImports);
32+
let resolved = resolveExportDeclaration(exp, noopImporter);
3733

3834
const declarations = exp.get('declaration', 'declarations');
3935
expect(resolved).toEqual([returnValue, returnValue]);
40-
expect(resolveToValue).toBeCalledWith(declarations.get(0), ignoreImports);
41-
expect(resolveToValue).toBeCalledWith(declarations.get(1), ignoreImports);
36+
expect(resolveToValue).toBeCalledWith(declarations.get(0), noopImporter);
37+
expect(resolveToValue).toBeCalledWith(declarations.get(1), noopImporter);
4238

4339
exp = statement('export function foo(){}');
44-
resolved = resolveExportDeclaration(exp, ignoreImports);
40+
resolved = resolveExportDeclaration(exp, noopImporter);
4541

4642
expect(resolved).toEqual([returnValue]);
47-
expect(resolveToValue).toBeCalledWith(
48-
exp.get('declaration'),
49-
ignoreImports,
50-
);
43+
expect(resolveToValue).toBeCalledWith(exp.get('declaration'), noopImporter);
5144

5245
exp = statement('export class Foo {}');
53-
resolved = resolveExportDeclaration(exp, ignoreImports);
46+
resolved = resolveExportDeclaration(exp, noopImporter);
5447

5548
expect(resolved).toEqual([returnValue]);
56-
expect(resolveToValue).toBeCalledWith(
57-
exp.get('declaration'),
58-
ignoreImports,
59-
);
49+
expect(resolveToValue).toBeCalledWith(exp.get('declaration'), noopImporter);
6050
});
6151

6252
it('resolves named exports', () => {
6353
const exp = statement('export {foo, bar, baz}; var foo, bar, baz;');
64-
const resolved = resolveExportDeclaration(exp, ignoreImports);
54+
const resolved = resolveExportDeclaration(exp, noopImporter);
6555

6656
const specifiers = exp.get('specifiers');
6757
expect(resolved).toEqual([returnValue, returnValue, returnValue]);
6858
expect(resolveToValue).toBeCalledWith(
6959
specifiers.get(0, 'local'),
70-
ignoreImports,
60+
noopImporter,
7161
);
7262
expect(resolveToValue).toBeCalledWith(
7363
specifiers.get(1, 'local'),
74-
ignoreImports,
64+
noopImporter,
7565
);
7666
expect(resolveToValue).toBeCalledWith(
7767
specifiers.get(2, 'local'),
78-
ignoreImports,
68+
noopImporter,
7969
);
8070
});
8171
});

0 commit comments

Comments
 (0)