Skip to content

test: verify notification page and unread badge - #118

Open
shutingcasey wants to merge 2 commits into
mainfrom
casey-my-claims-page
Open

test: verify notification page and unread badge#118
shutingcasey wants to merge 2 commits into
mainfrom
casey-my-claims-page

Conversation

@shutingcasey

@shutingcasey shutingcasey commented Jul 26, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Tests
    • Added coverage to verify students receive notifications when claims are rejected, picked up, or approved.
    • Added coverage confirming the unread notification badge updates immediately after a notification is marked as read.

@coderabbitai

coderabbitai Bot commented Jul 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: de2f2041-bf42-448e-aa38-632ca65c601c

📥 Commits

Reviewing files that changed from the base of the PR and between f400142 and 2a809be.

📒 Files selected for processing (2)
  • backend/tests/claims.integration.test.ts
  • foundit-ui/tests/components/NotificationsBadgeSync.test.tsx

📝 Walkthrough

Walkthrough

The changes expand claim-status integration assertions for persisted notifications and add a UI test covering synchronization between notification read state and the navbar unread badge.

Changes

Claim notification assertions

Layer / File(s) Summary
Claim status notification payloads
backend/tests/claims.integration.test.ts
Rejected, picked-up, and approved claim-status tests verify tx.notification.create payload fields, references, recipient, and select usage.

Unread notification badge synchronization

Layer / File(s) Summary
Unread badge synchronization
foundit-ui/tests/components/NotificationsBadgeSync.test.tsx
A component test verifies that marking the only unread notification as read updates the navbar badge from 1 to 0.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested reviewers: hnam10, renv39

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main theme of the changes: notification-related tests and unread badge behavior.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch casey-my-claims-page

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants