Skip to content

[codex] Broaden nr-app test coverage#243

Closed
guaka wants to merge 1 commit into
mainfrom
codex/broaden-nr-app-test-coverage
Closed

[codex] Broaden nr-app test coverage#243
guaka wants to merge 1 commit into
mainfrom
codex/broaden-nr-app-test-coverage

Conversation

@guaka

@guaka guaka commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Broaden nr-app coverage beyond the browser-specific tests by adding shared app test infrastructure and focused Jest coverage for high-value general app workflows.

What Changed

  • Added shared renderWithProviders() and a configurable real Redux store factory for screen/component tests.
  • Added reusable fixtures for Nostr events, map notes, Trustroots profile events, plus codes, and relay data.
  • Added workflow/component tests for app routing, welcome/onboarding states, map list tabs, notification subscribe/unsubscribe, and add-note publish validation.
  • Added reducer/selector tests for events, keystore, notifications, and settings.
  • Added coverage scripts and ignored generated coverage output.

Validation

  • pnpm --filter nr-app exec jest --watchman=false passed: 25 suites, 106 tests.
  • pnpm --filter nr-app exec jest --watchman=false --coverage passed.
  • pnpm test:app:coverage passed.
  • Scoped lint for the new/changed files passed.
  • Full pnpm --filter nr-app lint still reports pre-existing unrelated issues in current app files.

@guaka

guaka commented Jun 14, 2026

Copy link
Copy Markdown
Contributor Author

Folded the useful test coverage from this draft into #227 and updated nr-app-test-suite instead.

@guaka guaka closed this Jun 14, 2026
@guaka guaka deleted the codex/broaden-nr-app-test-coverage branch June 14, 2026 11:52
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