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
vuejs · Apr 21, 2021 · b755c3a · b755c3a
1 parent 7413b37
commit b755c3a
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/packages/test-utils/src/create-dom-event.js b/packages/test-utils/src/create-dom-event.js
@@ -63,7 +63,7 @@ function getOptions(eventParams) {
     // Any derived options should go here
     keyCode,
     code: keyCode,
-    key
+    key: key || options.key
   }
 }
 

diff --git a/test/specs/wrapper/trigger.spec.js b/test/specs/wrapper/trigger.spec.js
@@ -67,6 +67,21 @@ describeWithShallowAndMount('trigger', mountingMethod => {
     })
   })
 
+  describe('causes keydown handler to fire with the appropriate key when wrapper.trigger("keydown", { key: "k" }) is fired on a Component', async () => {
+    const keydownHandler = jest.fn()
+    const wrapper = mountingMethod(ComponentWithEvents, {
+      propsData: { keydownHandler }
+    })
+
+    await wrapper.find('.keydown').trigger('keydown', { key: 'k' })
+
+    const keyboardEvent = keydownHandler.mock.calls[0][0]
+
+    it('contains the key', () => {
+      expect(keyboardEvent.key).toEqual('k')
+    })
+  })
+
   it('causes keydown handler to fire when wrapper.trigger("keydown.enter") is fired on a Component', async () => {
     const keydownHandler = jest.fn()
     const wrapper = mountingMethod(ComponentWithEvents, {