Skip to content

fix: re-enable atomic swap tests (benchmarks, mutation, snapshot, chaos) - #943

Open
villadel wants to merge 1 commit into
AtomicIP:mainfrom
villadel:fix/re-enable-atomic-swap-tests
Open

fix: re-enable atomic swap tests (benchmarks, mutation, snapshot, chaos)#943
villadel wants to merge 1 commit into
AtomicIP:mainfrom
villadel:fix/re-enable-atomic-swap-tests

Conversation

@villadel

Copy link
Copy Markdown
Re-enables and fixes compile errors in four disabled test modules for the Atomic Swap contract:

- #819 `benchmarks.rs`: Fixed compile errors and re-enabled module. CPU budget tests cover `initiate_swap`, `accept_swap`, `reveal_key`, `cancel_swap`, and `get_swap`.
- #820 `mutation_tests.rs`: Re-enabled module and populated with real mutation cases covering zero/negative price, duplicate IP, auth checks, batch size limits, and expiry boundaries.
- #821 `snapshot_tests.rs`: Re-enabled module with fixed-input snapshots for `SwapRecord` (Pending/Accepted/Completed), `ProtocolConfig`, swap history, and `ArbitratorCommittee`.
- #822 `upgrade_chaos_tests.rs`: Fixed merge-conflict compile errors (`ContractError` variant names) and added tests for concurrent upgrades and mid-swap upgrade state preservation.

Also updated `.gitignore` with additional unnecessary file patterns (`*.log`, `*.tmp`, `*.pid`).

All commits authored by `villadel <marschkayla@gmail.com>`.

close #819
close #820
close #821
close #822

@drips-wave

drips-wave Bot commented Aug 28, 2026

Copy link
Copy Markdown

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant