[DanbabaJr]

Summary

Closes #2122

Added deterministic tests for the "useNotificationStream" hook to cover subscription management and cleanup behavior that were not previously covered by the existing frontend tests.

Changes

• Added tests to verify that duplicate subscriptions are handled correctly.
• Covered notification stream cleanup when a component unmounts.
• Verified that subscriptions are properly released to prevent lingering listeners.
• Added coverage for relevant success, failure, and boundary scenarios.
• Ensured predictable behavior across repeated subscription and cleanup operations.

Testing

• Ran the relevant frontend test suite.
• Verified subscription handling and unmount cleanup behavior.
• Confirmed the tests are suitable for CI execution.

Result

Strengthens notification stream reliability, prevents duplicate subscription issues, and ensures proper resource cleanup throughout the component lifecycle.

[github-actions]

Malware detection scan flagged this pull request

The scan found 1 indicator(s) in the PR diff. This PR is blocked until a maintainer reviews the flagged files.

• frontend/src/app/hooks/useNotificationStream.test.tsx — hidden-appended-config-payload (high)

This is an automated triage signal; maintainers must verify whether the finding is malicious or intentional.

[DanbabaJr]

PR File Line flagged
#2183 useNotificationStream.test.tsx function createSseResponse(signal?: AbortSignal): MockSse {
#2183 useNotificationStream.test.tsx async function flushStreamSetup() {
#2183 useNotificationStream.test.tsx function mockSSEStreams() {
#2183 useNotificationStream.test.tsx function seededQueryClient() {
#2182 LoanTimeline.test.tsx function makeEvent(overrides: Partial = {}): LoanEvent {
#2182 LoanTimeline.test.tsx function items() {

Malware suspicion is false positive