From 31a45ddaaa46d2f16b05a27b949dfd4ae09fcaa4 Mon Sep 17 00:00:00 2001
From: Matthew Shao <me@matshao.com>
Date: Sun, 26 Mar 2017 09:34:45 +0800
Subject: [web] Reach 100% coverage for ducks/settings.js

---
 web/src/js/__tests__/ducks/settingsSpec.js | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)
 create mode 100644 web/src/js/__tests__/ducks/settingsSpec.js

(limited to 'web')

diff --git a/web/src/js/__tests__/ducks/settingsSpec.js b/web/src/js/__tests__/ducks/settingsSpec.js
new file mode 100644
index 00000000..325cc8e2
--- /dev/null
+++ b/web/src/js/__tests__/ducks/settingsSpec.js
@@ -0,0 +1,23 @@
+jest.unmock('../../ducks/settings')
+jest.mock('../../utils')
+
+import reduceSettings from '../../ducks/settings'
+import * as SettingsActions from '../../ducks/settings'
+
+describe('setting reducer', () => {
+    it('should return initial state', () => {
+        expect(reduceSettings(undefined, {})).toEqual({})
+    })
+
+    it('should handle receive action', () => {
+        let action = { type: SettingsActions.RECEIVE, data: 'foo' }
+        expect(reduceSettings(undefined, action)).toEqual('foo')
+    })
+
+    it('should handle update action', () => {
+        let action = {type: SettingsActions.UPDATE, data: {id: 1} }
+        expect(reduceSettings(undefined, action)).toEqual({id: 1})
+
+        expect(reduceSettings(undefined, SettingsActions.update())).toEqual({})
+    })
+})
-- 
cgit v1.2.3


From de9ecb7c300219282252e2b61dd687a12af5d8f5 Mon Sep 17 00:00:00 2001
From: Matthew Shao <me@matshao.com>
Date: Sun, 26 Mar 2017 09:36:18 +0800
Subject: [web] Add JS coverage to codecov.

---
 web/package.json | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'web')

diff --git a/web/package.json b/web/package.json
index f0b3ae6a..8e4140bf 100644
--- a/web/package.json
+++ b/web/package.json
@@ -13,7 +13,9 @@
     ],
     "unmockedModulePathPatterns": [
       "react"
-    ]
+    ],
+      "coverageDirectory":"./coverage",
+      "collectCoverage": true
   },
   "dependencies": {
     "bootstrap": "^3.3.7",
-- 
cgit v1.2.3


From 17ac1ff6cb854e145d52f0509f218bbb14fcb2c3 Mon Sep 17 00:00:00 2001
From: Matthew Shao <me@matshao.com>
Date: Sun, 26 Mar 2017 21:34:47 +0800
Subject: [web] Exclude src/js/filt/filt.js from coverage report.

---
 web/package.json | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'web')

diff --git a/web/package.json b/web/package.json
index 8e4140bf..a1bd6561 100644
--- a/web/package.json
+++ b/web/package.json
@@ -15,7 +15,10 @@
       "react"
     ],
       "coverageDirectory":"./coverage",
-      "collectCoverage": true
+      "collectCoverage": true,
+      "coveragePathIgnorePatterns": [
+          "<rootDir>/src/js/filt/filt.js"
+      ]
   },
   "dependencies": {
     "bootstrap": "^3.3.7",
-- 
cgit v1.2.3


From 9a604b5cfe8bc8af2d3a087372131ab50788e802 Mon Sep 17 00:00:00 2001
From: Matthew Shao <me@matshao.com>
Date: Sun, 26 Mar 2017 21:58:47 +0800
Subject: [web] Minor fixes for the test.

---
 web/package.json                           | 10 +++++-----
 web/src/js/__tests__/ducks/settingsSpec.js |  7 +++++--
 2 files changed, 10 insertions(+), 7 deletions(-)

(limited to 'web')

diff --git a/web/package.json b/web/package.json
index a1bd6561..90ec031b 100644
--- a/web/package.json
+++ b/web/package.json
@@ -14,11 +14,11 @@
     "unmockedModulePathPatterns": [
       "react"
     ],
-      "coverageDirectory":"./coverage",
-      "collectCoverage": true,
-      "coveragePathIgnorePatterns": [
-          "<rootDir>/src/js/filt/filt.js"
-      ]
+    "coverageDirectory":"./coverage",
+    "collectCoverage": true,
+    "coveragePathIgnorePatterns": [
+        "<rootDir>/src/js/filt/filt.js"
+    ]
   },
   "dependencies": {
     "bootstrap": "^3.3.7",
diff --git a/web/src/js/__tests__/ducks/settingsSpec.js b/web/src/js/__tests__/ducks/settingsSpec.js
index 325cc8e2..cb9df918 100644
--- a/web/src/js/__tests__/ducks/settingsSpec.js
+++ b/web/src/js/__tests__/ducks/settingsSpec.js
@@ -1,8 +1,7 @@
 jest.unmock('../../ducks/settings')
 jest.mock('../../utils')
 
-import reduceSettings from '../../ducks/settings'
-import * as SettingsActions from '../../ducks/settings'
+import reduceSettings, * as SettingsActions from '../../ducks/settings'
 
 describe('setting reducer', () => {
     it('should return initial state', () => {
@@ -17,7 +16,11 @@ describe('setting reducer', () => {
     it('should handle update action', () => {
         let action = {type: SettingsActions.UPDATE, data: {id: 1} }
         expect(reduceSettings(undefined, action)).toEqual({id: 1})
+    })
+})
 
+describe('setting actions', () => {
+    it('should be possible to update setting', () => {
         expect(reduceSettings(undefined, SettingsActions.update())).toEqual({})
     })
 })
-- 
cgit v1.2.3