fix: Kraiken.setStakingPool() allows stakingPool == liquidityManager with no guard (#935)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
openhands 2026-03-18 08:09:43 +00:00
parent 256377ef1f
commit f3238a9685
2 changed files with 10 additions and 0 deletions

View file

@ -207,6 +207,14 @@ contract KraikenTest is Test {
kraiken.setStakingPool(makeAddr("anotherStakingPool"));
}
function testSetStakingPoolRejectsLiquidityManager() public {
Kraiken freshKraiken = new Kraiken("KRAIKEN", "KRK");
address lm = makeAddr("liquidityManager");
freshKraiken.setLiquidityManager(lm);
vm.expectRevert(Kraiken.InvalidAddress.selector);
freshKraiken.setStakingPool(lm);
}
function testSetStakingPoolOnlyDeployer() public {
Kraiken freshKraiken = new Kraiken("KRAIKEN", "KRK");
address nonDeployer = makeAddr("nonDeployer");