refactor: move onChange test to separate file

onChange test requires import from preact/compat which affects other tests in the same test file so I moved it to separate file

src/__tests__/events-compat.js

@@ -0,0 +1,27 @@
+import { h } from 'preact' // required by render
+import { fireEvent, render } from '..'
+import { forwardRef } from 'preact/compat' // required for this test to make sense
+
+test('calling `fireEvent` with `preact/compat` and onChange works too', () => {
+  const handler = jest.fn()
+
+  // forwardRef needs to be imported from preact/compat for this test to make sense.
+  // Preact behavior when using onChange is described here:
+  // https://preactjs.com/guide/v10/differences-to-react#use-oninput-instead-of-onchange
+  // We want to test if onChange event gets caught with fireEvent.change()
+  const {
+    container: { firstChild: input }
+  } = render(<input type="text" onChange={handler} />)
+
+  const targetProperties = { value: 'a' }
+  const otherProperties = { isComposing: true }
+  const init = {
+    target: targetProperties,
+    ...otherProperties
+  }
+
+  expect(fireEvent.change(input, init)).toBe(true)
+
+  expect(handler).toHaveBeenCalledTimes(1)
+  expect(handler).toHaveBeenCalledWith(expect.objectContaining(otherProperties))
+})

src/__tests__/events.js

@@ -1,6 +1,5 @@
 import { createRef, h } from 'preact'
 import { fireEvent, render } from '..'
-import { forwardRef } from 'preact/compat'
 
 const eventTypes = [
   {
@@ -181,30 +180,6 @@ test('onInput works', () => {
   expect(handler).toHaveBeenCalledWith(expect.objectContaining(otherProperties))
 })
 
-test.only('calling `fireEvent` with `preact/compat` and onChange works too', () => {
-  const handler = jest.fn()
-
-  // forwardRef needs to be imported from preact/compat for this test to make sense.
-  // Preact behavior when using onChange is described here:
-  // https://preactjs.com/guide/v10/differences-to-react#use-oninput-instead-of-onchange
-  // We want to test if onChange event gets caught with fireEvent.change()
-  const {
-    container: { firstChild: input }
-  } = render(<input type="text" onChange={handler} />)
-
-  const targetProperties = { value: 'a' }
-  const otherProperties = { isComposing: true }
-  const init = {
-    target: targetProperties,
-    ...otherProperties
-  }
-
-  expect(fireEvent.change(input, init)).toBe(true)
-
-  expect(handler).toHaveBeenCalledTimes(1)
-  expect(handler).toHaveBeenCalledWith(expect.objectContaining(otherProperties))
-})
-
 test('calling `fireEvent` directly works too', () => {
   const handler = jest.fn()
 

src/fire-event.js

@@ -16,6 +16,7 @@ Object.keys(domFireEvent).forEach((key) => {
     // making the event name out of sync.
     // The problematic code is in: preact/compat/src/render.js > handleDomVNode()
     const keyFiltered = key === 'change' ? 'input' : key
+
     return isInElem
       ? domFireEvent[keyFiltered](elem, init)
       : domFireEvent(