Overview
contracts/assetsup/src/tests/ has both an original and a _new suffixed file for several areas — insurance.rs + insurance_new.rs, tokenization.rs + tokenization_new.rs, plus standalone dividends_new.rs, voting_new.rs, transfer_restrictions_new.rs, and detokenization_new.rs with no non-_new counterpart. This naming pattern strongly suggests a module was re-implemented in a later pass without the older test file being reconciled or removed, which risks either duplicate coverage (wasted CI time) or, worse, the old file silently testing stale/removed behavior.
Tasks
Acceptance Criteria
Notes for Contributors
Requires Rust + Soroban. Workspace: contracts/ (soroban-sdk 23, members: assetsup, contrib, multisig-wallet, multisig-transfer, asset-maintenance). Run cargo test --all, cargo fmt --all -- --check, and cargo clippy --all-targets --all-features -- -D warnings before opening a PR — CI enforces all three. Comment below to be assigned.
Overview
contracts/assetsup/src/tests/has both an original and a_newsuffixed file for several areas —insurance.rs+insurance_new.rs,tokenization.rs+tokenization_new.rs, plus standalonedividends_new.rs,voting_new.rs,transfer_restrictions_new.rs, anddetokenization_new.rswith no non-_newcounterpart. This naming pattern strongly suggests a module was re-implemented in a later pass without the older test file being reconciled or removed, which risks either duplicate coverage (wasted CI time) or, worse, the old file silently testing stale/removed behavior.Tasks
X.rs/X_new.rspair, diff them: ifX_new.rssupersedesX.rs, removeX.rs(or merge any test case it has thatX_new.rsis missing) and drop the_newsuffix for clarity._new-suffixed files with no pair, just drop the suffix if there's no naming collision reason for it to still be there.Acceptance Criteria
assetsup/src/tests/carries a_newsuffix implying an un-reconciled duplicate, and no coverage was silently testing removed behavior.Notes for Contributors
Requires Rust + Soroban. Workspace:
contracts/(soroban-sdk 23, members: assetsup, contrib, multisig-wallet, multisig-transfer, asset-maintenance). Runcargo test --all,cargo fmt --all -- --check, andcargo clippy --all-targets --all-features -- -D warningsbefore opening a PR — CI enforces all three. Comment below to be assigned.