1.7 KiB
Documentation and Test Fixes
Date: 2025-12-24
Status: ✅ Fixed
✅ Fixed Issues
1. Documentation Tag Mismatches
The @return tags in documentation didn't match the actual return parameter names after renaming.
TransactionMirror.sol
Error: @return tx but parameter is mirroredTx
Fix: Updated to @return mirroredTx
OraclePriceFeed.sol
Error: @return needsUpdate but parameter is updateNeeded
Fix: Updated to @return updateNeeded
PriceFeedKeeper.sol
Error: @return needsUpdate but parameter is updateNeeded
Fix: Updated to @return updateNeeded
2. Test Event Reference Error
Error: Member "ValueTransferDeclared" not found in type(contract CompliantUSDT)
Issue: The event ValueTransferDeclared is defined in LegallyCompliantBase, not directly in CompliantUSDT. Even though CompliantUSDT inherits from it, the test needs to reference it through the base contract.
Fix:
- Added import for
LegallyCompliantBase - Changed event reference from
CompliantUSDT.ValueTransferDeclaredtoLegallyCompliantBase.ValueTransferDeclared
// Before
emit CompliantUSDT.ValueTransferDeclared(...);
// After
emit LegallyCompliantBase.ValueTransferDeclared(...);
📋 Files Modified
contracts/mirror/TransactionMirror.sol- Fixed@returntagcontracts/reserve/OraclePriceFeed.sol- Fixed@returntagcontracts/reserve/PriceFeedKeeper.sol- Fixed@returntagtest/compliance/CompliantUSDTTest.t.sol- Fixed event reference and added import
✅ Verification
All documentation tags now match their return parameters, and the test should compile successfully.
Last Updated: 2025-12-24