Skip to content

Commit 0f19120

Browse files
authored
Align dom-testing-library wrappers with react (#31)
* test: add test * fix: align dom-testing-library wrappers with react Co-authored-by: Niklas Mischkulnig <[email protected]>
1 parent 98e13eb commit 0f19120

File tree

2 files changed

+15
-7
lines changed

2 files changed

+15
-7
lines changed

src/__tests__/events.js

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ eventTypes.forEach(({
146146
})
147147
)
148148

149-
fireEvent[eventName](ref.current, init)
149+
expect(fireEvent[eventName](ref.current, init)).toBe(true)
150150

151151
expect(spy).toHaveBeenCalledTimes(1)
152152
})
@@ -161,24 +161,32 @@ test('onInput works', () => {
161161
(<input type="text" onInput={handler} />)
162162
)
163163

164-
fireEvent.input(input, { target: { value: 'a' } })
164+
expect(fireEvent.input(input, { target: { value: 'a' } })).toBe(true)
165165

166166
expect(handler).toHaveBeenCalledTimes(1)
167167
})
168168

169169
test('calling `fireEvent` directly works too', () => {
170-
const handleEvent = jest.fn()
170+
const handler = jest.fn()
171171

172172
const { container: { firstChild: button } } = render(
173-
(<button onClick={handleEvent} />)
173+
(<button onClick={handler} />)
174174
)
175175

176-
fireEvent(
176+
expect(fireEvent(
177177
button,
178178
new Event('MouseEvent', {
179179
bubbles: true,
180180
cancelable: true,
181181
button: 0
182182
})
183+
)).toBe(true)
184+
})
185+
186+
test('`fireEvent` returns false when prevented', () => {
187+
const { container: { firstChild: button } } = render(
188+
(<button onClick={(e) => { e.preventDefault() }} />)
183189
)
190+
191+
expect(fireEvent.click(button)).toBe(false)
184192
})

src/pure.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,9 @@ configureDTL({
1010
})
1111
return result
1212
},
13-
eventWrapper: async cb => {
13+
eventWrapper: cb => {
1414
let result
15-
await act(() => {
15+
act(() => {
1616
result = cb()
1717
})
1818
return result

0 commit comments

Comments
 (0)