fix: Don't inject to store if store is not defined

nuxt-modules · Sep 24, 2019 · e547639 · e547639
1 parent b28acc7
commit e547639
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 6 deletions.
diff --git a/src/plugins/main.js b/src/plugins/main.js
@@ -189,13 +189,15 @@ export default async (context) => {
   // Inject seo function
   Vue.prototype.$nuxtI18nSeo = nuxtI18nSeo
 
-  // Inject in store.
-  store.$i18n = app.i18n
+  if (store) {
+    // Inject in store.
+    store.$i18n = app.i18n
 
-  if (store && store.state.localeDomains) {
-    app.i18n.locales.forEach(locale => {
-      locale.domain = store.state.localeDomains[locale.code]
-    })
+    if (store.state.localeDomains) {
+      app.i18n.locales.forEach(locale => {
+        locale.domain = store.state.localeDomains[locale.code]
+      })
+    }
   }
 
   let locale = app.i18n.defaultLocale || null

diff --git a/test/fixture/base.config.js b/test/fixture/base.config.js
@@ -40,6 +40,10 @@ module.exports = {
         }
       },
       fallbackLocale: 'en'
+    },
+    vuex: {
+      syncLocale: true,
+      syncMessages: true
     }
   }
 }
diff --git a/test/module.test.js b/test/module.test.js
@@ -119,6 +119,24 @@ describe('basic', () => {
     })
   })
 
+  describe('store', () => {
+    test('injects $i18n in store', async () => {
+      const window = await nuxt.renderAndGetWindow(url('/'))
+      expect(window.$nuxt.$store.$i18n).toBeDefined()
+    })
+
+    test('syncs i18n locale and messages', async () => {
+      const window = await nuxt.renderAndGetWindow(url('/'))
+      expect(window.$nuxt.$store.state.i18n).toBeDefined()
+      expect(window.$nuxt.$store.state.i18n.locale).toBe('en')
+      expect(window.$nuxt.$store.state.i18n.messages).toEqual(expect.objectContaining({
+        about: 'About us',
+        home: 'Homepage',
+        posts: 'Posts'
+      }))
+    })
+  })
+
   test('/dynamicNested/1/2/3 contains link to /fr/imbrication-dynamique/1/2/3', async () => {
     const html = await get('/dynamicNested/1/2/3')
     expect(cleanUpScripts(html)).toMatchSnapshot()