fix: 🐛 don't fire events when clicking disabled checkbox

disabled checkboxes don't fire any event

BREAKING CHANGE: 🧨 clicking disabled checkbox no longer fires events

✅ Closes: testing-library#96

Author: Gpx

__tests__/click.js

@@ -1,5 +1,5 @@
 import React from "react";
-import { render, cleanup } from "react-testing-library";
+import { render, cleanup, fireEvent } from "react-testing-library";
 import "jest-dom/extend-expect";
 import userEvent from "../src";
 
@@ -67,6 +67,31 @@ describe("userEvent.click", () => {
     expect(getByTestId("element")).toHaveProperty("checked", true);
   });
 
+  it('should fire the correct events for <input type="checkbox" disabled>', () => {
+    const events = [];
+    const eventsHandler = jest.fn(evt => events.push(evt.type));
+    const { getByTestId } = render(
+      <input
+        data-testid="element"
+        type="checkbox"
+        onMouseOver={eventsHandler}
+        onMouseMove={eventsHandler}
+        onMouseDown={eventsHandler}
+        onFocus={eventsHandler}
+        onMouseUp={eventsHandler}
+        onClick={eventsHandler}
+        onChange={eventsHandler}
+        disabled
+      />
+    );
+
+    userEvent.click(getByTestId("element"));
+
+    expect(events).toEqual([]);
+
+    expect(getByTestId("element")).toHaveProperty("checked", false);
+  });
+
   it("should fire the correct events for <div>", () => {
     const events = [];
     const eventsHandler = jest.fn(evt => events.push(evt.type));

src/index.js

@@ -31,6 +31,8 @@ function clickLabel(label) {
 }
 
 function clickCheckbox(checkbox) {
+  if (checkbox.disabled) return;
+
   fireEvent.mouseOver(checkbox);
   fireEvent.mouseMove(checkbox);
   fireEvent.mouseDown(checkbox);