Merge pull request #69 from js-accounts/fix

Throw error if no tokens are set when resuming session

Author: TimMikeladze

packages/client/src/AccountsClient.js

@@ -144,6 +144,7 @@ export class AccountsClient {
     } else {
       this.clearTokens();
       this.clearUser();
+      throw new AccountsError('no tokens provided');
     }
   }
 

packages/client/src/AccountsClient.spec.js

@@ -336,9 +336,13 @@ describe('Accounts', () => {
       Accounts.config({}, {});
       Accounts.instance.clearTokens = jest.fn(() => Accounts.instance.clearTokens);
       Accounts.instance.clearUser = jest.fn(() => Accounts.instance.clearUser);
-      await Accounts.refreshSession();
-      expect(Accounts.instance.clearTokens.mock.calls.length).toEqual(1);
-      expect(Accounts.instance.clearUser.mock.calls.length).toEqual(1);
+      try {
+        await Accounts.refreshSession();
+      } catch (err) {
+        expect(err.message).toEqual('no tokens provided');
+        expect(Accounts.instance.clearTokens.mock.calls.length).toEqual(1);
+        expect(Accounts.instance.clearUser.mock.calls.length).toEqual(1);
+      }
     });
     it('clears tokens, users and throws error if bad refresh token provided', async () => {
       Accounts.config({}, {});

packages/client/src/config.js

@@ -21,6 +21,7 @@ export default {
   profilePath: '/',
   changePasswordPath: null,
   homePath: '/',
+  signOutPath: '/',
   // TODO enable all of these
   // onSubmitHook: () => {},
   // onPreSignUpHook: () => new Promise(resolve => resolve()),
@@ -29,6 +30,6 @@ export default {
   onResetPasswordHook: () => redirect(AccountsClient.options().loginPath),
   onVerifyEmailHook: () => redirect(AccountsClient.options().profilePath),
   onSignedInHook: () => redirect(AccountsClient.options().homePath),
-  onSignedOutHook: () => redirect(AccountsClient.options().homePath),
+  onSignedOutHook: () => redirect(AccountsClient.options().signOutPath),
   loginOnSignUp: true,
 };