Skip to content

test(fonts): cover text-subset faces at bundled weights - #7315

Open
sx4im wants to merge 1 commit into
heygen-com:mainfrom
sx4im:test-7313-text-subset-bundled-weights
Open

sx4im wants to merge 1 commit into
heygen-com:mainfrom
sx4im:test-7313-text-subset-bundled-weights

Conversation

@sx4im

@sx4im sx4im commented Nov 6, 2026

Copy link
Copy Markdown

Summary

Regression test only, no source changes.

The bug in #7313 (non-Latin text at a bundled weight falling back to a system font) was already fixed upstream by #7299 after the 0.9.19 release. This test guards that fix: it checks the Google text= subset face is emitted at weights 400 and 700 beside the bundled Latin-only faces.

Related issue

Related to #7313

Verification

New test passes locally in about 1.5s. It uses a mocked fetch with the real Google text= response shape and a temp font cache dir, so it needs no network.

bun run --cwd packages/core test src/fonts/deterministicFonts-textSubsetBundledWeights.test.ts
# 1 passed

oxlint and oxfmt clean on the new file.

Regression test for heygen-com#7313: the Google text= subset must be emitted at
each bundled weight beside the bundled face, or non-Latin text at 400/700
falls back to a system font in Chromium.
SanHsien added a commit to SanHsien/hyperframes that referenced this pull request Nov 6, 2026
…, record duplicate rule

Review PR heygen-com#7304-heygen-com#7335: 10 merged already adopted or deferred; 10 open or draft PRs
pinned (heygen-com#7304, heygen-com#7308, heygen-com#7311, heygen-com#7315, heygen-com#7321, heygen-com#7330-heygen-com#7334); open duplicates of
pinned heygen-com#7281/heygen-com#7285/heygen-com#7286/heygen-com#7296 are tracked by those pins. strict exit 0;
monitored 110 -> 120.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant