test: remove 'depositERC20' related tests and fix outdated ones

xWerk · Jul 25, 2024 · 5a39d8e · 5a39d8e
1 parent 1b4b1e1
commit 5a39d8e
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 80 deletions.
diff --git a/test/unit/concrete/container/deposit-erc20/depositERC20.t.sol b/test/unit/concrete/container/deposit-erc20/depositERC20.t.sol
diff --git a/test/unit/concrete/container/deposit-erc20/depositERC20.tree b/test/unit/concrete/container/deposit-erc20/depositERC20.tree
diff --git a/test/unit/concrete/container/receive/receive.t.sol b/test/unit/concrete/container/receive/receive.t.sol
@@ -13,9 +13,9 @@ contract Receive_Unit_Concrete_Test is Container_Unit_Concrete_Test {
         // Make Bob the caller for this test suite
         vm.startPrank({ msgSender: users.bob });
 
-        // Expect the {AssetDeposited} event to be emitted upon ETH deposit
+        // Expect the {NativeDeposited} event to be emitted upon ETH deposit
         vm.expectEmit();
-        emit Events.AssetDeposited({ sender: users.bob, asset: address(0), amount: 1 ether });
+        emit Events.NativeDeposited({ sender: users.bob, amount: 1 ether });
 
         // Run the test
         (bool success, ) = address(container).call{ value: 1 ether }("");

diff --git a/test/unit/concrete/container/receive/receive.tree b/test/unit/concrete/container/receive/receive.tree
@@ -1,2 +1,2 @@
 receive.t.sol
-└── it should emit an {AssetDeposited} event
+└── it should emit an {NativeDeposited} event
diff --git a/test/unit/concrete/container/withdraw-erc20/withdrawERC20.t.sol b/test/unit/concrete/container/withdraw-erc20/withdrawERC20.t.sol
@@ -40,8 +40,8 @@ contract WithdrawERC20_Unit_Concrete_Test is Container_Unit_Concrete_Test {
         // Approve the {Container} contract to spend USDT tokens on behalf of Eve
         usdt.approve({ spender: address(container), amount: 100e6 });
 
-        // Deposit sufficient ERC-20 tokens into the container to enable the withdrawal
-        container.depositERC20({ asset: IERC20(address(usdt)), amount: 100e6 });
+        // Deposit enough ERC-20 tokens into the container to enable the withdrawal
+        usdt.transfer({ recipient: address(container), amount: 100e6 });
         _;
     }
 

diff --git a/test/utils/Events.sol b/test/utils/Events.sol
@@ -9,11 +9,10 @@ abstract contract Events {
                                     CONTAINER
     //////////////////////////////////////////////////////////////////////////*/
 
-    /// @notice Emitted when an `amount` amount of `asset` ERC-20 asset is deposited on the container
+    /// @notice Emitted when an `amount` amount of `asset` native tokens (ETH) is deposited on the container
     /// @param sender The address of the depositor
-    /// @param asset The address of the deposited ERC-20 token
     /// @param amount The amount of the deposited ERC-20 token
-    event AssetDeposited(address indexed sender, address indexed asset, uint256 amount);
+    event NativeDeposited(address indexed sender, uint256 amount);
 
     /// @notice Emitted when an `amount` amount of `asset` ERC-20 asset is withdrawn from the container
     /// @param sender The address to which the tokens were transferred