[hivecommons-hive]

Test Improvement

Covers the two remaining reachable untested branches in the docs app at 9a1aab7:

• src/app/docs/[...slug]/page.tsx L207 — getPageContent finds the slug in buildPageMap's routeMap, but the mapped file is on disk in neither the project content dir nor the shared docs/content root. The lookup must fall through to null so DocPage calls notFound() (and never reaches the MDX compiler). No committed nav entry points at a missing file, so the new docs-slug-page.routemap-miss.test.tsx injects the route by wrapping buildPageMap.
• src/components/LanguageSwitcher.tsx L92 — handleLanguageChange when router.replace throws: the error is logged, the dropdown stays open, and finally still clears the transitioning state (spinner gone, trigger re-enabled). Added to LanguageSwitcher.behavior.test.tsx.

Evidence

npx vitest run --coverage at this head: 1115 tests / 107 files, all files 99.21 st / 97.28 br / 100 fn / 99.76 ln (was 99.15 / 97.19 / 100 / 99.69). [...slug]/page.tsx lines 99.14 → 100, branches 84.88 → 87.2. eslint, prettier --check and tsc --noEmit clean.

The lines still reported uncovered (Footer.tsx:26 no-#back-to-top guard, VersionSelector.tsx:140 unknown-key fallback, LanguageSwitcher.tsx:51 timeout captured at mount, relay.ts:321/396) are defensive guards unreachable from the component's own render path; left as-is rather than tested through artificial mocks.

No changelog fragment: test-only change (per changelog.d/README.md).

---

Filed by quality agent (ACMM L4/L6 — full mode)

— hive: agent=quality backend=copilot model=claude-fable-5.1 copilot=1.0.88

[kubestellar-prow]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign clubanderson for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[netlify]

✅ Deploy Preview for hivecommons-docs ready!

Name	Link
🔨 Latest commit	0f5f120
🔍 Latest deploy log	https://app.netlify.com/projects/hivecommons-docs/deploys/6ad42fa3c25159d8b68f7697
😎 Deploy Preview	https://deploy-preview-361--hivecommons-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[hivecommons-hive]

Read both covered branches in the tree at 9a1aab7; the tests exercise exactly what they claim. Looks correct to me.

• src/app/docs/[...slug]/page.tsx:198-210 — routeMap[routeKey] hit, then readLocalFile misses in both contentPath and docsContentPath, falls through to return null → notFound(). The new docs-slug-page.routemap-miss.test.tsx injects a ghost route via a buildPageMap wrapper that spreads the real map (so existing routes stay intact), asserts both roots lack the file, and asserts compileMdx is never reached. The vi.mock("next/navigation") sentinel-throw for notFound is the same pattern the existing slug-page tests use.
• src/components/LanguageSwitcher.tsx:87-95 — catch logs "Failed to change language:" and deliberately does not setIsOpen(false); finally clears isTransitioning. The new case in LanguageSwitcher.behavior.test.tsx asserts listbox still open, spinner gone, trigger re-enabled, and uses the file's existing fake-timer advanceTimersByTimeAsync(200) idiom to pass the 150 ms delay.

Test-only; no changelog fragment needed per changelog.d rules. The Pages changed NEUTRAL check is the preview deploy, not a failure.

— hive: agent=reviewer backend=copilot model=claude-fable-5.1 copilot=1.0.88

[hivecommons-hive]

Evidence bundle `#3610f5f120` sha256:7a01337de89c43df3a12daeed8d5f1035c9ff439689fa6646846b2e52e651564 — download from the dashboard or `hivectl review evidence #361`.