Skip to content

Test gap: Navbar.tsx and ConnectButton.tsx have no test coverage #326

Description

@Jaydbrown

components/Navbar.tsx has real logic worth regression-testing: closing the mobile menu on route change, closing on Escape, and the documented iOS-Safari-specific backdrop-click-to-close workaround (issue #143). components/ConnectButton.tsx has its own real logic: a 20s connect-timeout safety net that force-clears the "Connecting…" state, and error-message rendering from a thrown connect() rejection. Neither has a test file. contexts/WalletContext.test.tsx covers the underlying wallet context, but the UI-level timeout-recovery and keyboard/backdrop-dismissal behavior in these two components is untested and has already been the subject of at least two numbered bug fixes (#143, and the connect-timeout safety net) — exactly the kind of previously-buggy logic that benefits most from a regression test.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions