Fix test

immutable · Nov 13, 2023 · 17ab7d4 · 17ab7d4
1 parent c4b0252
commit 17ab7d4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/test/unit/child/ChildERC20Bridge.t.sol b/test/unit/child/ChildERC20Bridge.t.sol
@@ -82,7 +82,7 @@ contract ChildERC20BridgeUnitTest is Test, IChildERC20BridgeEvents, IChildERC20B
         });
 
         vm.expectRevert(ZeroAddress.selector);
-        bridge.initialize(roles, address(0), ROOT_BRIDGE_ADAPTOR, address(1), ROOT_CHAIN_NAME, address(1));
+        bridge.initialize(roles, address(1), ROOT_BRIDGE_ADAPTOR, address(1), ROOT_CHAIN_NAME, address(1));
     }
 
     function test_RevertIf_InitializeWithAZeroAddressPauser() public {
@@ -96,7 +96,7 @@ contract ChildERC20BridgeUnitTest is Test, IChildERC20BridgeEvents, IChildERC20B
         });
 
         vm.expectRevert(ZeroAddress.selector);
-        bridge.initialize(roles, address(0), ROOT_BRIDGE_ADAPTOR, address(1), ROOT_CHAIN_NAME, address(1));
+        bridge.initialize(roles, address(1), ROOT_BRIDGE_ADAPTOR, address(1), ROOT_CHAIN_NAME, address(1));
     }
 
     function test_RevertIf_InitializeWithAZeroAddressUnpauser() public {
@@ -110,7 +110,7 @@ contract ChildERC20BridgeUnitTest is Test, IChildERC20BridgeEvents, IChildERC20B
         });
 
         vm.expectRevert(ZeroAddress.selector);
-        bridge.initialize(roles, address(0), ROOT_BRIDGE_ADAPTOR, address(1), ROOT_CHAIN_NAME, address(1));
+        bridge.initialize(roles, address(1), ROOT_BRIDGE_ADAPTOR, address(1), ROOT_CHAIN_NAME, address(1));
     }
 
     function test_RevertIf_InitializeWithAZeroAddressVariableManager() public {
@@ -124,7 +124,7 @@ contract ChildERC20BridgeUnitTest is Test, IChildERC20BridgeEvents, IChildERC20B
         });
 
         vm.expectRevert(ZeroAddress.selector);
-        bridge.initialize(roles, address(0), ROOT_BRIDGE_ADAPTOR, address(1), ROOT_CHAIN_NAME, address(1));
+        bridge.initialize(roles, address(1), ROOT_BRIDGE_ADAPTOR, address(1), ROOT_CHAIN_NAME, address(1));
     }
 
     function test_RevertIf_InitializeWithAZeroAddressAdaptorManager() public {
@@ -138,7 +138,7 @@ contract ChildERC20BridgeUnitTest is Test, IChildERC20BridgeEvents, IChildERC20B
         });
 
         vm.expectRevert(ZeroAddress.selector);
-        bridge.initialize(roles, address(0), ROOT_BRIDGE_ADAPTOR, address(1), ROOT_CHAIN_NAME, address(1));
+        bridge.initialize(roles, address(1), ROOT_BRIDGE_ADAPTOR, address(1), ROOT_CHAIN_NAME, address(1));
     }
 
     function test_RevertIf_InitializeWithAZeroAddressAdapter() public {

diff --git a/test/unit/root/RootERC20Bridge.t.sol b/test/unit/root/RootERC20Bridge.t.sol
@@ -526,7 +526,9 @@ contract RootERC20BridgeUnitTest is Test, IRootERC20BridgeEvents, IRootERC20Brid
 
     function test_RevertIf_updateRootBridgeAdaptorCalledByNonOwner() public {
         vm.prank(address(0xf00f00));
-        vm.expectRevert(NotVariableManager.selector);
+        vm.expectRevert(
+            abi.encodeWithSelector(NotVariableManager.selector, 0xf00f00)
+        );
         rootBridge.updateRootBridgeAdaptor(address(0x11111));
     }