From cc9ee67dc22139b022497bba35b78745cb1f5d48 Mon Sep 17 00:00:00 2001
From: Alex Andreev <alex.andreev.email@gmail.com>
Date: Wed, 11 May 2022 17:33:01 +0300
Subject: [PATCH] Fix <Switch/> checked callback attribute value (#5360)

---
 .../switch/__tests__/switch.test.tsx          | 20 +++++++++++++++++++
 src/renderer/components/switch/switch.tsx     |  8 +++++++-
 2 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/src/renderer/components/switch/__tests__/switch.test.tsx b/src/renderer/components/switch/__tests__/switch.test.tsx
index 20b262ab01..0781ba24b3 100644
--- a/src/renderer/components/switch/__tests__/switch.test.tsx
+++ b/src/renderer/components/switch/__tests__/switch.test.tsx
@@ -48,4 +48,24 @@ describe("<Switch/>", () => {
 
     expect(onClick).not.toHaveBeenCalled();
   });
+
+  it("returns true checked attribute in a onChange callback", () => {
+    const onClick = jest.fn();
+    const { getByTestId } = render(<Switch onChange={onClick} checked={true}/>);
+    const switcher = getByTestId("switch");
+
+    fireEvent.click(switcher);
+
+    expect(onClick).toHaveBeenCalledWith(false, expect.any(Object));
+  });
+
+  it("returns false checked attribute in a onChange callback", () => {
+    const onClick = jest.fn();
+    const { getByTestId } = render(<Switch onChange={onClick}/>);
+    const switcher = getByTestId("switch");
+
+    fireEvent.click(switcher);
+
+    expect(onClick).toHaveBeenCalledWith(true, expect.any(Object));
+  });
 });
diff --git a/src/renderer/components/switch/switch.tsx b/src/renderer/components/switch/switch.tsx
index 5052941e84..2f91faf1a9 100644
--- a/src/renderer/components/switch/switch.tsx
+++ b/src/renderer/components/switch/switch.tsx
@@ -17,7 +17,13 @@ export function Switch({ children, disabled, onChange, ...props }: SwitchProps)
   return (
     <label className={cssNames(styles.Switch, { [styles.disabled]: disabled })} data-testid="switch">
       {children}
-      <input type="checkbox" role="switch" disabled={disabled} onChange={(event) => onChange?.(props.checked, event)} {...props}/>
+      <input
+        type="checkbox"
+        role="switch"
+        disabled={disabled}
+        onChange={(event) => onChange?.(event.target.checked, event)}
+        {...props}
+      />
     </label>
   );
 }