Skip to content

Commit 28b3da0

Browse files
skovySunil Pai
authored andcommitted
Add TypeScript tests for the rename-unsafe-lifecycles transform (#236)
* Add TypeScript examples for the rename-unsafe-lifecycles * Run the TypeScript test case
1 parent a282cb7 commit 28b3da0

File tree

3 files changed

+73
-10
lines changed

3 files changed

+73
-10
lines changed
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
import React from "react";
2+
3+
class Component extends React.Component {
4+
public componentWillMount() {}
5+
public componentDidMount() {}
6+
public componentWillReceiveProps() {}
7+
public shouldComponentUpdate() {}
8+
public componentWillUpdate() {}
9+
public componentDidUpdate() {}
10+
public componentWillUnmount() {}
11+
public render() {
12+
return null;
13+
}
14+
}
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
import React from "react";
2+
3+
class Component extends React.Component {
4+
public UNSAFE_componentWillMount() {}
5+
public componentDidMount() {}
6+
public UNSAFE_componentWillReceiveProps() {}
7+
public shouldComponentUpdate() {}
8+
public UNSAFE_componentWillUpdate() {}
9+
public componentDidUpdate() {}
10+
public componentWillUnmount() {}
11+
public render() {
12+
return null;
13+
}
14+
}

transforms/__tests__/rename-unsafe-lifecycles-test.js

Lines changed: 45 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,6 @@
1010

1111
'use strict';
1212

13-
jest.mock('../rename-unsafe-lifecycles', () => {
14-
return Object.assign(require.requireActual('../rename-unsafe-lifecycles'), {
15-
parser: 'flow'
16-
});
17-
});
18-
1913
const tests = [
2014
'arrow-functions',
2115
'create-react-class',
@@ -30,12 +24,53 @@ const tests = [
3024
const defineTest = require('jscodeshift/dist/testUtils').defineTest;
3125

3226
describe('rename-unsafe-lifecycles', () => {
33-
tests.forEach(test =>
27+
describe('flow', () => {
28+
beforeEach(() => {
29+
jest.mock('../rename-unsafe-lifecycles', () => {
30+
return Object.assign(
31+
require.requireActual('../rename-unsafe-lifecycles'),
32+
{
33+
parser: 'flow'
34+
}
35+
);
36+
});
37+
});
38+
39+
afterEach(() => {
40+
jest.resetModules();
41+
});
42+
43+
tests.forEach(test =>
44+
defineTest(
45+
__dirname,
46+
'rename-unsafe-lifecycles',
47+
null,
48+
`rename-unsafe-lifecycles/${test}`
49+
)
50+
);
51+
});
52+
53+
describe('typescript', () => {
54+
beforeEach(() => {
55+
jest.mock('../rename-unsafe-lifecycles', () => {
56+
return Object.assign(
57+
require.requireActual('../rename-unsafe-lifecycles'),
58+
{
59+
parser: 'tsx'
60+
}
61+
);
62+
});
63+
});
64+
65+
afterEach(() => {
66+
jest.resetModules();
67+
});
68+
3469
defineTest(
3570
__dirname,
3671
'rename-unsafe-lifecycles',
3772
null,
38-
`rename-unsafe-lifecycles/${test}`
39-
)
40-
);
73+
'rename-unsafe-lifecycles/typescript/class.tsx'
74+
);
75+
});
4176
});

0 commit comments

Comments
 (0)