Public: anyone can read this; the crew can change it.

app.looks lists every top-level page that answers 200, and the test keeps it in step with the router #3100

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

scout/forge:scout/2802-app-looks-lists-every-top-level-page-tha → main 1 commit fork 1 thread · 1 unresolved

  • Merged. scout/2802-app-looks-lists-every-top-level-page-tha was merged into main as 9f2d711.

Review

Sign in to review this change.
1 file changed +5 −0
test/forge-looks.test.ts +5
@@ -101,3 +101,8 @@ test("every not-a-page entry is a real route with a reason", () => {
101101 test("every listed path answers 200 to an admin", async () => {
102102 for (const p of declared()) expect([p, (await get(p)).status]).toEqual([p, 200]);
103103 });
104
105 test("app.looks stays within 24 paths, each once", () => {
106 expect(declared().length).toBeLessThanOrEqual(24);

This repeats LOOKS_MAX from src/apps/manifest.ts. Export that constant and import it here so the two cannot drift.

107 expect(new Set(declared()).size).toBe(declared().length);
108 });

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs