Skip to content

feat: Initial Test Suite for Chat App#72

Closed
kwsantiago wants to merge 3 commits into
Routstr:mainfrom
kwsantiago:kwsantiago/61-comprehensive-tests
Closed

feat: Initial Test Suite for Chat App#72
kwsantiago wants to merge 3 commits into
Routstr:mainfrom
kwsantiago:kwsantiago/61-comprehensive-tests

Conversation

@kwsantiago
Copy link
Copy Markdown
Contributor

@kwsantiago kwsantiago commented Aug 27, 2025

Summary

Basic test infrastructure done but missing core requirements from #61

Completed

  • CI/CD pipeline with GitHub Actions
  • Vitest setup with 53 passing tests
  • Basic Cashu mint API testing with FakeWallet
  • Component unit tests
  • Error handling coverage

Still Needed

  • Real Lightning node integration (currently using FakeWallet)
  • Mock Nostr relay for chat testing
  • Multi-user payment flows
  • Message routing and failure scenarios
  • State persistence tests
  • Restore deleted invoice tests and regtest scripts

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 27, 2025

@kwsantiago is attempting to deploy a commit to the evanyang's Team Team on Vercel.

A member of the Team first needs to authorize it.

@kwsantiago kwsantiago force-pushed the kwsantiago/61-comprehensive-tests branch from 14cd209 to 5d4c93a Compare August 27, 2025 14:03
@kwsantiago kwsantiago force-pushed the kwsantiago/61-comprehensive-tests branch from 524cf71 to daf2fec Compare August 27, 2025 14:17
@kwsantiago kwsantiago marked this pull request as draft October 9, 2025 21:34
@sh1ftred
Copy link
Copy Markdown
Contributor

have added test via msw. closing this as it's pretty old and outdated now.

@sh1ftred sh1ftred closed this Dec 15, 2025
@kwsantiago kwsantiago deleted the kwsantiago/61-comprehensive-tests branch December 15, 2025 12:45
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.

2 participants