feat: add no vary on root index test (#31)

Author: orinokai

.eslintrc.cjs

@@ -25,5 +25,12 @@ module.exports = {
  'import/no-anonymous-default-export': 'off',
  },
  },
+ {
+ files: ['src/**/*.test.*'],
+ rules: {
+ 'max-statements': off,
+ 'max-lines-per-function': off,
+ },
+ },
  ],
 }

src/helpers/headers.test.ts

@@ -22,20 +22,26 @@ describe('headers', () => {
 
  setVaryHeaders(headers, request, defaultConfig)
 
- expect(headers.set).toBeCalledWith('netlify-vary', 'cookie=__prerender_bypass|__next_preview_data')
+ expect(headers.set).toBeCalledWith(
+ 'netlify-vary',
+ 'cookie=__prerender_bypass|__next_preview_data',
+ )
  })
 
  test('with expected vary headers', () => {
  const givenHeaders = {
- 'vary': 'Accept, Accept-Language',
+ vary: 'Accept, Accept-Language',
  }
  const headers = new Headers(givenHeaders)
  const request = new Request(defaultUrl)
  vi.spyOn(headers, 'set')
 
  setVaryHeaders(headers, request, defaultConfig)
 
- expect(headers.set).toBeCalledWith('netlify-vary', 'header=Accept|Accept-Language,cookie=__prerender_bypass|__next_preview_data')
+ expect(headers.set).toBeCalledWith(
+ 'netlify-vary',
+ 'header=Accept|Accept-Language,cookie=__prerender_bypass|__next_preview_data',
+ )
  })
 
  test('with no languages if i18n config has localeDetection disabled', () => {
@@ -53,7 +59,30 @@ describe('headers', () => {
 
  setVaryHeaders(headers, request, config)
 
- expect(headers.set).toBeCalledWith('netlify-vary', 'cookie=__prerender_bypass|__next_preview_data')
+ expect(headers.set).toBeCalledWith(
+ 'netlify-vary',
+ 'cookie=__prerender_bypass|__next_preview_data',
+ )
+ })
+
+ test('with no languages if path is root index', () => {
+ const headers = new Headers()
+ const request = new Request(`${defaultUrl}/another/path`)
+ const config = {
+ ...defaultConfig,
+ i18n: {
+ locales: ['en', 'de'],
+ defaultLocale: 'default',
+ },
+ }
+ vi.spyOn(headers, 'set')
+
+ setVaryHeaders(headers, request, config)
+
+ expect(headers.set).toBeCalledWith(
+ 'netlify-vary',
+ 'cookie=__prerender_bypass|__next_preview_data',
+ )
  })
 
  test('with expected languages if i18n config has locales', () => {
@@ -70,7 +99,10 @@ describe('headers', () => {
 
  setVaryHeaders(headers, request, config)
 
- expect(headers.set).toBeCalledWith('netlify-vary', 'language=en|de|fr,cookie=__prerender_bypass|__next_preview_data|NEXT_LOCALE')
+ expect(headers.set).toBeCalledWith(
+ 'netlify-vary',
+ 'language=en|de|fr,cookie=__prerender_bypass|__next_preview_data|NEXT_LOCALE',
+ )
  })
 
  test('with expected languages if i18n config has locales and basePath matches the current path', () => {
@@ -88,7 +120,10 @@ describe('headers', () => {
 
  setVaryHeaders(headers, request, config)
 
- expect(headers.set).toBeCalledWith('netlify-vary', 'language=en|de|fr,cookie=__prerender_bypass|__next_preview_data|NEXT_LOCALE')
+ expect(headers.set).toBeCalledWith(
+ 'netlify-vary',
+ 'language=en|de|fr,cookie=__prerender_bypass|__next_preview_data|NEXT_LOCALE',
+ )
  })
  })
  })
@@ -138,8 +173,16 @@ describe('headers', () => {
 
  setCacheControlHeaders(headers)
 
- expect(headers.set).toHaveBeenNthCalledWith(1, 'cache-control', 'public, max-age=0, must-revalidate')
- expect(headers.set).toHaveBeenNthCalledWith(2, 'netlify-cdn-cache-control', 'public, max-age=0, must-revalidate')
+ expect(headers.set).toHaveBeenNthCalledWith(
+ 1,
+ 'cache-control',
+ 'public, max-age=0, must-revalidate',
+ )
+ expect(headers.set).toHaveBeenNthCalledWith(
+ 2,
+ 'netlify-cdn-cache-control',
+ 'public, max-age=0, must-revalidate',
+ )
  })
 
  test('should remove "s-maxage" from "cache-control" header', () => {
@@ -152,7 +195,11 @@ describe('headers', () => {
  setCacheControlHeaders(headers)
 
  expect(headers.set).toHaveBeenNthCalledWith(1, 'cache-control', 'public')
- expect(headers.set).toHaveBeenNthCalledWith(2, 'netlify-cdn-cache-control', 'public, s-maxage=604800')
+ expect(headers.set).toHaveBeenNthCalledWith(
+ 2,
+ 'netlify-cdn-cache-control',
+ 'public, s-maxage=604800',
+ )
  })
 
  test('should remove "stale-while-revalidate" from "cache-control" header', () => {
@@ -165,10 +212,14 @@ describe('headers', () => {
  setCacheControlHeaders(headers)
 
  expect(headers.set).toHaveBeenNthCalledWith(1, 'cache-control', 'max-age=604800')
- expect(headers.set).toHaveBeenNthCalledWith(2, 'netlify-cdn-cache-control', 'max-age=604800, stale-while-revalidate=86400')
+ expect(headers.set).toHaveBeenNthCalledWith(
+ 2,
+ 'netlify-cdn-cache-control',
+ 'max-age=604800, stale-while-revalidate=86400',
+ )
  })
 
- test('should set default "cache-control" header if it contains both "s-maxage" and "stale-whie-revalidate"', () => {
+ test('should set default "cache-control" header if it contains only "s-maxage" and "stale-whie-revalidate"', () => {
  const givenHeaders = {
  'cache-control': 's-maxage=604800, stale-while-revalidate=86400',
  }
@@ -177,8 +228,16 @@ describe('headers', () => {
 
  setCacheControlHeaders(headers)
 
- expect(headers.set).toHaveBeenNthCalledWith(1, 'cache-control', 'public, max-age=0, must-revalidate')
- expect(headers.set).toHaveBeenNthCalledWith(2, 'netlify-cdn-cache-control', 's-maxage=604800, stale-while-revalidate=86400')
+ expect(headers.set).toHaveBeenNthCalledWith(
+ 1,
+ 'cache-control',
+ 'public, max-age=0, must-revalidate',
+ )
+ expect(headers.set).toHaveBeenNthCalledWith(
+ 2,
+ 'netlify-cdn-cache-control',
+ 's-maxage=604800, stale-while-revalidate=86400',
+ )
  })
  })