Improve tests, coverage

AgileVentures · Feb 4, 2019 · 6fc0629 · 6fc0629
1 parent 083df8b
commit 6fc0629
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 22 deletions.
diff --git a/src/reducers/projectsReducer.js b/src/reducers/projectsReducer.js
@@ -1,4 +1,4 @@
-import { GET_PROJECTS, FETCH_PROJECTS_FAILURE } from '../types'
+import { GET_PROJECTS } from '../types'
 import initialState from './initialState'
 
 const projectsReducer = (state = initialState.projects, action) => {

diff --git a/src/tests/actions/getProjectsAction.test.js b/src/tests/actions/getProjectsAction.test.js
@@ -25,7 +25,7 @@ describe('fetchProjects action', () => {
       '/api/v1/projects',
       {
         status: 200,
-        response: { projects: projectsResponse, languages: { 'Autograder': [] }, followers: [], documents: [] }
+        response: { projects: projectsResponse, languages: { 'Autograder': ['Ruby'] }, followers: [], documents: [] }
       }
     )
 

diff --git a/src/tests/store/loggedInUser.test.js b/src/tests/store/loggedInUser.test.js
@@ -7,11 +7,6 @@ describe('Store', () => {
       payload: { id: 2, email: 'someEmail@example.com' }
     })
 
-    expect(store.getState()).toEqual({
-      loggedInUser: { id: 2, email: 'someEmail@example.com' },
-      signedUpUser: {},
-      users: [],
-      projects: []
-    })
+    expect(store.getState().loggedInUser).toEqual({ id: 2, email: 'someEmail@example.com' })
   })
 })
diff --git a/src/tests/store/signedUpUser.test.js b/src/tests/store/signedUpUser.test.js
@@ -7,11 +7,6 @@ describe('Store', () => {
       payload: { id: 2, email: 'someEmail@example.com' }
     })
 
-    expect(store.getState()).toEqual({
-      signedUpUser: { id: 2, email: 'someEmail@example.com' },
-      loggedInUser: {},
-      users: [],
-      projects: []
-    })
+    expect(store.getState().signedUpUser).toEqual({ id: 2, email: 'someEmail@example.com' })
   })
 })
diff --git a/src/tests/store/store.test.js b/src/tests/store/store.test.js
@@ -9,9 +9,9 @@ describe('Store', () => {
 
     expect(store.getState()).toEqual({
       users: ['Run the tests'],
-      projects: [],
       loggedInUser: {},
-      signedUpUser: {}
+      signedUpUser: {},
+      projects: []
     })
   })
 })
diff --git a/src/tests/store/users.test.js b/src/tests/store/users.test.js
@@ -7,11 +7,6 @@ describe('Store', () => {
       payload: ['Run the tests']
     })
 
-    expect(store.getState()).toEqual({
-      users: ['Run the tests'],
-      loggedInUser: {},
-      signedUpUser: {},
-      projects: []
-    })
+    expect(store.getState().users).toEqual(['Run the tests'])
   })
 })