mirror of
https://github.com/hanzoai/next.js.git
synced 2026-10-30 14:46:38 +00:00
Jiwon ChoiandGitHub
6357b5482e
### Why? `next/font` depends on the Google Fonts metrics from the `@capsizecss/metrics`. Added support for Geist at https://github.com/seek-oss/capsize/pull/217, so we update version. ### How? - Bump `@capsizecss/metrics` to 3.4.0 - Update the test to match the metrics. #### Bonus - Modified the test to use `.toMatchInlineSnapshot` to identify and update future changes more easily. ### RFC The `@capsizecss` team is planning to [automate the updating process](https://github.com/seek-oss/capsize/pull/217#pullrequestreview-2431209858). Should we continue to depend on them or replace them with our custom script to fetch Google Font API (requires API key)? x-ref: https://github.com/vercel/next.js/issues/47115 x-ref: [Twitter Feedback](https://x.com/illyism/status/1855500917589602706)
71 lines
2.3 KiB
TypeScript
71 lines
2.3 KiB
TypeScript
import { createNext, FileRef } from 'e2e-utils'
|
|
import { NextInstance } from 'e2e-utils'
|
|
import { join } from 'path'
|
|
import webdriver from 'next-webdriver'
|
|
|
|
const mockedGoogleFontResponses = require.resolve(
|
|
'./google-font-mocked-responses.js'
|
|
)
|
|
|
|
describe('next/font/google fetch error', () => {
|
|
const isDev = (global as any).isNextDev
|
|
let next: NextInstance
|
|
|
|
if ((global as any).isNextDeploy) {
|
|
it('should skip next deploy for now', () => {})
|
|
return
|
|
}
|
|
|
|
beforeAll(async () => {
|
|
next = await createNext({
|
|
files: {
|
|
pages: new FileRef(join(__dirname, 'google-fetch-error/pages')),
|
|
},
|
|
env: {
|
|
NEXT_FONT_GOOGLE_MOCKED_RESPONSES: mockedGoogleFontResponses,
|
|
},
|
|
skipStart: true,
|
|
})
|
|
})
|
|
afterAll(() => next.destroy())
|
|
|
|
if (isDev) {
|
|
it('should use a fallback font in dev', async () => {
|
|
await next.start()
|
|
const outputIndex = next.cliOutput.length
|
|
const browser = await webdriver(next.url, '/')
|
|
|
|
const ascentOverride = await browser.eval(
|
|
'Array.from(document.fonts.values()).find(font => font.family.includes("Inter Fallback")).ascentOverride'
|
|
)
|
|
expect(ascentOverride).toMatchInlineSnapshot(`"90.44%"`)
|
|
|
|
const descentOverride = await browser.eval(
|
|
'Array.from(document.fonts.values()).find(font => font.family.includes("Inter Fallback")).descentOverride'
|
|
)
|
|
expect(descentOverride).toMatchInlineSnapshot(`"22.52%"`)
|
|
|
|
const lineGapOverride = await browser.eval(
|
|
'Array.from(document.fonts.values()).find(font => font.family.includes("Inter Fallback")).lineGapOverride'
|
|
)
|
|
expect(lineGapOverride).toMatchInlineSnapshot(`"0%"`)
|
|
|
|
const sizeAdjust = await browser.eval(
|
|
'Array.from(document.fonts.values()).find(font => font.family.includes("Inter Fallback")).sizeAdjust'
|
|
)
|
|
expect(sizeAdjust).toMatchInlineSnapshot(`"107.12%"`)
|
|
|
|
expect(next.cliOutput.slice(outputIndex)).toInclude(
|
|
'Failed to download `Inter` from Google Fonts. Using fallback font instead.'
|
|
)
|
|
})
|
|
} else {
|
|
it('should error when not in dev', async () => {
|
|
await expect(next.start()).rejects.toThrow('next build failed')
|
|
expect(next.cliOutput).toInclude(
|
|
'Failed to fetch `Inter` from Google Fonts.'
|
|
)
|
|
})
|
|
}
|
|
})
|