test: add unit test for in-app notification filtering
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 90/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- typescript
- Domain
- testing
Research direction
Start in tests/notifications.test.ts and review the existing tests for deliverInApp and dispatchNotification. Add coverage for an empty or unconfigured recipient state, asserting it resolves with the expected fallback status without an unhandled exception. Run npm test and confirm the new test passes.
Written by the indexing model from the issue text.
Description
🎯 What needs to be done
In tests/notifications.test.ts, add a unit test asserting that notification delivery helpers handle empty recipient lists gracefully without throwing unhandled exceptions.
Why this matters
Ensures resilient notification delivery when no channels are configured or available.
📍 Where to work
tests/notifications.test.ts
🧭 Implementation guidance
- Open
tests/notifications.test.ts. - Add a test case testing
deliverInAppordispatchNotificationwith an empty or unconfigured state. - Assert that it resolves safely with the expected fallback status.
✅ Acceptance criteria
- Unit test added in
tests/notifications.test.ts. - Test passes when running
npm test.
🧪 Verification
- Dominant language
- TypeScript
- Stars
- 5
- Forks
- 7
- Avg merge
- 58m
- Merged PRs (30d)
- 42
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from TheVicky1/Pact_OS
-
area:dashboard beginner friendly bug community contributions-welcome copy difficulty:beginner difficulty:easy good first issue hacktoberfest help wanted open-source up-for-grabs
Difficulty 1/5 Under an hour Newbie friendliness 92/100
-
accessibility area:tasks beginner friendly bug community contributions-welcome difficulty:beginner difficulty:easy good first issue hacktoberfest help wanted open-source up-for-grabs
Difficulty 2/5 Under an hour Newbie friendliness 92/100
-
area:habits beginner friendly bug community contributions-welcome difficulty:beginner difficulty:easy good first issue hacktoberfest help wanted open-source ui up-for-grabs
Difficulty 1/5 Under an hour Newbie friendliness 90/100
-
area:tasks beginner friendly bug community contributions-welcome difficulty:beginner difficulty:easy good first issue hacktoberfest help wanted open-source testing up-for-grabs
Difficulty 2/5 Under an hour Newbie friendliness 92/100
-
area:core beginner friendly bug community contributions-welcome difficulty:beginner difficulty:easy good first issue hacktoberfest help wanted jsdoc open-source up-for-grabs
Difficulty 1/5 Under an hour Newbie friendliness 95/100
All issues in TheVicky1/Pact_OS
Similar issues
-
calcite-components needs triage refactor
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Esri/calcite-design-system#15203 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 91/100
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficulty 1/5 Under an hour Newbie friendliness 95/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Automattic/studio#4908 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100