test: ensure solution settler is called in bcowpool transaction

defi-wonderland · Jul 17, 2024 · 7c89b62 · 7c89b62
1 parent ac4399e
commit 7c89b62
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/test/unit/BCoWPool/BCoWPool.t.sol b/test/unit/BCoWPool/BCoWPool.t.sol
@@ -20,6 +20,10 @@ contract BCoWPool is BCoWPoolBase {
     assumeNotForgeAddress(_settler);
     vm.mockCall(_settler, abi.encodePacked(ISettlement.domainSeparator.selector), abi.encode(_separator));
     vm.mockCall(_settler, abi.encodePacked(ISettlement.vaultRelayer.selector), abi.encode(_relayer));
+    // it should ask the solution settler for the domain separator
+    vm.expectCall(_settler, abi.encodePacked(ISettlement.domainSeparator.selector));
+    // it should ask the solution settler for the vault relayer
+    vm.expectCall(_settler, abi.encodePacked(ISettlement.vaultRelayer.selector));
     MockBCoWPool pool = new MockBCoWPool(_settler, _appData);
     // it should set the solution settler
     assertEq(address(pool.SOLUTION_SETTLER()), _settler);

diff --git a/test/unit/BCoWPool/BCoWPool.tree b/test/unit/BCoWPool/BCoWPool.tree
@@ -1,7 +1,9 @@
 BCoWPool::Constructor
 └── when called
     ├── it should set the solution settler
+    ├── it should ask the solution settler for the domain separator
     ├── it should set the domain separator
+    ├── it should ask the solution settler for the vault relayer
     ├── it should set the vault relayer
     └── it should set the app data