Skip to content

Commit 7f46856

Browse files
muzeamuzea
committed
test: wrap text with act
1 parent 3d00479 commit 7f46856

File tree

2 files changed

+84
-58
lines changed

2 files changed

+84
-58
lines changed

‎tests/scroll.test.js‎

Lines changed: 37 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,9 @@ describe('List.Scroll', () => {
5252
jest.useFakeTimers();
5353
const listRef = React.createRef();
5454
const wrapper = genList({ itemHeight: 20, height: 100, data: genData(100), ref: listRef });
55-
jest.runAllTimers();
55+
act(() => {
56+
jest.runAllTimers();
57+
});
5658

5759
listRef.current.scrollTo(null);
5860
expect(wrapper.find('.rc-virtual-list-scrollbar-thumb').props().style.display).not.toEqual(
@@ -65,8 +67,10 @@ describe('List.Scroll', () => {
6567
it('value scroll', () => {
6668
const listRef = React.createRef();
6769
const wrapper = genList({ itemHeight: 20, height: 100, data: genData(100), ref: listRef });
68-
listRef.current.scrollTo(903);
69-
jest.runAllTimers();
70+
act(() => {
71+
listRef.current.scrollTo(903);
72+
jest.runAllTimers();
73+
});
7074
expect(wrapper.find('ul').instance().scrollTop).toEqual(903);
7175

7276
wrapper.unmount();
@@ -79,40 +83,56 @@ describe('List.Scroll', () => {
7983

8084
describe('index scroll', () => {
8185
it('work', () => {
82-
listRef.current.scrollTo({ index: 30, align: 'top' });
83-
jest.runAllTimers();
86+
act(() => {
87+
listRef.current.scrollTo({ index: 30, align: 'top' });
88+
jest.runAllTimers();
89+
});
8490
expect(wrapper.find('ul').instance().scrollTop).toEqual(600);
8591
});
8692

8793
it('out of range should not crash', () => {
8894
expect(() => {
89-
listRef.current.scrollTo({ index: 99999999999, align: 'top' });
90-
jest.runAllTimers();
95+
act(() => {
96+
listRef.current.scrollTo({ index: 99999999999, align: 'top' });
97+
jest.runAllTimers();
98+
});
9199
}).not.toThrow();
92100
});
93101
});
94102

95103
it('scroll top should not out of range', () => {
96-
listRef.current.scrollTo({ index: 0, align: 'bottom' });
97-
jest.runAllTimers();
104+
act(() => {
105+
listRef.current.scrollTo({ index: 0, align: 'bottom' });
106+
jest.runAllTimers();
107+
});
98108
expect(wrapper.find('ul').instance().scrollTop).toEqual(0);
99109
});
100110

101111
it('key scroll', () => {
102-
listRef.current.scrollTo({ key: '30', align: 'bottom' });
103-
jest.runAllTimers();
112+
act(() => {
113+
listRef.current.scrollTo({ key: '30', align: 'bottom' });
114+
jest.runAllTimers();
115+
});
104116
expect(wrapper.find('ul').instance().scrollTop).toEqual(520);
105117
});
106118

107119
it('smart', () => {
108-
listRef.current.scrollTo(0);
109-
listRef.current.scrollTo({ index: 30 });
110-
jest.runAllTimers();
120+
act(() => {
121+
listRef.current.scrollTo(0);
122+
});
123+
act(() => {
124+
listRef.current.scrollTo({ index: 30 });
125+
jest.runAllTimers();
126+
});
111127
expect(wrapper.find('ul').instance().scrollTop).toEqual(520);
112128

113-
listRef.current.scrollTo(800);
114-
listRef.current.scrollTo({ index: 30 });
115-
jest.runAllTimers();
129+
act(() => {
130+
listRef.current.scrollTo(800);
131+
});
132+
act(() => {
133+
listRef.current.scrollTo({ index: 30 });
134+
jest.runAllTimers();
135+
});
116136
expect(wrapper.find('ul').instance().scrollTop).toEqual(600);
117137
});
118138
});

‎tests/touch.test.js‎

Lines changed: 47 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import React from 'react';
2+
import { act } from 'react-dom/test-utils';
23
import { mount } from 'enzyme';
34
import { spyElementPrototypes } from './utils/domHook';
45
import List from '../src';
@@ -55,22 +56,24 @@ describe('List.Touch', () => {
5556
return wrapper.find('.rc-virtual-list-holder').instance();
5657
}
5758

58-
// start
59-
const touchEvent = new Event('touchstart');
60-
touchEvent.touches = [{ pageY: 100 }];
61-
getElement().dispatchEvent(touchEvent);
59+
act(() => {
60+
// start
61+
const touchEvent = new Event('touchstart');
62+
touchEvent.touches = [{ pageY: 100 }];
63+
getElement().dispatchEvent(touchEvent);
6264

63-
// move
64-
const moveEvent = new Event('touchmove');
65-
moveEvent.touches = [{ pageY: 90 }];
66-
getElement().dispatchEvent(moveEvent);
65+
// move
66+
const moveEvent = new Event('touchmove');
67+
moveEvent.touches = [{ pageY: 90 }];
68+
getElement().dispatchEvent(moveEvent);
6769

68-
// end
69-
const endEvent = new Event('touchend');
70-
getElement().dispatchEvent(endEvent);
70+
// end
71+
const endEvent = new Event('touchend');
72+
getElement().dispatchEvent(endEvent);
7173

72-
// smooth
73-
jest.runAllTimers();
74+
// smooth
75+
jest.runAllTimers();
76+
});
7477
expect(wrapper.find('ul').instance().scrollTop > 10).toBeTruthy();
7578

7679
wrapper.unmount();
@@ -83,35 +86,39 @@ describe('List.Touch', () => {
8386
return wrapper.find('.rc-virtual-list-holder').instance();
8487
}
8588

86-
// start
87-
const touchEvent = new Event('touchstart');
88-
touchEvent.touches = [{ pageY: 500 }];
89-
getElement().dispatchEvent(touchEvent);
89+
act(() => {
90+
// start
91+
const touchEvent = new Event('touchstart');
92+
touchEvent.touches = [{ pageY: 500 }];
93+
getElement().dispatchEvent(touchEvent);
9094

91-
// move
92-
const preventDefault = jest.fn();
93-
const moveEvent = new Event('touchmove');
94-
moveEvent.touches = [{ pageY: 0 }];
95-
moveEvent.preventDefault = preventDefault;
96-
getElement().dispatchEvent(moveEvent);
95+
// move
96+
const preventDefault = jest.fn();
97+
const moveEvent = new Event('touchmove');
98+
moveEvent.touches = [{ pageY: 0 }];
99+
moveEvent.preventDefault = preventDefault;
100+
getElement().dispatchEvent(moveEvent);
101+
});
97102

98103
// Call preventDefault
99104
expect(preventDefault).toHaveBeenCalled();
100105

101-
// ======= Not call since scroll to the bottom =======
102-
jest.runAllTimers();
103-
preventDefault.mockReset();
106+
act(() => {
107+
// ======= Not call since scroll to the bottom =======
108+
jest.runAllTimers();
109+
preventDefault.mockReset();
104110

105-
// start
106-
const touchEvent2 = new Event('touchstart');
107-
touchEvent2.touches = [{ pageY: 500 }];
108-
getElement().dispatchEvent(touchEvent2);
111+
// start
112+
const touchEvent2 = new Event('touchstart');
113+
touchEvent2.touches = [{ pageY: 500 }];
114+
getElement().dispatchEvent(touchEvent2);
109115

110-
// move
111-
const moveEvent2 = new Event('touchmove');
112-
moveEvent2.touches = [{ pageY: 0 }];
113-
moveEvent2.preventDefault = preventDefault;
114-
getElement().dispatchEvent(moveEvent2);
116+
// move
117+
const moveEvent2 = new Event('touchmove');
118+
moveEvent2.touches = [{ pageY: 0 }];
119+
moveEvent2.preventDefault = preventDefault;
120+
getElement().dispatchEvent(moveEvent2);
121+
});
115122

116123
expect(preventDefault).not.toHaveBeenCalled();
117124
});
@@ -121,12 +128,11 @@ describe('List.Touch', () => {
121128
const preventDefault = jest.fn();
122129
const wrapper = genList({ itemHeight: 20, height: 100, data: genData(100) });
123130

124-
const touchEvent = new Event('touchstart');
125-
touchEvent.preventDefault = preventDefault;
126-
wrapper
127-
.find('.rc-virtual-list-scrollbar')
128-
.instance()
129-
.dispatchEvent(touchEvent);
131+
act(() => {
132+
const touchEvent = new Event('touchstart');
133+
touchEvent.preventDefault = preventDefault;
134+
wrapper.find('.rc-virtual-list-scrollbar').instance().dispatchEvent(touchEvent);
135+
});
130136

131137
expect(preventDefault).toHaveBeenCalled();
132138
});

0 commit comments

Comments
 (0)