chore: move defunct test to its own file

Author: martinkrung

contracts/RewardManager.vy

@@ -57,30 +57,6 @@ def send_reward_token(_reward_receiver: address, _amount: uint256, _epoch: uint2
         Gauge(_reward_receiver).deposit_reward_token(self.reward_token, _amount, _epoch)
 
 
-@external
-def deposit_send_reward_token(_reward_receiver: address, _amount: uint256, _epoch: uint256 = WEEK):
-    """
-    @notice deposit reward token from sender to contract and then forwarded to gauge
-    @param _reward_receiver gauges to receiver reward
-    @param _amount amount of reward token to deposit
-    @param _epoch The duration the rewards are distributed across in seconds. Between 3 days and a year, week by default
-    """
-    assert msg.sender in self.managers, 'only reward managers can call this function'
-    assert _reward_receiver in self.reward_receivers, 'only reward receiver which are allowed'
-    assert 3 * WEEK / 7 <= _epoch and _epoch <= WEEK * 4 * 12, 'epoch duration must be between 3 days and a year'
-
-    # deposit reward token from sender to this contract
-    assert ERC20(self.reward_token).transferFrom(msg.sender, self, _amount, default_return_value=True)
-
-    assert ERC20(self.reward_token).approve(_reward_receiver, _amount, default_return_value=True)
-
-    # legacy gauges have no epoch parameter
-    # new deposit_reward_token has epoch parameter default to WEEK
-    if _epoch == WEEK:
-        LegacyGauge(_reward_receiver).deposit_reward_token(self.reward_token, _amount)
-    else:
-        Gauge(_reward_receiver).deposit_reward_token(self.reward_token, _amount, _epoch)
-
 @external
 def recover_token(_token: address, _amount: uint256):
     """

tests/RewardManager/test_deposit_reward.py

@@ -44,23 +44,6 @@ def test_reward_manager_send_reward_token_epoch(alice, bob, diana, reward_token,
     assert amount == balance_recoverd
     assert amount == reward_manager_balance
 
-def test_reward_manager_deposit_simple(bob, recovery_gauge, reward_manager):
-    reward_manager.deposit_send_reward_token(recovery_gauge, 10 ** 18,  sender=bob)
-
-def test_reward_manager_deposit_token(alice, bob, diana, reward_token, recovery_gauge, reward_manager):
-    amount = 10 ** 17
-    reward_manager.deposit_send_reward_token(recovery_gauge, amount,  sender=bob)
-    assert recovery_gauge.recover_token(sender=alice)
-    balance_recoverd = reward_token.balanceOf(diana, sender=alice)
-    assert amount == balance_recoverd
-
-def test_reward_manager_deposit_epoch(alice, bob, diana, reward_token, recovery_gauge, reward_manager):
-    amount = 10 ** 17
-    reward_manager.deposit_send_reward_token(recovery_gauge, amount, 8 * DAY, sender=bob)
-    assert recovery_gauge.recover_token(sender=alice)
-    balance_recoverd = reward_token.balanceOf(diana, sender=alice)
-    assert amount == balance_recoverd
-
 def test_reward_manager_sent_revert(alice, recovery_gauge, reward_manager):
     with ape.reverts("only reward managers can call this function"):
         reward_manager.send_reward_token(recovery_gauge, 10 ** 18, sender=alice)
@@ -73,18 +56,6 @@ def test_reward_manager_epoch_revert_too_long(bob, recovery_gauge, reward_manage
     with ape.reverts("epoch duration must be between 3 days and a year"):
         reward_manager.send_reward_token(recovery_gauge, 10 ** 18, 53 * WEEK, sender=bob)
 
-def test_reward_manager_deposit_sent_revert(alice, recovery_gauge, reward_manager):
-    with ape.reverts("only reward managers can call this function"):
-        reward_manager.deposit_send_reward_token(recovery_gauge, 10 ** 18, sender=alice)
-
-def test_reward_manager_deposit_epoch_revert_too_short(bob, recovery_gauge, reward_manager):
-    with ape.reverts("epoch duration must be between 3 days and a year"):
-        reward_manager.deposit_send_reward_token(recovery_gauge, 10 ** 18, DAY, sender=bob)
-
-def test_reward_manager_deposit_epoch_revert_too_long(bob, recovery_gauge, reward_manager):
-    with ape.reverts("epoch duration must be between 3 days and a year"):
-        reward_manager.deposit_send_reward_token(recovery_gauge, 10 ** 18, 53 * WEEK, sender=bob)
-
 def test_recover_token(bob, charlie, diana, lost_token, reward_manager):
     amount = 10 ** 18
     lost_token.transferFrom(charlie, reward_manager, amount, sender=charlie)