Skip to content

test: add unit tests for money, strkey, retention, supervise (#436 #437 #438 #439) - #578

Merged
Markadrian6399 merged 2 commits into
StellarGateLabs:mainfrom
Young850:feat/436-439-unit-test-coverage
Aug 27, 2026
Merged

test: add unit tests for money, strkey, retention, supervise (#436 #437 #438 #439)#578
Markadrian6399 merged 2 commits into
StellarGateLabs:mainfrom
Young850:feat/436-439-unit-test-coverage

Conversation

@Young850

Copy link
Copy Markdown

What and why

Closes #436
Closes #437
Closes #438
Closes #439

How it was verified

  • cargo test passes
  • cargo fmt --check and cargo clippy --all-targets -- -D warnings pass
  • New behaviour is covered by a test that fails without this change

Risk

  • Adds or changes a database migration (forward-only — see DEPLOYMENT.md)
  • Changes a public API response shape or error code
  • Touches payment verification, settlement, or amount handling
  • Touches authentication, the SSRF guard, or webhook signing
  • Adds or updates a dependency (cargo deny check all passes)
  • Requires a config/env change to deploy (documented in README + .env.example)

Notes for the reviewer

@drips-wave

drips-wave Bot commented Aug 27, 2026

Copy link
Copy Markdown

@Young850 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Markadrian6399
Markadrian6399 merged commit 6574cc8 into StellarGateLabs:main Aug 27, 2026
2 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants