add tests for checkAuth function

Author: Bernabe Gonzalez

.babelrc

@@ -6,8 +6,7 @@
   "env": {
     "development": {
       "presets": [
-        "latest",
-        "react-hmre"
+        "latest"
       ]
     },
     "production": {
@@ -17,10 +16,6 @@
             "modules": false
           }
         }],
-      ],
-      "plugins": [
-        "transform-react-constant-elements",
-        "transform-react-remove-prop-types"
       ]
     },
     "test": {

package.json

@@ -35,12 +35,9 @@
     "babel-eslint": "7.1.1",
     "babel-jest": "18.0.0",
     "babel-loader": "6.3.0",
-    "babel-plugin-transform-react-constant-elements": "6.23.0",
-    "babel-plugin-transform-react-remove-prop-types": "0.3.2",
     "babel-polyfill": "6.23.0",
     "babel-preset-latest": "6.22.0",
     "babel-preset-react": "6.23.0",
-    "babel-preset-react-hmre": "1.1.1",
     "babel-preset-stage-1": "6.22.0",
     "babel-register": "6.23.0",
     "coveralls": "2.11.16",

src/__tests__/index.spec.js

@@ -60,6 +60,100 @@ describe('API functions', () => {
     });
   });
 
+  describe('checkAuth', () => {
+    let nextState;
+    let replace;
+    let next;
+    beforeEach(() => {
+      nextState = { location: { pathname: 'test' } };
+      replace = jest.fn();
+      next = jest.fn();
+    });
+
+    describe('with logged user', () => {
+      beforeEach(() => {
+        __setUser(user);
+        __setSession(session);
+      });
+
+      test('does call next function', (done) => {
+        const next = jest.fn(() => {
+          expect(next).toHaveBeenCalled();
+          done();
+        });
+        sessionService.checkAuth(nextState, replace, next);
+      });
+
+      describe('with option refreshOnCheckAuth enable', () => {
+        test('change authenticated flag to true and save the user', (done) => {
+          sessionService.setOptions(store, true);
+          // wait for change the redux store
+          const unsubscribe = store.subscribe(() => {
+            const state = store.getState();
+            expect(state.authenticated).toEqual(true);
+            // wait to change the user
+            const { user } = state;
+            if (!(Object.keys(user).length === 0 && user.constructor === Object)) {
+              expect(state.user).toMatchObject(user);
+              unsubscribe();
+              done();
+            }
+          });
+
+          sessionService.checkAuth(nextState, replace, next);
+        });
+      });
+    });
+
+    describe('without logged user', () => {
+      beforeEach(() => {
+        __setUser(undefined);
+        __setSession(undefined);
+        sessionService.setOptions(store, false, 'redirectionPath');
+      });
+
+      test('does call replace function', (done) => {
+        const replace = jest.fn(() => {
+          expect(replace).toHaveBeenCalled();
+          done();
+        });
+        sessionService.checkAuth(nextState, replace, next);
+      });
+
+      test('does redirect to the redirectPath', (done) => {
+        const expectedArg = {
+          pathname: 'redirectionPath',
+          state: { nextPathname: nextState.location.pathname }
+        };
+        const replace = jest.fn(() => {
+          expect(replace).toHaveBeenCalledWith(expectedArg);
+          done();
+        });
+        sessionService.checkAuth(nextState, replace, next);
+      });
+
+      describe('with option refreshOnCheckAuth enable', () => {
+        test('change authenticated flag to false and the user to empty object', (done) => {
+          sessionService.setOptions(store, true);
+          // wait for change the redux store
+          const unsubscribe = store.subscribe(() => {
+            const state = store.getState();
+            expect(state.authenticated).toEqual(false);
+            // wait to empty the user
+            const { user } = state;
+            if ((Object.keys(user).length === 0 && user.constructor === Object)) {
+              expect(state.user).toEqual({});
+              unsubscribe();
+              done();
+            }
+          });
+
+          sessionService.checkAuth(nextState, replace, next);
+        });
+      });
+    });
+  });
+
   describe('saveSession', () => {
     describe('localforage returns success', () => {
       test('change authenticated flag to true value', (done) => {

src/index.js

@@ -22,13 +22,19 @@ export class sessionService {
     return instance;
   }
 
+  static setOptions(store, refreshOnCheckAuth = false, redirectPath = 'login') {
+    instance.store = store;
+    instance.refreshOnCheckAuth = refreshOnCheckAuth;
+    instance.redirectPath = redirectPath;
+  }
+
   static initSessionService(store, refreshOnCheckAuth = false, redirectPath = 'login') {
     instance = new sessionService(store, refreshOnCheckAuth, redirectPath);
     sessionService.refreshFromLocalStorage();
   }
 
   static refreshFromLocalStorage() {
-    sessionService.loadSession()
+    return sessionService.loadSession()
     .then(() => {
       instance.store.dispatch(getSessionSuccess());
       sessionService.loadUser().then((user) => {
@@ -52,6 +58,7 @@ export class sessionService {
     })
     .catch(() => {
       refreshOnCheckAuth && store.dispatch(getSessionError());
+      refreshOnCheckAuth && store.dispatch(getUserSessionError());
       replace({
         pathname: instance.redirectPath,
         state: { nextPathname: nextState.location.pathname }
@@ -84,7 +91,7 @@ export class sessionService {
     return removeItem(constant.USER_SESSION).then(() => {
       instance.store.dispatch(getSessionError());
       browserHistory.replace(instance.redirectPath);
-    });
+    }).catch(err => err);
   }
 
   static saveUser(user) {
@@ -110,7 +117,7 @@ export class sessionService {
   static deleteUser() {
     return removeItem(constant.USER_DATA).then(() => {
       instance.store.dispatch(getUserSessionError());
-    });
+    }).catch(err => err);
   }
 }