fix: assert font provider via import, not bunny( substring (MAN-139)
All checks were successful
CI / static (pull_request) Successful in 2m37s
CI / unit (pull_request) Successful in 3m55s
CI / feature (pull_request) Successful in 2m59s
CI / container (pull_request) Successful in 1m12s
CI / browser (pull_request) Successful in 5m40s

Comment in vite.config.js still said bunny(), so assertStringNotContainsString failed in unit coverage.

Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
2026-08-13 23:47:52 -03:00
parent 290585d713
commit 444153bf3d
2 changed files with 12 additions and 4 deletions

View File

@@ -72,9 +72,19 @@ class HeritageEditorialTokensTest extends TestCase
{ {
$vite = (string) file_get_contents(base_path('vite.config.js')); $vite = (string) file_get_contents(base_path('vite.config.js'));
$this->assertMatchesRegularExpression(
"/^import \\{ fontsource \\} from 'laravel-vite-plugin\\/fonts';/m",
$vite,
);
$this->assertDoesNotMatchRegularExpression(
'/^import \\{[^}]*\\bbunny\\b/m',
$vite,
);
$this->assertDoesNotMatchRegularExpression(
'/^import \\{[^}]*\\bgoogle\\b/m',
$vite,
);
$this->assertStringContainsString("fontsource('EB Garamond'", $vite); $this->assertStringContainsString("fontsource('EB Garamond'", $vite);
$this->assertStringNotContainsString('bunny(', $vite);
$this->assertStringNotContainsString('google(', $vite);
$this->assertStringNotContainsString('Instrument Sans', $vite); $this->assertStringNotContainsString('Instrument Sans', $vite);
} }

View File

@@ -76,8 +76,6 @@ export default defineConfig({
], ],
refresh: true, refresh: true,
fonts: [ fonts: [
// fontsource reads node_modules so `vite build` stays offline.
// bunny() timed out on fonts.bunny.net in CI (run 31, job feature).
fontsource('EB Garamond', { fontsource('EB Garamond', {
weights: [400, 500, 600], weights: [400, 500, 600],
}), }),