fix(setValue): trigger input event for checkbox, radio and select

freakzlike · freakzlike · commit 07d3b6339dbc · 2023-03-31T10:29:11.000+02:00
diff --git a/src/domWrapper.ts b/src/domWrapper.ts
@@ -93,6 +93,7 @@ export class DOMWrapper<NodeType extends Node> extends BaseWrapper<NodeType> {
     }
 
     element.checked = checked
+    this.trigger('input')
     return this.trigger('change')
   }
 
@@ -118,6 +119,8 @@ export class DOMWrapper<NodeType extends Node> extends BaseWrapper<NodeType> {
       } else {
         element.value = value
       }
+
+      this.trigger('input')
       return this.trigger('change')
     } else if (tagName === 'INPUT' || tagName === 'TEXTAREA') {
       element.value = value
@@ -146,7 +149,9 @@ export class DOMWrapper<NodeType extends Node> extends BaseWrapper<NodeType> {
       parentElement = parentElement.parentElement!
     }
 
-    return new DOMWrapper(parentElement).trigger('change')
+    const parentWrapper = new DOMWrapper(parentElement)
+    parentWrapper.trigger('input')
+    return parentWrapper.trigger('change')
   }
 }
 
diff --git a/tests/setValue.spec.ts b/tests/setValue.spec.ts
@@ -33,6 +33,11 @@ describe('setValue', () => {
   })
 
   describe('on select and option', () => {
+    const renderOptions = () => [
+      h('option', { value: 'A' }),
+      h('option', { value: 'B' })
+    ]
+
     it('sets element of select value', async () => {
       const wrapper = mount(ComponentWithInput)
       const select = wrapper.find<HTMLSelectElement>('select')
@@ -42,7 +47,7 @@ describe('setValue', () => {
       expect(wrapper.text()).toContain('selectB')
 
       expect(wrapper.emitted('change')).toHaveLength(1)
-      expect(wrapper.emitted('input')).toBeUndefined()
+      expect(wrapper.emitted('input')).toHaveLength(1)
     })
 
     it('as an option of a select as selected', async () => {
@@ -68,11 +73,7 @@ describe('setValue', () => {
 
       const Component = {
         setup() {
-          return () =>
-            h('select', { onChange: handle }, [
-              h('option', { value: 'A' }),
-              h('option', { value: 'B' })
-            ])
+          return () => h('select', { onChange: handle }, renderOptions())
         }
       }
 
@@ -98,7 +99,7 @@ describe('setValue', () => {
       expect(wrapper.vm.multiselectVal).toEqual(['selectA', 'selectC'])
 
       expect(wrapper.emitted('change')).toHaveLength(1)
-      expect(wrapper.emitted('input')).toBeUndefined()
+      expect(wrapper.emitted('input')).toHaveLength(1)
     })
 
     it('overrides elements of multiselect', async () => {
@@ -114,7 +115,37 @@ describe('setValue', () => {
       expect(wrapper.vm.multiselectVal).toEqual(['selectB'])
 
       expect(wrapper.emitted('change')).toHaveLength(2)
-      expect(wrapper.emitted('input')).toBeUndefined()
+      expect(wrapper.emitted('input')).toHaveLength(2)
+    })
+
+    it('does trigger input and change event on select', async () => {
+      const onInput = vi.fn()
+      const onChange = vi.fn()
+      const Comp = defineComponent({
+        setup() {
+          return () => h('select', { onInput, onChange }, renderOptions())
+        }
+      })
+
+      await mount(Comp).find('select').setValue('A')
+
+      expect(onInput).toHaveBeenCalledTimes(1)
+      expect(onChange).toHaveBeenCalledTimes(1)
+    })
+
+    it('does trigger input and change event on option select', async () => {
+      const onInput = vi.fn()
+      const onChange = vi.fn()
+      const Comp = defineComponent({
+        setup() {
+          return () => h('select', { onInput, onChange }, renderOptions())
+        }
+      })
+
+      await mount(Comp).findAll('option')[1].setValue()
+
+      expect(onInput).toHaveBeenCalledTimes(1)
+      expect(onChange).toHaveBeenCalledTimes(1)
     })
   })
 
@@ -202,6 +233,46 @@ describe('setValue', () => {
       const fn = radioFoo.setValue(false)
       await expect(fn).rejects.toThrowError(message)
     })
+
+    it('does trigger input and change event on checkbox', async () => {
+      const onInput = vi.fn()
+      const onChange = vi.fn()
+      const Comp = defineComponent({
+        setup() {
+          return () =>
+            h('input', {
+              onInput,
+              onChange,
+              type: 'checkbox'
+            })
+        }
+      })
+
+      await mount(Comp).find('input').setValue()
+
+      expect(onInput).toHaveBeenCalledTimes(1)
+      expect(onChange).toHaveBeenCalledTimes(1)
+    })
+
+    it('does trigger input and change event on radio', async () => {
+      const onInput = vi.fn()
+      const onChange = vi.fn()
+      const Comp = defineComponent({
+        setup() {
+          return () =>
+            h('input', {
+              onInput,
+              onChange,
+              type: 'radio'
+            })
+        }
+      })
+
+      await mount(Comp).find('input').setValue()
+
+      expect(onInput).toHaveBeenCalledTimes(1)
+      expect(onChange).toHaveBeenCalledTimes(1)
+    })
   })
 
   it('throws error if element is not valid', () => {

Original file line number	Diff line number	Diff line change
`@@ -93,6 +93,7 @@ export class DOMWrapper<NodeType extends Node> extends BaseWrapper<NodeType> {`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`element.checked = checked`
	`96`	`+ this.trigger('input')`
`96`	`97`	`return this.trigger('change')`
`97`	`98`	`}`
`98`	`99`
`@@ -118,6 +119,8 @@ export class DOMWrapper<NodeType extends Node> extends BaseWrapper<NodeType> {`
`118`	`119`	`} else {`
`119`	`120`	`element.value = value`
`120`	`121`	`}`
	`122`	`+`
	`123`	`+ this.trigger('input')`
`121`	`124`	`return this.trigger('change')`
`122`	`125`	`} else if (tagName === 'INPUT' \|\| tagName === 'TEXTAREA') {`
`123`	`126`	`element.value = value`
`@@ -146,7 +149,9 @@ export class DOMWrapper<NodeType extends Node> extends BaseWrapper<NodeType> {`
`146`	`149`	`parentElement = parentElement.parentElement!`
`147`	`150`	`}`
`148`	`151`
`149`		`- return new DOMWrapper(parentElement).trigger('change')`
	`152`	`+ const parentWrapper = new DOMWrapper(parentElement)`
	`153`	`+ parentWrapper.trigger('input')`
	`154`	`+ return parentWrapper.trigger('change')`
`150`	`155`	`}`
`151`	`156`	`}`
`152`	`157`