Add unit/component tests for the useSessionExpiry hook and SessionExpiryNotification

[Baskarayelu]

📋 Description

lib/client/useSessionExpiry.ts drives the session-expiry warning rendered by components/SessionExpiryNotification.tsx and wired through components/SessionExpiryProvider.tsx. There are session tests under tests/session, but the client hook's timer/countdown logic and re-authentication trigger are not directly covered.

Why this matters: a wallet-authenticated session that expires mid-flow without a clear warning can drop a user in the middle of a send. The countdown and re-auth prompt are trust-critical.

🎯 Requirements & Context

Functional requirements

• Add tests/session/use-session-expiry.test.ts (matched by the existing tests/session/** Vitest include).
• Use fake timers to assert: the warning appears at the threshold, the countdown decrements, expiry triggers the documented action, and re-auth/extend clears the warning.
• Cover cleanup: timers cleared on unmount (no leaks).

Context & constraints

• Vitest environment: node; mock timers with vi.useFakeTimers().
• If the hook reads wall-clock directly, accept an injectable clock for testability (minimal change).

🛠️ Suggested Execution

git checkout -b test/use-session-expiry

• Write the suite, add injectable clock if needed.

npm run lint
npm run test:coverage

• Edge cases: session already expired on mount, extend before expiry, rapid mount/unmount, threshold boundary tick.

Example commit message

test(session): cover useSessionExpiry countdown + cleanup

Adds Vitest fake-timer suite for warning threshold, countdown, expiry
action, re-auth reset, and unmount cleanup.

✅ Acceptance Criteria & Guidelines

Requirement	Target
Warning/countdown/expiry covered	Required
Timer cleanup asserted	Required
Hook coverage	≥ 90%
Deterministic (fake timers)	Required
Lint clean	Required
Timeframe	96 hours from assignment

💬 Community & Support

Join the RemitWise contributor community on Discord: https://discord.gg/CtQuPZFMA

Comment to claim before starting. 🚀

[Menjay7]

Hi! I came across this issue and would love to work on it. I have experience with similar tasks and I'm confident I can investigate, implement a clean solution, and thoroughly test it before submitting a PR. If the issue is still open, I would appreciate it if you could assign it to me. Thank you!

[Anichris-koded]

I am a professional frontend developer with strong experience in testing React applications, custom hooks, and user-facing components. I can create comprehensive unit and component tests for the useSessionExpiry hook and SessionExpiryNotification, ensuring accurate session timeout handling, notification behavior, and edge-case coverage. My focus is on improving code quality, increasing test coverage, and delivering reliable, maintainable applications that perform consistently in production.

[jaynomyaro]

Hi! I’m a full-stack developer with a strong blockchain background. I’d love to handle this issue and can deliver a clean, reliable solution. Kindly assign it to me.

[Senorespecial]

I would like to work on this issue please assign

[grantfox-oss]

🦊 GrantFox — @Senorespecial has been assigned to this issue!

Next steps:

1. Open a Pull Request referencing this issue (e.g., Closes #475)
2. Your PR will be reviewed by the Remitwise-Org maintainers

Good luck! Track your progress on GrantFox.