From 896d1737d412f43bbeeaeeea65f8d6d81369f1e0 Mon Sep 17 00:00:00 2001
From: Matthias Kretschmann <m@kretschmann.io>
Date: Sun, 26 May 2019 23:52:27 +0200
Subject: [PATCH] fix ThemeSwitch test

---
 src/components/molecules/ThemeSwitch.jsx | 10 +++++-----
 src/store/AppProvider.jsx                |  2 +-
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/components/molecules/ThemeSwitch.jsx b/src/components/molecules/ThemeSwitch.jsx
index 8437f5e..9670e07 100644
--- a/src/components/molecules/ThemeSwitch.jsx
+++ b/src/components/molecules/ThemeSwitch.jsx
@@ -13,19 +13,19 @@ const Animation = posed.aside(fadeIn)
 
 const ThemeToggle = ({ dark }) => (
   <span id="toggle" className={styles.checkboxContainer} aria-live="assertive">
-    <Day className={dark ? null : 'active'} />
+    <Day className={!dark ? null : 'active'} />
     <span className={styles.checkboxFake} />
     <Night className={dark ? 'active' : null} />
   </span>
 )
 
 ThemeToggle.propTypes = {
-  dark: PropTypes.bool
+  dark: PropTypes.bool.isRequired
 }
 
 const ThemeToggleInput = ({ dark, toggleDark }) => (
   <input
-    onChange={toggleDark()}
+    onChange={() => toggleDark()}
     type="checkbox"
     name="toggle"
     value="toggle"
@@ -35,8 +35,8 @@ const ThemeToggleInput = ({ dark, toggleDark }) => (
 )
 
 ThemeToggleInput.propTypes = {
-  dark: PropTypes.bool,
-  toggleDark: PropTypes.func
+  dark: PropTypes.bool.isRequired,
+  toggleDark: PropTypes.func.isRequired
 }
 
 export default class ThemeSwitch extends PureComponent {
diff --git a/src/store/AppProvider.jsx b/src/store/AppProvider.jsx
index dd659d0..c6f2dab 100644
--- a/src/store/AppProvider.jsx
+++ b/src/store/AppProvider.jsx
@@ -7,7 +7,7 @@ import { getCountry } from '../utils/getCountry'
 export default class AppProvider extends PureComponent {
   state = {
     dark: false,
-    toggleDark: () => this.toggleDark,
+    toggleDark: () => this.toggleDark(),
     geolocation: null
   }