Merge pull request #2 from Gpx/labels

Labels

.gitignore

@@ -1 +1,2 @@
 node_modules
+coverage/

__tests__/events.js

@@ -1,27 +1,129 @@
 import React from 'react'
-import { render } from 'react-testing-library'
+import { render, cleanup } from 'react-testing-library'
 import 'jest-dom/extend-expect'
 import userEvent from '../src'
 
-test('fireEvent.click simulates a user click', () => {
-  const { getByTestId } = render(
-    <React.Fragment>
-      <input data-testid="A" />
-      <input data-testid="B" />
-    </React.Fragment>
-  )
+afterEach(cleanup)
 
-  const a = getByTestId('A')
-  const b = getByTestId('B')
+describe('fireEvent.click', () => {
+  it('should fire the correct events for <input>', () => {
+    const onMouseOver = jest.fn()
+    const onMouseMove = jest.fn()
+    const onMouseDown = jest.fn()
+    const onFocus = jest.fn()
+    const onMouseUp = jest.fn()
+    const onClick = jest.fn()
+    const { getByTestId } = render(
+      <input
+        data-testid="input"
+        onMouseOver={onMouseOver}
+        onMouseMove={onMouseMove}
+        onMouseDown={onMouseDown}
+        onFocus={onFocus}
+        onMouseUp={onMouseUp}
+        onClick={onClick}
+      />
+    )
 
-  expect(a).not.toHaveFocus()
-  expect(b).not.toHaveFocus()
+    expect(onMouseOver).not.toHaveBeenCalled()
+    expect(onMouseMove).not.toHaveBeenCalled()
+    expect(onMouseDown).not.toHaveBeenCalled()
+    expect(onFocus).not.toHaveBeenCalled()
+    expect(onMouseUp).not.toHaveBeenCalled()
+    expect(onClick).not.toHaveBeenCalled()
 
-  userEvent.click(a)
-  expect(a).toHaveFocus()
-  expect(b).not.toHaveFocus()
+    userEvent.click(getByTestId('input'))
 
-  userEvent.click(b)
-  expect(a).not.toHaveFocus()
-  expect(a).not.toHaveFocus()
+    expect(onMouseOver).toHaveBeenCalledTimes(1)
+    expect(onMouseMove).toHaveBeenCalledTimes(1)
+    expect(onMouseDown).toHaveBeenCalledTimes(1)
+    expect(onFocus).toHaveBeenCalledTimes(1)
+    expect(onMouseUp).toHaveBeenCalledTimes(1)
+    expect(onClick).toHaveBeenCalledTimes(1)
+  })
+
+  it('should fire the correct events for <div>', () => {
+    const onMouseOver = jest.fn()
+    const onMouseMove = jest.fn()
+    const onMouseDown = jest.fn()
+    const onFocus = jest.fn()
+    const onMouseUp = jest.fn()
+    const onClick = jest.fn()
+    const { getByTestId } = render(
+      <div
+        data-testid="div"
+        onMouseOver={onMouseOver}
+        onMouseMove={onMouseMove}
+        onMouseDown={onMouseDown}
+        onFocus={onFocus}
+        onMouseUp={onMouseUp}
+        onClick={onClick}
+      />
+    )
+
+    expect(onMouseOver).not.toHaveBeenCalled()
+    expect(onMouseMove).not.toHaveBeenCalled()
+    expect(onMouseDown).not.toHaveBeenCalled()
+    expect(onFocus).not.toHaveBeenCalled()
+    expect(onMouseUp).not.toHaveBeenCalled()
+    expect(onClick).not.toHaveBeenCalled()
+
+    userEvent.click(getByTestId('div'))
+
+    expect(onMouseOver).toHaveBeenCalledTimes(1)
+    expect(onMouseMove).toHaveBeenCalledTimes(1)
+    expect(onMouseDown).toHaveBeenCalledTimes(1)
+    expect(onFocus).not.toHaveBeenCalled()
+    expect(onMouseUp).toHaveBeenCalledTimes(1)
+    expect(onClick).toHaveBeenCalledTimes(1)
+  })
+
+  it('toggles the focus', () => {
+    const { getByTestId } = render(
+      <React.Fragment>
+        <input data-testid="A" />
+        <input data-testid="B" />
+      </React.Fragment>
+    )
+
+    const a = getByTestId('A')
+    const b = getByTestId('B')
+
+    expect(a).not.toHaveFocus()
+    expect(b).not.toHaveFocus()
+
+    userEvent.click(a)
+    expect(a).toHaveFocus()
+    expect(b).not.toHaveFocus()
+
+    userEvent.click(b)
+    expect(a).not.toHaveFocus()
+    expect(a).not.toHaveFocus()
+  })
+
+  it('gives focus when clicking a <label> with htmlFor', () => {
+    const { getByTestId } = render(
+      <React.Fragment>
+        <label htmlFor="input" data-testid="label">
+          Label
+        </label>
+        <input id="input" data-testid="input" />
+      </React.Fragment>
+    )
+    userEvent.click(getByTestId('label'))
+    expect(getByTestId('input')).toHaveFocus()
+  })
+
+  it('gives focus when clicking a <label> without htmlFor', () => {
+    const { getByTestId } = render(
+      <React.Fragment>
+        <label data-testid="label">
+          <span>Label</span>
+          <input data-testid="input" />
+        </label>
+      </React.Fragment>
+    )
+    userEvent.click(getByTestId('label'))
+    expect(getByTestId('input')).toHaveFocus()
+  })
 })

src/index.js

@@ -10,12 +10,32 @@ const userEvent = {
       fireEvent.mouseLeave(focusedElement)
     }
 
-    fireEvent.mouseOver(element)
-    fireEvent.mouseMove(element)
-    fireEvent.mouseDown(element)
-    element.focus()
-    fireEvent.mouseUp(element)
-    fireEvent.click(element)
+    if (element.tagName === 'LABEL') {
+      fireEvent.mouseOver(element)
+      fireEvent.mouseMove(element)
+      fireEvent.mouseDown(element)
+      fireEvent.mouseUp(element)
+      fireEvent.click(element)
+
+      if (element.htmlFor) {
+        const input = document.getElementById(element.htmlFor)
+        input.focus()
+        fireEvent.click(element)
+      } else {
+        const input = element.querySelector('input')
+        input.focus()
+        element.focus()
+        fireEvent.click(input)
+        fireEvent.click(element)
+      }
+    } else {
+      fireEvent.mouseOver(element)
+      fireEvent.mouseMove(element)
+      fireEvent.mouseDown(element)
+      element.focus()
+      fireEvent.mouseUp(element)
+      fireEvent.click(element)
+    }
 
     wasAnotherElementFocused && focusedElement.blur()
   },