Commit eb0e1335 authored by Maurelian's avatar Maurelian

test(ctb): Add test for incomplete emptying below minOwners

parent 33c7c8b1
......@@ -214,6 +214,24 @@ contract LivenessModule_RemoveOwners_TestFail is LivenessModule_TestInit {
/// @dev Tests if remove owners reverts if it removes too many owners without removing all of them
function test_removeOwners_belowMinButNotEmptied_reverts() external {
// Remove all but one owner
uint256 numOwners = safeInstance.owners.length - 2;
address[] memory ownersToRemove = new address[](numOwners);
for (uint256 i = 0; i < numOwners; i++) {
ownersToRemove[i] = safeInstance.owners[i];
}
address[] memory prevOwners = safeInstance.getPrevOwners(ownersToRemove);
_warpPastLivenessInterval();
vm.expectRevert(
"LivenessModule: must remove all owners and transfer to fallback owner if numOwners < minOwners"
);
livenessModule.removeOwners(prevOwners, ownersToRemove);
}
/// @dev Tests if remove owners reverts if it removes too many owners transferring to the fallback owner
function test_removeOwners_belowEmptiedButNotFallback_reverts() external {
// Remove all but one owner
uint256 numOwners = safeInstance.owners.length - 1;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment