Merge pull request #178 from mrbatista/master

Clear user data on logout error

Close #178

Author: bajtos

lib/services.template.ejs

@@ -109,6 +109,11 @@ if (typeof module !== 'undefined' && typeof exports !== 'undefined' &&
                   LoopBackAuth.clearStorage();
                   return response.resource;
                 },
+                responseError: function(responseError) {
+                  LoopBackAuth.clearUser();
+                  LoopBackAuth.clearStorage();
+                  return responseError.resource;
+                },
               },
 <% } -%>
 <% if (action.hasResourceParams) { -%>

test.e2e/spec/services.spec.js

@@ -553,6 +553,27 @@ define(['angular', 'given', 'util'], function(angular, given, util) {
           .catch(util.throwHttpError);
       });
 
+      it('clears authentication data on logout error', function() {
+        return givenLoggedInUser()
+          .then(function() {
+            var auth = $injector.get('LoopBackAuth');
+            expect(auth.accessTokenId, 'accessTokenId').to.not.equal(null);
+            auth.accessTokenId = 'fake_access_token';
+            return Customer.logout().$promise;
+          })
+          .then(function() {
+            var auth = $injector.get('LoopBackAuth');
+            expect(auth.accessTokenId, 'accessTokenId').to.equal(null);
+            expect(auth.currentUserId, 'currentUserId').to.equal(null);
+
+            // Check that localStorage was cleared too.
+            auth = getNew('LoopBackAuth');
+            expect(auth.accessTokenId, 'stored accessTokenId').to.equal(null);
+            expect(auth.currentUserId, 'stored currentUserId').to.equal(null);
+          })
+          .catch(util.throwHttpError);
+      });
+
       it('clears authentication data in local and session storage ' +
         'on logout when rememberMe=true and page has been reloaded after login',
       function() {