Skip to content

Commit 25558f4

Browse files
committed
fix: MockList should not trigger onSkipRender
1 parent f245d22 commit 25558f4

File tree

2 files changed

+16
-0
lines changed

2 files changed

+16
-0
lines changed

src/mock.tsx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,10 @@ import OriginList from './List';
33
import Filler from './Filler';
44

55
class List<T> extends OriginList<T> {
6+
componentDidUpdate() {
7+
// Do nothing since this is a mock!
8+
}
9+
610
render() {
711
const {
812
style,

tests/mock.test.js

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,4 +22,16 @@ describe('MockList', () => {
2222
).toBe(String(i));
2323
}
2424
});
25+
26+
it('do not trigger `onSkipRender`', () => {
27+
const onSkipRender = jest.fn();
28+
const wrapper = mount(
29+
<MockList data={[0]} itemKey={id => id}>
30+
{id => <span>{id}</span>}
31+
</MockList>,
32+
);
33+
34+
wrapper.setProps({ data: [0, 1] });
35+
expect(onSkipRender).not.toHaveBeenCalled();
36+
});
2537
});

0 commit comments

Comments
 (0)