From fc8996444096c2a8d67a15cc5487923709ca71bb Mon Sep 17 00:00:00 2001
From: Gpx <polvara@gmail.com>
Date: Tue, 25 Sep 2018 09:17:19 +0200
Subject: [PATCH] =?UTF-8?q?fix:=20=F0=9F=90=9B=20click()=20handles=20check?=
 =?UTF-8?q?boxes=20correctly?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Issues: #14
---
 __tests__/click.js | 28 ++++++++++++++++++++++++++++
 package.json       |  7 ++++++-
 src/index.js       |  2 --
 3 files changed, 34 insertions(+), 3 deletions(-)

diff --git a/__tests__/click.js b/__tests__/click.js
index 95dcea7b..fb0c634c 100644
--- a/__tests__/click.js
+++ b/__tests__/click.js
@@ -193,4 +193,32 @@ describe("fireEvent.click", () => {
       expect(getByTestId("input")).toHaveFocus();
     }
   );
+
+  it('checks <input type="checkbox"> 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" type="checkbox" />
+      </React.Fragment>
+    );
+    expect(getByTestId("input")).toHaveProperty("checked", false);
+    userEvent.click(getByTestId("label"));
+    expect(getByTestId("input")).toHaveProperty("checked", true);
+  });
+
+  it('checks <input type="checkbox"> when clicking a <label> without htmlFor', () => {
+    const { getByTestId } = render(
+      <React.Fragment>
+        <label data-testid="label">
+          Label
+          <input id="input" data-testid="input" type="checkbox" />
+        </label>
+      </React.Fragment>
+    );
+    expect(getByTestId("input")).toHaveProperty("checked", false);
+    userEvent.click(getByTestId("label"));
+    expect(getByTestId("input")).toHaveProperty("checked", true);
+  });
 });
diff --git a/package.json b/package.json
index b30ae788..2840e598 100644
--- a/package.json
+++ b/package.json
@@ -2,7 +2,12 @@
   "name": "user-event",
   "version": "0.0.0-development",
   "description": "Simulate user events for react-testing-library",
-  "keywords": ["react-testing-library", "dom-testing-library", "react", "testing"],
+  "keywords": [
+    "react-testing-library",
+    "dom-testing-library",
+    "react",
+    "testing"
+  ],
   "main": "dist/index.js",
   "scripts": {
     "test": "jest",
diff --git a/src/index.js b/src/index.js
index f2134d5c..b85d34e1 100644
--- a/src/index.js
+++ b/src/index.js
@@ -11,7 +11,6 @@ function clickLabel(label) {
   fireEvent.mouseMove(label);
   fireEvent.mouseDown(label);
   fireEvent.mouseUp(label);
-  fireEvent.click(label);
 
   if (label.htmlFor) {
     const input = document.getElementById(label.htmlFor);
@@ -21,7 +20,6 @@ function clickLabel(label) {
     const input = label.querySelector("input,textarea");
     input.focus();
     label.focus();
-    fireEvent.click(input);
     fireEvent.click(label);
   }
 }