test: merge issue #1889 test

specs/fixtures/basic/nuxt.config.ts

@@ -20,6 +20,8 @@ export default defineNuxtConfig({
       }
     ],
     defaultLocale: 'en',
+    // TODO: remove this later, apply in test `setup`
+    // `false` will not be overwritten by `runtimeConfig` making this fixture less reusable
     detectBrowserLanguage: false,
     vueI18n: './config/i18n.config.ts'
   }

specs/fixtures/basic/pages/index.vue

@@ -66,5 +66,8 @@ useHead(() => ({
       go to product foo
     </NuxtLink>
     <NuxtLink id="link-history" :to="localePath({ name: 'history' })">go to history</NuxtLink>
+    <NuxtLink id="link-define-i18n-route-false" exact :to="localePath('/define-i18n-route-false')">
+      go to defineI18nRoute(false)
+    </NuxtLink>
   </div>
 </template>

specs/routing_strategies/prefix.spec.ts

@@ -1,7 +1,7 @@
 import { describe, test, expect } from 'vitest'
 import { fileURLToPath } from 'node:url'
 import { setup, url, fetch } from '../utils'
-import { getText, getData, renderPage, waitForURL } from '../helper'
+import { getText, getData, renderPage, waitForURL, startServerWithRuntimeConfig } from '../helper'
 
 import type { Response } from 'playwright'
 
@@ -12,12 +12,23 @@ await setup({
   nuxtConfig: {
     i18n: {
       strategy: 'prefix',
-      defaultLocale: 'en'
+      defaultLocale: 'en',
+      // fixture uses `false` which cannot be overwritten using runtimeConfig
+      detectBrowserLanguage: {}
     }
   }
 })
 
 describe('strategy: prefix', async () => {
+  beforeEach(async () => {
+    // use original fixture `detectBrowserLanguage` value as default for tests, overwrite here needed
+    await startServerWithRuntimeConfig({
+      public: {
+        i18n: { detectBrowserLanguage: false }
+      }
+    })
+  })
+
   test.each([
     ['/', '/en'],
     ['/about', '/en/about'],
@@ -107,4 +118,38 @@ describe('strategy: prefix', async () => {
     // current locale
     expect(await getText(page, '#lang-switcher-current-locale code')).toEqual('fr')
   })
+
+  test('(#1889) navigation to page with `defineI18nRoute(false)`', async () => {
+    const restore = await startServerWithRuntimeConfig({
+      public: {
+        i18n: {
+          detectBrowserLanguage: {
+            useCookie: true,
+            alwaysRedirect: false,
+            redirectOn: 'root'
+          }
+        }
+      }
+    })
+
+    const { page } = await renderPage('/', { locale: 'en' })
+    await waitForURL(page, '/en')
+
+    // switch 'fr' locale
+    await page.locator('#lang-switcher-with-set-locale a').click()
+    await waitForURL(page, '/fr')
+    expect(await getText(page, '#home-header')).toEqual('Accueil')
+
+    // navigate to disabled route
+    await page.locator('#link-define-i18n-route-false').click()
+    await waitForURL(page, '/define-i18n-route-false')
+
+    expect(await getText(page, '#disable-route-text')).toEqual('Page with disabled localized route')
+
+    // back to home
+    await page.locator('#goto-home').click()
+    expect(await getText(page, '#home-header')).toEqual('Accueil')
+
+    await restore()
+  })
 })