fix(toast): cancel exit timer on show change or unmount to prevent stale onClose (closes #23)

[Ranjeet2063]

Summary of Changes

Resolves issue #23 where the 300ms exit timer cleanup in components/Toast.jsx and components/Notification.tsx was returned from inside the outer setTimeout callback where it was never executed. When a new toast was shown within that 300ms window or the component was unmounted mid-exit, the stale onClose fired and hid the new toast prematurely.

Key Changes

1. Restructured Timer Cleanup:
   • In components/Toast.jsx, tracked timer and exitTimer in the useEffect body.
   • Returned a proper cleanup function that clears both timer and exitTimer whenever show, onClose, or time changes or on unmount.
   • Applied the exact same corrected cleanup pattern to components/Notification.tsx (Toast component).
2. Comprehensive Unit Tests:
   • Created tests/components/Toast.test.tsx using Vitest fake timers verifying for both components:
     1. Showing toast A, then re-showing toast B within 300ms cancels stale onClose for A without prematurely invoking onClose for B.
     2. Unmounting mid-exit transition cancels the exit timer and does not fire onClose.

Verification (2x Verified)

• npx vitest run tests/components/Toast.test.tsx (4/4 tests pass)
• npm run type-check (0 errors)
• npm run lint (0 errors)
• npx prettier --check (100% formatted)

Closes #23

[vercel]

@Ranjeet2063 is attempting to deploy a commit to the Oluremi's projects Team on Vercel.

A member of the Team first needs to authorize it.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.