Skip to content

Commit b755c3a

Browse files
author
Ana Margarida Silva
authored
fix #1824: trigger keyboard event regression (#1826)
* fix(dom-event): fix key override in trigger events fix #1824 * test(dom-event): test for trigger events with keys
1 parent 7413b37 commit b755c3a

File tree

2 files changed

+16
-1
lines changed

2 files changed

+16
-1
lines changed

packages/test-utils/src/create-dom-event.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ function getOptions(eventParams) {
6363
// Any derived options should go here
6464
keyCode,
6565
code: keyCode,
66-
key
66+
key: key || options.key
6767
}
6868
}
6969

test/specs/wrapper/trigger.spec.js

+15
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,21 @@ describeWithShallowAndMount('trigger', mountingMethod => {
6767
})
6868
})
6969

70+
describe('causes keydown handler to fire with the appropriate key when wrapper.trigger("keydown", { key: "k" }) is fired on a Component', async () => {
71+
const keydownHandler = jest.fn()
72+
const wrapper = mountingMethod(ComponentWithEvents, {
73+
propsData: { keydownHandler }
74+
})
75+
76+
await wrapper.find('.keydown').trigger('keydown', { key: 'k' })
77+
78+
const keyboardEvent = keydownHandler.mock.calls[0][0]
79+
80+
it('contains the key', () => {
81+
expect(keyboardEvent.key).toEqual('k')
82+
})
83+
})
84+
7085
it('causes keydown handler to fire when wrapper.trigger("keydown.enter") is fired on a Component', async () => {
7186
const keydownHandler = jest.fn()
7287
const wrapper = mountingMethod(ComponentWithEvents, {

0 commit comments

Comments
 (0)