Skip to content

test(profile): add page.test.tsx covering loading, disconnected, and connected states (fixes #240) - #516

Open
ZacLou wants to merge 1 commit into
conduit-protocol:mainfrom
ZacLou:fix/issue-240-profile-tests
Open

test(profile): add page.test.tsx covering loading, disconnected, and connected states (fixes #240)#516
ZacLou wants to merge 1 commit into
conduit-protocol:mainfrom
ZacLou:fix/issue-240-profile-tests

Conversation

@ZacLou

@ZacLou ZacLou commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds the missing test file for app/profile/page.tsx, matching the coverage present in sibling route directories (app/dashboard/__tests__/page.test.tsx, app/streams/__tests__/page.test.tsx).

Coverage

  • loading — renders skeleton pulse while connecting is true
  • disconnected — shows connect prompt and link to home
  • connected — displays wallet provider, shortened public key, CopyHashButton, and quick links
  • Edge cases: null publicKey despite connected=true; null walletName falls back to Unknown

Acceptance Criteria

  • Test file added with coverage comparable to sibling route directories
  • No behaviour change to the page component itself

@ZacLou
ZacLou force-pushed the fix/issue-240-profile-tests branch 4 times, most recently from d328295 to 4cc7d0c Compare September 4, 2026 04:17
@ZacLou
ZacLou force-pushed the fix/issue-240-profile-tests branch from 4cc7d0c to b4ec9e8 Compare September 4, 2026 04:22
@ZacLou

ZacLou commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

Hi @Jaydbrown — this PR adds the missing profile page tests (fixes #240). All CI checks (lint, typecheck, unit tests, build) are now green. Could you take a look when you have a moment? Thanks!

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.

1 participant