Skip to content

docs: add screen-level test matrix for wallet flows - #511

Merged
El-swaggerito merged 1 commit into
Axionvera:mainfrom
habnark:docs/443-screen-test-matrix
Jul 29, 2026
Merged

docs: add screen-level test matrix for wallet flows#511
El-swaggerito merged 1 commit into
Axionvera:mainfrom
habnark:docs/443-screen-test-matrix

Conversation

@habnark

@habnark habnark commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Closes #443

Contributors couldn't tell which screens require tests or what "done" looks like for visual verification, leading to under-tested mobile PRs (per the issue).

  • Add docs/screen-test-matrix.md: every screen under app/ (verified against the actual file tree, not the older/incomplete docs/screen-inventory.md), grouped into the categories the issue calls out — onboarding (welcome/create/import/success), balance, send, receive/QR, activity, address book, vault, settings — plus diagnostics.
  • Each row lists required test types (component/store/flow/visual), cross-references the actual existing test file(s) in tests/ and tests/ (verified by reading them, not guessing from filenames), and a coverage status. This surfaced real gaps: onboarding has zero automated coverage, Settings has zero, and a few screens (Receive, Vault Detail, Diagnostics) only have logic-level tests rather than screen-level ones.
  • Visual verification expectations per section, cross-linking the existing UI State Catalogue, Accessibility Checklist, and Design System docs rather than duplicating them.
  • Link the matrix from README.md and from CONTRIBUTING.md's testing section.

Summary

Closes #

Test Plan and Evidence

Self-Assessment

Complete the
Contributor Self-Assessment before
requesting review. Include concise evidence below; use
Not applicable — <reason> instead of leaving a required area unexplained.

  • Scope: The change matches the linked issue and contains no unrelated
    or unfinished work.
  • Tests: Appropriate automated and manual evidence is included above,
    or a no-test justification is provided.
  • CI: Relevant local checks pass and required CI checks pass on the
    latest commit.
  • Documentation: Affected documentation, comments, examples,
    screenshots, and links are updated, or no update is needed.
  • Known limitations: Limitations, assumptions, risks, skipped checks,
    and follow-up work are disclosed below.
  • Acceptance criteria: Each issue criterion is mapped to implementation
    or verification evidence below.

CI Status

Documentation

Known Limitations

Acceptance Criteria

Acceptance criterion Implementation or evidence Status
Pass / Not applicable

Screenshots or Recordings

Closes Axionvera#443

Contributors couldn't tell which screens require tests or what "done"
looks like for visual verification, leading to under-tested mobile
PRs (per the issue).

- Add docs/screen-test-matrix.md: every screen under app/ (verified
  against the actual file tree, not the older/incomplete
  docs/screen-inventory.md), grouped into the categories the issue
  calls out — onboarding (welcome/create/import/success), balance,
  send, receive/QR, activity, address book, vault, settings — plus
  diagnostics.
- Each row lists required test types (component/store/flow/visual),
  cross-references the actual existing test file(s) in __tests__/ and
  tests/ (verified by reading them, not guessing from filenames), and
  a coverage status. This surfaced real gaps: onboarding has zero
  automated coverage, Settings has zero, and a few screens (Receive,
  Vault Detail, Diagnostics) only have logic-level tests rather than
  screen-level ones.
- Visual verification expectations per section, cross-linking the
  existing UI State Catalogue, Accessibility Checklist, and Design
  System docs rather than duplicating them.
- Link the matrix from README.md and from CONTRIBUTING.md's testing
  section.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@El-swaggerito
El-swaggerito merged commit 250d52e into Axionvera:main Jul 29, 2026
1 check passed
@grantfox-oss grantfox-oss Bot mentioned this pull request Jul 29, 2026
6 tasks
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.

Add mobile screen test matrix

2 participants