Merge branch 'develop'

Author: vlavrynovych

bower.json

@@ -4,7 +4,7 @@
   "authors": [
     "Volodymyr Lavrynovych <volodyalavrynovych@gmail.com>"
   ],
-  "version": "0.7.0",
+  "version": "0.7.1",
   "description": "Provides ability to easily handle most of the logic related to the authentication process and page load for the AngularJS SPA",
   "main": "dist/angular-spa-auth.min.js",
   "keywords": [

dist/angular-spa-auth.js

@@ -151,8 +151,12 @@
             }
 
             function goTo(route) {
-                $location.path(route);
-                info('Redirected to the ' + route);
+                if(route == config.uiRoutes.login && service.isAuthenticated()) {
+                    $location.path(getHome());
+                } else {
+                    $location.path(route);
+                    info('Redirected to the ' + route);
+                }
             }
 
             function isAuthenticated() {

dist/angular-spa-auth.min.js

@@ -1,7 +1,7 @@
 {
   "name": "angular-spa-auth",
   "title": "Angular SPA Auth",
-  "version": "0.7.0",
+  "version": "0.7.1",
   "author": "Volodymyr Lavrynovych <volodyalavrynovych@gmail.com>",
   "contributors": [
     "Volodymyr Lavrynovych <volodyalavrynovych@gmail.com>"

dist/angular-spa-auth.min.js.map

@@ -151,8 +151,12 @@
             }
 
             function goTo(route) {
-                $location.path(route);
-                info('Redirected to the ' + route);
+                if(route == config.uiRoutes.login && service.isAuthenticated()) {
+                    $location.path(getHome());
+                } else {
+                    $location.path(route);
+                    info('Redirected to the ' + route);
+                }
             }
 
             function isAuthenticated() {

package.json

@@ -31,6 +31,7 @@ describe('angular-spa-auth', function () {
         it('User is not logged in on start', function () {
             //given:
             expect($rootScope.currentUser).toBeUndefined();
+            events = [];
 
             $rootScope.$on("$routeChangeStart", function(e) {
                 e.path = $location.path();
@@ -118,6 +119,48 @@ describe('angular-spa-auth', function () {
             checkEvent(true);
         });
 
+
+        it('Refresh when you are on the login page and login', function () {
+            //given:
+            expect($rootScope.currentUser).toBeUndefined();
+            events = [];
+            AuthService.config.uiRoutes.target = AuthService.config.uiRoutes.login;
+
+            $rootScope.$on("$routeChangeStart", function(e) {
+                e.path = $location.path();
+                e.i = events.length;
+                events.push(e);
+            });
+
+            //when: isAuthenticated method is triggered but we still do not know status of user
+            changeRoute(AuthService.config.uiRoutes.login);
+
+            //then: login page is available
+            expect($location.path()).toEqual(AuthService.config.uiRoutes.login);
+            expect($rootScope.currentUser).toBeUndefined();
+            checkEvent(false);
+
+            //when: trigger after init
+            $httpBackend.flush();
+
+            //then: we are on login page and user is set to false, because we checked his authentication status
+            expect($location.path()).toEqual(AuthService.config.uiRoutes.login);
+            expect($rootScope.currentUser).toEqual(false);
+
+            //when: try to login using some credentials
+            AuthService.login(credentials);
+            $httpBackend.flush();
+
+            //then: successfully authenticated and redirected to the home page EVEN if the target is login
+            expect($location.path()).toEqual(AuthService.config.uiRoutes.home);
+
+            //when: try again to go to the login page
+            changeRoute(AuthService.config.uiRoutes.login);
+
+            //then: successfully authenticated and redirected to the home page EVEN if the target is login
+            expect($location.path()).toEqual(AuthService.config.uiRoutes.home);
+        });
+
         function changeRoute(path) {
             $location.path(path);
             $rootScope.$broadcast("$routeChangeStart", {