Skip to content

Commit 6381a44

Browse files
committed
fix: add additional owner check
1 parent 597983c commit 6381a44

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

e2e/multisig-migration.test.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -167,9 +167,14 @@ describe("Should change LBC owner to the multisig.ts", function () {
167167
expect(newOwner.toLowerCase()).to.equal(multisigAddress.toLowerCase());
168168

169169
// Verify old owner can no longer perform owner functions
170+
const adminAddress = await upgrades.erc1967.getAdminAddress(proxyAddress);
171+
const adminContract = await ethers.getContractAt(
172+
"LiquidityBridgeContractAdmin",
173+
adminAddress
174+
);
170175
await expect(
171-
contract.connect(impersonatedSigner).setProviderStatus(1, false)
172-
).to.be.revertedWith("LBC005");
176+
adminContract.upgrade(proxyAddress, multisigAddress)
177+
).to.revertedWith("Ownable: caller is not the owner");
173178

174179
console.info(
175180
`Ownership of LiquidityBridgeContract proxy changed to multisig in ${deploymentNetwork}`

0 commit comments

Comments
 (0)