Skip to content

test: add shared multi-user contract harness - #49

Open
sa-salami wants to merge 1 commit into
Lokt-In:stagingfrom
sa-salami:fix/issue-36
Open

sa-salami wants to merge 1 commit into
Lokt-In:stagingfrom
sa-salami:fix/issue-36

Conversation

@sa-salami

Copy link
Copy Markdown

Summary

  • add a shared test_utils crate for token setup, funded users, deterministic ledger time, minting, and solvency assertions
  • migrate the five contract test suites to the shared token helpers
  • add a multi-user solvency example and the missing extern crate std declaration in the plans tests

Verification

  • cargo test --workspace — 58 tests passed

Fixes #36

@drips-wave

drips-wave Bot commented Sep 27, 2026

Copy link
Copy Markdown

@sa-salami 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

@sa-salami

Copy link
Copy Markdown
Author

@Lokt-In @sa-salami Fixed in commit e9facbd225790009b675c8e90b19deb428100482. Added the shared five-contract test harness, migrated token helpers, added deterministic ledger/solvency helpers, and included a multi-user solvency example. Verification: cargo test --workspace passed all 58 tests. The PR body references Fixes #36; please review and merge when CI completes.

@sa-salami

Copy link
Copy Markdown
Author

Hi @Mackenzie-OO7 👋 Friendly reminder that this PR for #36 is ready for review — there are no merge conflicts and no outstanding change requests. Could you take a look and merge it when you get a chance? Happy to make any changes you'd like. Thank you!

This branch has not been deployed

No deployments
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.

test(contracts): add multi-user test harness

2 participants