TokenStaking: removes refreshKeepStakeOwner

Author: vzotova

contracts/staking/IStaking.sol

@@ -70,12 +70,6 @@ interface IStaking {
         address authorizer
     ) external;
 
-    /// @notice Refresh Keep stake owner. Can be called only by the old owner
-    ///         or their staking provider.
-    /// @dev The staking provider in T staking contract is the legacy KEEP
-    ///      staking contract operator.
-    function refreshKeepStakeOwner(address stakingProvider) external;
-
     /// @notice Allows the Governance to set the minimum required stake amount.
     ///         This amount is required to protect against griefing the staking
     ///         contract and individual applications are allowed to require

contracts/staking/TokenStaking.sol

@@ -408,28 +408,6 @@ contract TokenStaking is Initializable, IStaking, Checkpoints {
         );
     }
 
-    /// @notice Refresh Keep stake owner. Can be called only by the old owner
-    ///         or their staking provider.
-    /// @dev The staking provider in T staking contract is the legacy KEEP
-    ///      staking contract operator.
-    function refreshKeepStakeOwner(address stakingProvider)
-        external
-        override
-        onlyOwnerOrStakingProvider(stakingProvider)
-    {
-        StakingProviderInfo storage stakingProviderStruct = stakingProviders[
-            stakingProvider
-        ];
-        address newOwner = keepStake.resolveOwner(stakingProvider);
-
-        emit OwnerRefreshed(
-            stakingProvider,
-            stakingProviderStruct.owner,
-            newOwner
-        );
-        stakingProviderStruct.owner = newOwner;
-    }
-
     /// @notice Allows the Governance to set the minimum required stake amount.
     ///         This amount is required to protect against griefing the staking
     ///         contract and individual applications are allowed to require

docs/rfc-1-staking-contract.adoc

@@ -307,10 +307,6 @@ additionally appointing beneficiary and authorizer roles. Caches the amount
 staked in NU staking contract. Can be called only by the original delegation
 owner.
 
-==== `refreshKeepStakeOwner(address stakingProvider) external onlyOwnerOf(stakingProvider)`
-
-Refresh Keep stake owner. Can be called only by the old owner.
-
 ==== `setMinimumStakeAmount(uint96 amount) external onlyGovernance`
 
 Allows the governance to set the minimum required stake amount. This amount is

test/staking/TokenStaking.test.js

@@ -852,152 +852,6 @@ describe("TokenStaking", () => {
     })
   })
 
-  describe("refreshKeepManagedGrantOwner", () => {
-    context("when staking provider has no delegated stake", () => {
-      it("should revert", async () => {
-        await expect(
-          tokenStaking
-            .connect(stakingProvider)
-            .refreshKeepStakeOwner(stakingProvider.address)
-        ).to.be.revertedWith("Not owner or provider")
-      })
-    })
-
-    context("when caller is neither old owner nor staking provider", () => {
-      it("should revert", async () => {
-        await tToken
-          .connect(staker)
-          .approve(tokenStaking.address, initialStakerBalance)
-        await tokenStaking
-          .connect(staker)
-          .stake(
-            stakingProvider.address,
-            beneficiary.address,
-            authorizer.address,
-            initialStakerBalance
-          )
-        await expect(
-          tokenStaking
-            .connect(authorizer)
-            .refreshKeepStakeOwner(stakingProvider.address)
-        ).to.be.revertedWith("Not owner or provider")
-      })
-    })
-
-    const contextRefreshKeepStakeOwner = (getCaller) => {
-      context("when grantee was not changed", () => {
-        let tx
-
-        beforeEach(async () => {
-          const createdAt = 1
-          await keepStakingMock.setOperator(
-            stakingProvider.address,
-            staker.address,
-            beneficiary.address,
-            authorizer.address,
-            createdAt,
-            0,
-            initialStakerBalance
-          )
-          await keepStakingMock.setEligibility(
-            stakingProvider.address,
-            tokenStaking.address,
-            true
-          )
-          await tokenStaking.stakeKeep(stakingProvider.address)
-
-          tx = await tokenStaking
-            .connect(getCaller())
-            .refreshKeepStakeOwner(stakingProvider.address)
-        })
-
-        it("should not update owner", async () => {
-          expect(
-            await tokenStaking.rolesOf(stakingProvider.address)
-          ).to.deep.equal([
-            staker.address,
-            beneficiary.address,
-            authorizer.address,
-          ])
-        })
-
-        it("should emit OwnerRefreshed", async () => {
-          await expect(tx)
-            .to.emit(tokenStaking, "OwnerRefreshed")
-            .withArgs(stakingProvider.address, staker.address, staker.address)
-        })
-      })
-
-      context("when grantee was changed", () => {
-        let tx
-
-        beforeEach(async () => {
-          const createdAt = 1
-          await keepStakingMock.setOperator(
-            stakingProvider.address,
-            otherStaker.address,
-            beneficiary.address,
-            authorizer.address,
-            createdAt,
-            0,
-            initialStakerBalance
-          )
-          await keepStakingMock.setEligibility(
-            stakingProvider.address,
-            tokenStaking.address,
-            true
-          )
-          await tokenStaking.stakeKeep(stakingProvider.address)
-
-          await keepStakingMock.setOperator(
-            stakingProvider.address,
-            staker.address,
-            beneficiary.address,
-            authorizer.address,
-            createdAt,
-            0,
-            initialStakerBalance
-          )
-          tx = await tokenStaking
-            .connect(otherStaker)
-            .refreshKeepStakeOwner(stakingProvider.address)
-        })
-
-        it("should update owner", async () => {
-          expect(
-            await tokenStaking.rolesOf(stakingProvider.address)
-          ).to.deep.equal([
-            staker.address,
-            beneficiary.address,
-            authorizer.address,
-          ])
-        })
-
-        it("should emit OwnerRefreshed", async () => {
-          await expect(tx)
-            .to.emit(tokenStaking, "OwnerRefreshed")
-            .withArgs(
-              stakingProvider.address,
-              otherStaker.address,
-              staker.address
-            )
-        })
-      })
-    }
-
-    context("when caller is the old owner", () => {
-      contextRefreshKeepStakeOwner(() => {
-        return staker
-      })
-    })
-
-    context("when caller is the staking provider", () => {
-      contextRefreshKeepStakeOwner(() => {
-        return stakingProvider
-      })
-    })
-  })
-
   describe("approveApplication", () => {
     context("when caller is not the governance", () => {
       it("should revert", async () => {