@i18n-micro/core 1.0.28 → 1.1.0

package/tests/route-service.test.ts

@@ -13,7 +13,6 @@ describe('RouteService', () => {
   let routeService: RouteService
   let mockRouter: jest.Mocked<Router>
   let mockI18nConfig: ModuleOptionsExtend
-  let setCookieMock: jest.Mock
   let navigateToMock: jest.Mock
 
   beforeEach(() => {
@@ -47,20 +46,15 @@ describe('RouteService', () => {
       hashMode: false,
     }
 
-    // Mock setCookie and navigateTo
-    setCookieMock = jest.fn()
     navigateToMock = jest.fn()
 
-    // Initialize RouteService
+    // Initialize RouteService (no setCookie - plugin handles cookies)
     routeService = new RouteService(
       mockI18nConfig,
       mockRouter,
       null,
       null,
       navigateToMock,
-      setCookieMock,
-      null,
-      null,
     )
   })
 
@@ -71,14 +65,14 @@ describe('RouteService', () => {
 
   test('getCurrentLocale should return the hash locale if hashMode is enabled', () => {
     const mockI18nConfigWithHashMode = { ...mockI18nConfig, hashMode: true }
-    routeService = new RouteService(mockI18nConfigWithHashMode, mockRouter, 'de', null, navigateToMock, setCookieMock, null, null)
+    routeService = new RouteService(mockI18nConfigWithHashMode, mockRouter, 'de', null, navigateToMock)
     const locale = routeService.getCurrentLocale()
     expect(locale).toBe('de')
   })
 
   test('getCurrentLocale should return the noPrefix locale if noPrefix strategy is enabled', () => {
     const mockI18nConfigWithNoPrefix: ModuleOptionsExtend = { ...mockI18nConfig, strategy: 'no_prefix' }
-    routeService = new RouteService(mockI18nConfigWithNoPrefix, mockRouter, null, 'ru', navigateToMock, setCookieMock, null, null)
+    routeService = new RouteService(mockI18nConfigWithNoPrefix, mockRouter, null, 'ru', navigateToMock)
     const locale = routeService.getCurrentLocale()
     expect(locale).toBe('ru')
   })
@@ -92,7 +86,7 @@ describe('RouteService', () => {
         { code: 'ru', iso: 'ru-RU', displayName: 'Russian' },
       ],
     }
-    routeService = new RouteService(mockI18nConfigWithDisplayName, mockRouter, null, null, navigateToMock, setCookieMock, null, null)
+    routeService = new RouteService(mockI18nConfigWithDisplayName, mockRouter, null, null, navigateToMock)
     const route = { name: 'localized-about-en' } as RouteLocationNormalizedLoaded
     const displayName = routeService.getCurrentName(route)
     expect(displayName).toBe('English')
@@ -157,26 +151,6 @@ describe('RouteService', () => {
     expect(fullPath).toBe('https://example.com/de/about')
   })
 
-  test('updateCookies should update cookies for hashMode and noPrefixStrategy', () => {
-    const mockI18nConfigWithHashMode: ModuleOptionsExtend = {
-      ...mockI18nConfig,
-      hashMode: true,
-      strategy: 'prefix_except_default',
-    }
-    routeService = new RouteService(mockI18nConfigWithHashMode, mockRouter, 'de', null, navigateToMock, setCookieMock, null, null)
-    routeService.updateCookies('de')
-    expect(setCookieMock).toHaveBeenCalledWith('hash-locale', 'de')
-
-    const mockI18nConfigWithNoPrefix: ModuleOptionsExtend = {
-      ...mockI18nConfig,
-      strategy: 'no_prefix',
-      localeCookie: 'custom-locale',
-    }
-    routeService = new RouteService(mockI18nConfigWithNoPrefix, mockRouter, null, 'ru', navigateToMock, setCookieMock, null, null)
-    routeService.updateCookies('ru')
-    expect(setCookieMock).toHaveBeenCalledWith('custom-locale', 'ru')
-  })
-
   test('resolveLocalizedRoute should return the correct localized route', () => {
     const to = { path: '/about' } as RouteLocationAsPathGeneric
     mockRouter.resolve.mockReturnValueOnce({
@@ -210,21 +184,38 @@ describe('RouteService', () => {
     expect(locale).toBe('ru') // Should extract from URL
   })
 
-  test('getCurrentLocale should use cookie locale when route params and URL path are missing', () => {
+  test('getCurrentLocale should return defaultLocale for prefix_except_default when URL has no locale prefix', () => {
     routeService = new RouteService(
-      mockI18nConfig,
+      mockI18nConfig, // strategy: prefix_except_default
       mockRouter,
       null,
       null,
       navigateToMock,
-      setCookieMock,
-      'de', // cookie locale
-      'user-locale', // cookie name
+      () => 'de', // getter from plugin (cookie/state)
+    )
+    mockRouter.currentRoute.value.params = {} // Remove locale from params
+    mockRouter.currentRoute.value.path = '/' // No locale in path
+    const locale = routeService.getCurrentLocale()
+    // For prefix_except_default: URL without locale = defaultLocale (SEO requirement)
+    // Getter is NOT used because URL explicitly indicates defaultLocale
+    expect(locale).toBe('en')
+  })
+
+  test('getCurrentLocale should use getDefaultLocale for no_prefix strategy', () => {
+    const noPrefixConfig = { ...mockI18nConfig, strategy: 'no_prefix' as const }
+    routeService = new RouteService(
+      noPrefixConfig,
+      mockRouter,
+      null,
+      'de', // noPrefixDefault
+      navigateToMock,
+      () => 'de', // getter from plugin (cookie/state)
     )
     mockRouter.currentRoute.value.params = {} // Remove locale from params
     mockRouter.currentRoute.value.path = '/' // No locale in path
     const locale = routeService.getCurrentLocale()
-    expect(locale).toBe('de') // Should use cookie
+    // For no_prefix: getter/noPrefixDefault determines locale
+    expect(locale).toBe('de')
   })
 
   test('getCurrentLocale should prioritize route params over URL path', () => {
@@ -234,55 +225,19 @@ describe('RouteService', () => {
     expect(locale).toBe('de') // Should use route params
   })
 
-  test('getCurrentLocale should prioritize URL path over cookie', () => {
+  test('getCurrentLocale should prioritize URL path over getDefaultLocale', () => {
     routeService = new RouteService(
       mockI18nConfig,
       mockRouter,
       null,
       null,
       navigateToMock,
-      setCookieMock,
-      'de', // cookie locale
-      'user-locale', // cookie name
+      () => 'de', // getter
     )
     mockRouter.currentRoute.value.params = {} // No locale in params
     mockRouter.currentRoute.value.path = '/ru/sdfsdf' // URL has locale
     const locale = routeService.getCurrentLocale()
-    expect(locale).toBe('ru') // Should use URL path, not cookie
-  })
-
-  test('updateCookies should update cookie for regular strategy', () => {
-    routeService = new RouteService(
-      mockI18nConfig,
-      mockRouter,
-      null,
-      null,
-      navigateToMock,
-      setCookieMock,
-      null,
-      'user-locale', // cookie name
-    )
-    routeService.updateCookies('ru')
-    expect(setCookieMock).toHaveBeenCalledWith('user-locale', 'ru')
-  })
-
-  test('updateCookies should use localeCookie fallback for regular strategy', () => {
-    const mockI18nConfigWithLocaleCookie: ModuleOptionsExtend = {
-      ...mockI18nConfig,
-      localeCookie: 'custom-regular-cookie',
-    }
-    routeService = new RouteService(
-      mockI18nConfigWithLocaleCookie,
-      mockRouter,
-      null,
-      null,
-      navigateToMock,
-      setCookieMock,
-      null,
-      null,
-    )
-    routeService.updateCookies('ru')
-    expect(setCookieMock).toHaveBeenCalledWith('custom-regular-cookie', 'ru')
+    expect(locale).toBe('ru') // Should use URL path, not getter
   })
 
   test('getCurrentName should return null if displayName is missing', () => {
@@ -294,7 +249,7 @@ describe('RouteService', () => {
         { code: 'ru', iso: 'ru-RU' },
       ],
     }
-    routeService = new RouteService(mockI18nConfigWithoutDisplayName, mockRouter, null, null, navigateToMock, setCookieMock, null, null)
+    routeService = new RouteService(mockI18nConfigWithoutDisplayName, mockRouter, null, null, navigateToMock)
     const route = { name: 'localized-about-en' } as RouteLocationNormalizedLoaded
     const displayName = routeService.getCurrentName(route)
     expect(displayName).toBeNull()
@@ -370,11 +325,6 @@ describe('RouteService', () => {
     })
   })
 
-  test('updateCookies should use default cookie when hashMode and noPrefixStrategy are disabled', () => {
-    routeService.updateCookies('de')
-    expect(setCookieMock).toHaveBeenCalledWith('user-locale', 'de')
-  })
-
   test('getCurrentLocale should extract locale from fullPath with query params when route.path is missing', () => {
     mockRouter.currentRoute.value.params = {}
     mockRouter.currentRoute.value.path = '' // Simulate missing path (e.g. error state)