correct loading value and resulting tests

scm-ui/src/users/modules/users.js

@@ -219,8 +219,12 @@ export default function reducer(state: any = {}, action: any = {}) {
   switch (action.type) {
     case FETCH_USERS:
       return {
-        loading: true,
-        error: null
+        ...state,
+        users: {
+          error: null,
+          entries: null,
+          loading: true
+        }
       };
     case DELETE_USER:
       return {
@@ -246,10 +250,14 @@ export default function reducer(state: any = {}, action: any = {}) {
         usersByNames
       };
     case FETCH_USERS_FAILURE:
+      return {
+        ...state,
+        error: action.payload,
+        loading: false
+       };  
     case DELETE_USER_FAILURE:
         return {
           ...state,
-        login: false,
           error: action.payload,
           loading: false
         };

scm-ui/src/users/modules/users.test.js

@@ -157,8 +157,8 @@ describe("fetch tests", () => {
 describe("reducer tests", () => {
   test("users request", () => {
     var newState = reducer({}, { type: FETCH_USERS });
-    expect(newState.loading).toBeTruthy();
-    expect(newState.error).toBeNull();
+    expect(newState.users.loading).toBeTruthy();
+    expect(newState.users.error).toBeNull();
   });
 
   test("fetch users successful", () => {