Skip to content

Commit b659dc1

Browse files
committed
Fix speculation rules test for mixed list/document rules
Home page is now a list rule (source+urls) instead of document rule (where). Test was iterating over all rules assuming where.href_matches exists on every rule — now checks list rules separately.
1 parent 8b63e8e commit b659dc1

1 file changed

Lines changed: 9 additions & 3 deletions

File tree

packages/adapter-vite/__tests__/ssg-postprocess.test.ts

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -521,10 +521,16 @@ Deno.test('buildSpeculationRulesJson generates heuristic prerender rules from ro
521521
r.where && !r.where.href_matches
522522
),
523523
);
524-
// Top-level page with wildcard
524+
// Top-level page with wildcard (document rule — has where.href_matches)
525525
assertExists(
526-
parsed.prerender.some((r: { where: { href_matches: string } }) =>
527-
r.where.href_matches === '/about/*'
526+
parsed.prerender.some((r: { where?: { href_matches: string } }) =>
527+
r.where?.href_matches === '/about/*'
528+
),
529+
);
530+
// Home page should be a list rule (source + urls, no where)
531+
assertExists(
532+
parsed.prerender.some((r: { source?: string; urls?: string[] }) =>
533+
r.source === 'list' && r.urls?.includes('/')
528534
),
529535
);
530536
// Dynamic routes (with :) should be excluded

0 commit comments

Comments
 (0)