fix: do not push locale redirect transition page in user history

1024pix · Jan 6, 2023 · 7daee2b · 7daee2b
1 parent 80a37b6
commit 7daee2b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/pages/pix-site/index.vue b/pages/pix-site/index.vue
@@ -35,7 +35,7 @@ export default {
   mounted() {
     const chosenLocale = this.getLocaleFromCookie()
     if (chosenLocale) {
-      return this.$router.push(`/${chosenLocale}/`)
+      return this.$router.replace(`/${chosenLocale}/`)
     }
     this.shouldDisplayLocaleChoice = true
   },

diff --git a/tests/pages/pix-site/index.test.js b/tests/pages/pix-site/index.test.js
@@ -7,7 +7,7 @@ describe('Index Page', () => {
 
   beforeEach(() => {
     $router = {
-      push: jest.fn(),
+      replace: jest.fn(),
     }
 
     let cookieJar = ''
@@ -32,7 +32,7 @@ describe('Index Page', () => {
         await Vue.nextTick()
 
         // then
-        expect($router.push).not.toHaveBeenCalled()
+        expect($router.replace).not.toHaveBeenCalled()
         const localeLinks = wrapper.findAll('locale-link-stub')
         expect(localeLinks.length).toBe(4)
       })
@@ -51,7 +51,7 @@ describe('Index Page', () => {
         await Vue.nextTick()
 
         // then
-        expect($router.push).toHaveBeenCalledWith('/fr/')
+        expect($router.replace).toHaveBeenCalledWith('/fr/')
         const localeLinks = wrapper.findAll('locale-link-stub')
         expect(localeLinks.length).toBe(0)
       })