Merge pull request #6012 from vuestorefront/#5565/empty-payload-clearCurrentUser

fix: #5565/empty payload clear current user

Author: Fifciu

CHANGELOG.md

@@ -45,6 +45,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - ESLint warnings caused by the double import - @lukaszjedrasik
 - Fix Order History Pagination - @AishwaryShrivastav / @lukaszjedrasik ([#4599](https://github.com/vuestorefront/vue-storefront/issues/4599))
 - Fix: Updating URL's params/query params with proper child SKU if options changes - @lukaszjedrasik ([#5981](https://github.com/vuestorefront/vue-storefront/issues/5981))
+- Fix: Passing `newToken: null` as a payload inside `clearCurrentUser` action - @lukaszjedrasik ([#5565](https://github.com/vuestorefront/vue-storefront/issues/5565))
 
 ### Changed / Improved
 

core/modules/user/store/actions.ts

@@ -221,7 +221,7 @@ const actions: ActionTree<UserState, RootState> = {
     }
   },
   clearCurrentUser ({ commit, dispatch }) {
-    commit(types.USER_TOKEN_CHANGED, '')
+    commit(types.USER_TOKEN_CHANGED, { newToken: null })
     commit(types.USER_GROUP_TOKEN_CHANGED, '')
     commit(types.USER_GROUP_CHANGED, null)
     commit(types.USER_INFO_LOADED, null)

core/modules/user/test/unit/store/actions.spec.ts

@@ -396,7 +396,7 @@ describe('User actions', () => {
 
       (userActions as any).clearCurrentUser(contextMock)
 
-      expect(contextMock.commit).toHaveBeenNthCalledWith(1, types.USER_TOKEN_CHANGED, '')
+      expect(contextMock.commit).toHaveBeenNthCalledWith(1, types.USER_TOKEN_CHANGED, { newToken: null })
       expect(contextMock.commit).toHaveBeenNthCalledWith(2, types.USER_GROUP_TOKEN_CHANGED, '')
       expect(contextMock.commit).toHaveBeenNthCalledWith(3, types.USER_GROUP_CHANGED, null)
       expect(contextMock.commit).toHaveBeenNthCalledWith(4, types.USER_INFO_LOADED, null)