From 59ab8f7de70470f4bb7ca4a8cffe42bff83c90ea Mon Sep 17 00:00:00 2001
From: Maxime Millette-Coulombe <mmcoulombe@inocybe.com>
Date: Wed, 3 Jun 2015 16:39:39 -0400
Subject: [PATCH] Increase test coverage: Add unit tests to the login module
 part 2

Change-Id: I02ffbec59100992d709e0de6eb666813fef29f04
Signed-off-by: Maxime Millette-Coulombe <mmcoulombe@inocybe.com>
---
 .../src/main/resources/login/login.spec.js    | 43 +++++++++++++++++--
 1 file changed, 40 insertions(+), 3 deletions(-)

diff --git a/modules/common-login-resources/src/main/resources/login/login.spec.js b/modules/common-login-resources/src/main/resources/login/login.spec.js
index 23ab183a..f16c9907 100644
--- a/modules/common-login-resources/src/main/resources/login/login.spec.js
+++ b/modules/common-login-resources/src/main/resources/login/login.spec.js
@@ -1,17 +1,23 @@
 define(['common/login/login.controller'], function() {
   describe('Login Module', function() {
-    var scope, state, controller, AuthMock;
+    var scope, state, controller, location, _AuthMock;
+    var url = '/test';
 
     beforeEach(module('ui.router'));
+    beforeEach(module('app.common.layout'));
     beforeEach(module('app.common.login', function($provide) {
-      AuthMock = jasmine.createSpyObj('AuthMock', ['isAuthed']);
+      AuthMock = {
+        isAuthed: function() {},
+        login: function() {}
+      };
       $provide.value('Auth', AuthMock);
     }));
 
-    beforeEach(inject( function($rootScope, $controller, $state) {
+    beforeEach(inject( function($rootScope, $controller, $state, $location) {
       scope = $rootScope.$new();
       controller = $controller;
       state = $state;
+      location = $location;
     }));
 
     it('Should load the login state', function() {
@@ -20,5 +26,36 @@ define(['common/login/login.controller'], function() {
       controller('LoginCtrl', {$scope: scope, $state: state});
       expect(state.href(stateName, {})).toBe('#/login');
     });
+
+    it('Should redirect any url to login if not logged', function() {
+      var stateName = 'login';
+      spyOn(AuthMock,'isAuthed').andReturn(false);
+      location.url(url);
+      controller('LoginCtrl', {$scope: scope, $state: state});
+      state.go('main');
+
+      expect(AuthMock.isAuthed).toHaveBeenCalled();
+      expect(state.is("login"));
+      expect(location.url()).toEqual('/login');
+    });
+
+    it('Should not redirect if logged', function() {
+      spyOn(AuthMock,'isAuthed').andReturn(true);
+      location.url(url);
+      controller('LoginCtrl', {$scope: scope, $state: state});
+      state.go('main');
+
+      expect(AuthMock.isAuthed).toHaveBeenCalled();
+      expect(state.is("main"));
+      expect(location.url()).toEqual(url);
+    });
+
+    it('Should call the Auth module', function() {
+      spyOn(AuthMock,'login');
+      controller('LoginCtrl', {$scope: scope, $state: state});
+
+      scope.sendLogin();
+      expect(AuthMock.login).toHaveBeenCalled();
+    });
   });
 });
-- 
2.36.6