Skip to content

test: cover multi-offering payment-token independence - #866

Merged
thlpkee20-wq merged 2 commits into
RevoraOrg:masterfrom
Emmyranky:feat/multi-token-offering-tests
Aug 31, 2026
Merged

test: cover multi-offering payment-token independence#866
thlpkee20-wq merged 2 commits into
RevoraOrg:masterfrom
Emmyranky:feat/multi-token-offering-tests

Conversation

@Emmyranky

Copy link
Copy Markdown

I have implemented the multi-token offering independence tests as requested.

Summary of Changes:

  1. New Test Module: Created src/test_multi_token_independence.rs which implements the test scenario:
    • Registers two offerings (A and B) under the same issuer/namespace with different token addresses.
    • Deposits different payment tokens (X and Y) to A and B respectively.
    • Verifies get_payment_token returns the correct token for each offering independently.
    • Asserts that a cross-deposit (depositing Token Y into Offering A) fails with PaymentTokenMismatch.
  2. Module Registration: Added #[cfg(test)] mod test_multi_token_independence; to src/lib.rs to register the new
    tests.
  3. Branching: Created a new branch feat/multi-token-offering-tests.

Verification:

  • The implementation follows the established integration test pattern used in the repository.
  • The test logic directly asserts the requirements for independence and cross-talk prevention.
  • Compilation and test execution were not possible locally due to missing C++ build tools, but the code adheres to
    the project structure, conventions, and patterns, and is expected to pass CI as per the project's documentation.

Closes #841

@drips-wave

drips-wave Bot commented Aug 30, 2026

Copy link
Copy Markdown

@Emmyranky 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

@thlpkee20-wq
thlpkee20-wq merged commit d168b53 into RevoraOrg:master Aug 31, 2026
2 of 3 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

Development

Successfully merging this pull request may close these issues.

Add multi-token offering support tests covering distinct payment tokens across offerings under the same issuer namespace

3 participants