diff --git a/contracts/test/_fixture.js b/contracts/test/_fixture.js index a3c4b13836..2ca9e71680 100644 --- a/contracts/test/_fixture.js +++ b/contracts/test/_fixture.js @@ -1454,10 +1454,6 @@ async function nativeStakingSSVStrategyFixture() { await nativeStakingSSVStrategy .connect(sGovernor) .setAccountingGovernor(governorAddr); - - await nativeStakingSSVStrategy - .connect(sGovernor) - .setStrategist(strategist.address); } return fixture; diff --git a/contracts/test/strategies/nativeSSVStaking.js b/contracts/test/strategies/nativeSSVStaking.js index 2caacbebb9..910a5d8ba0 100644 --- a/contracts/test/strategies/nativeSSVStaking.js +++ b/contracts/test/strategies/nativeSSVStaking.js @@ -186,16 +186,6 @@ describe("Unit test: Native SSV Staking Strategy", function () { ).to.be.revertedWith("Caller is not the Governor"); }); - it("Only governor can change the strategist", async () => { - const { nativeStakingSSVStrategy, strategist } = fixture; - - await expect( - nativeStakingSSVStrategy - .connect(strategist) - .setStrategist(strategist.address) - ).to.be.revertedWith("Caller is not the Governor"); - }); - it("Change the accounting governor", async () => { const { nativeStakingSSVStrategy, governor, strategist } = fixture; @@ -217,26 +207,6 @@ describe("Unit test: Native SSV Staking Strategy", function () { strategist.address ); }); - - it("Change the strategist", async () => { - const { nativeStakingSSVStrategy, governor, strategist } = fixture; - - const tx = await nativeStakingSSVStrategy - .connect(governor) - .setStrategist(governor.address); - - const events = (await tx.wait()).events || []; - const strategistAddressChanged = events.find( - (e) => e.event === "StrategistAddressChanged" - ); - - expect(strategistAddressChanged).to.not.be.undefined; - expect(strategistAddressChanged.event).to.equal( - "StrategistAddressChanged" - ); - expect(strategistAddressChanged.args[0]).to.equal(strategist.address); - expect(strategistAddressChanged.args[1]).to.equal(governor.address); - }); }); describe("Accounting", function () { @@ -345,7 +315,7 @@ describe("Unit test: Native SSV Staking Strategy", function () { } const WithdrawnEvent = events.find( - (e) => e.event === "AccuntingFullyWithdrawnValidator" + (e) => e.event === "AccountingFullyWithdrawnValidator" ); if (expectedValidatorsFullWithdrawals > 0) { expect(WithdrawnEvent).to.not.be.undefined; @@ -374,7 +344,7 @@ describe("Unit test: Native SSV Staking Strategy", function () { } const SlashEvent = events.find( - (e) => e.event === "AccuntingValidatorSlashed" + (e) => e.event === "AccountingValidatorSlashed" ); if (slashDetected) { expect(SlashEvent).to.not.be.undefined;