feat(other): add unit and e2e tests for info modal route (#657)

[mahula]

Test coverage and CR1 fix for PR #657

Bug fix (CR1 from review)

• Remove setTimeout firing after unmount in ModalContent.tsx
• Remove resulting unused setChapter

Unit Tests (Vitest) — InfoRedirect.spec.tsx (5 tests)

• U1: Navigates to /info when enabled and pathname is /
• U2: Does not navigate when enabled=false
• U3: Does not navigate when pathname ≠ /
• U4: Only navigates once (useRef guard)
• U5: Renders nothing (returns null)

E2E Tests (Cypress) — info-modal.cy.ts (4 tests)

• E1: Visiting /info renders the modal
• E2: NavBar ? icon navigates to /info
• E3: ✕ button closes modal and navigates to /
• E4: Content "Close" button closes modal and navigates to /

Design decisions

• No component tests for Modal.tsx: Zero-logic wrapper, tested implicitly via E2E
• Each test covers a distinct code path — no redundancy within or across test levels
• 9 total tests across 2 files covering all behavioral changes introduced by refactor(lib): move info modal to dedicated /info route #657

---

Pull Request opened by Augment Code with guidance from the PR author

[mahula]

augment review

[augmentcode]

🤖 Augment PR Summary

Summary: Improves coverage and stability around the route-based Info modal introduced earlier.

Changes:

• Removes a post-unmount setTimeout from ModalContent to avoid delayed state updates after closing
• Adds Vitest unit tests for InfoRedirect to validate one-time redirect behavior to /info
• Adds Cypress E2E tests covering opening/closing the /info modal via direct navigation and the NavBar link

Technical Notes: E2E assertions rely on the overlay’s Tailwind utility classes (tw:card, tw:backdrop-brightness-75) and validate both close mechanisms route back to /.

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. 1 suggestions posted.

Comment augment review to trigger a new review at any time.