test: fix transfer ownership fuzzer

nayms · Dec 24, 2024 · 449ca12 · 449ca12
1 parent 0fd9e46
commit 449ca12
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/test/T03NaymsOwnership.t.sol b/test/T03NaymsOwnership.t.sol
@@ -48,10 +48,10 @@ contract T03NaymsOwnershipTest is D03ProtocolDefaults, MockAccounts {
     function testFuzz_TransferOwnership(address newOwner, address notSysAdmin, address anotherSysAdmin) public {
         vm.assume(newOwner != anotherSysAdmin && newOwner != account0 && newOwner != address(0) && anotherSysAdmin != address(0));
         vm.assume(anotherSysAdmin != address(0));
+        vm.assume(!nayms.isInGroup(newOwner._getIdForAddress(), systemContext, LC.GROUP_SYSTEM_MANAGERS));
 
-        bytes32 notSysAdminId = LibHelpers._getIdForAddress(address(notSysAdmin));
         // note: for this test, assume that the notSysAdmin address is not a system admin
-        vm.assume(!nayms.isInGroup(notSysAdminId, systemContext, LC.GROUP_SYSTEM_ADMINS));
+        vm.assume(!nayms.isInGroup(notSysAdmin._getIdForAddress(), systemContext, LC.GROUP_SYSTEM_ADMINS));
 
         vm.label(newOwner, "newOwner");
         vm.label(notSysAdmin, "notSysAdmin");