Skip to content

Commit ad7cc9c

Browse files
committed
add clarifications; unify test names
1 parent 6bbcc5e commit ad7cc9c

File tree

3 files changed

+11
-10
lines changed

3 files changed

+11
-10
lines changed

src/__tests__/auto-cleanup-skip.js renamed to src/__tests__/auto-cleanup-skip.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { View } from 'react-native';
44
let render;
55
beforeAll(() => {
66
process.env.RNTL_SKIP_AUTO_CLEANUP = 'true';
7-
const rtl = require('../');
7+
const rtl = require('..');
88
render = rtl.render;
99
});
1010

src/__tests__/auto-cleanup.js renamed to src/__tests__/auto-cleanup.test.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,10 @@ class Test extends React.Component<*> {
2020
}
2121
}
2222

23+
afterEach(() => {
24+
jest.useRealTimers();
25+
});
26+
2327
// This just verifies that by importing RNTL in an environment which supports afterEach (like jest)
2428
// we'll get automatic cleanup between tests.
2529
test('component is mounted, but not umounted before test ends', () => {

src/flushMicroTasks.js

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,22 @@
11
// @flow
2-
32
import { printDeprecationWarning } from './helpers/errors';
43

5-
type Thenable = { then: (() => mixed) => mixed };
4+
type Thenable<T> = { then: (() => T) => mixed };
65

76
/**
87
* Wait for microtasks queue to flush
98
*/
10-
export default function flushMicrotasksQueue(): Thenable {
9+
export default function flushMicrotasksQueue<T>(): Thenable<T> {
1110
printDeprecationWarning('flushMicrotasksQueue');
1211
return flushMicroTasks();
1312
}
1413

15-
// let's try to get node's version of setImmediate, bypassing fake timers if
16-
// any. $FlowFixMe - timers is internal Node module
17-
const enqueueTask = require('timers').setImmediate;
18-
19-
export function flushMicroTasks(): Thenable {
14+
export function flushMicroTasks<T>(): Thenable<T> {
2015
return {
16+
// using "thenable" instead of a Promise, because otherwise it breaks when
17+
// using "modern" fake timers
2118
then(resolve) {
22-
enqueueTask(resolve);
19+
setImmediate(resolve);
2320
},
2421
};
2522
}

0 commit comments

Comments
 (0)