Skip to content

Test gap: app/stream/[id]/page.tsx has no test coverage #321

Description

@Jaydbrown

app/stream/[id]/page.tsx is one of the most logic-heavy pages in the app — it derives stream status (deriveStatus), guards against out-of-order async responses via a loadSeq ref (isCurrent()), computes isSender/isRecipient from the connected wallet, and conditionally renders four different states (disconnected/loading/error/loaded) — but has no corresponding test file. app/create/__tests__/page.test.tsx, app/dashboard/__tests__/page.test.tsx (plus page.race.test.tsx specifically for its race-condition guard), and app/streams/__tests__/page.test.tsx all have coverage for comparable route-level logic; app/stream/[id]/page.tsx — arguably the page with the most non-trivial state machine of any of them — has none. Given app/dashboard/__tests__/page.race.test.tsx exists specifically to test dashboard's request-race handling, this page's own (correctly-implemented) loadSeq/isCurrent() race guard is exactly the kind of logic that file's sibling pages already consider test-worthy.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Stellar WaveIssues in the Stellar wave programbugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions