fix: deployments

yearn · Dec 15, 2023 · e048be1 · e048be1
1 parent cda3434
commit e048be1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 18 deletions.
diff --git a/contracts/debtAllocators/YieldManager/YieldManager.sol b/contracts/debtAllocators/YieldManager/YieldManager.sol
@@ -69,9 +69,10 @@ contract YieldManager is Governance {
 
     constructor(
         address _governance,
-        address _strategyManager
+        bytes4[] memory _selectors
     ) Governance(_governance) {
-        strategyManager = _strategyManager;
+        // Deploy a new strategy manager
+        strategyManager = address(new StrategyManager(_governance, _selectors));
         // Default to 1 BP loss
         maxDebtUpdateLoss = 1;
     }

diff --git a/tests/conftest.py b/tests/conftest.py
@@ -410,28 +410,18 @@ def generic_debt_allocator(generic_debt_allocator_factory, project, vault, daddy
 
 
 @pytest.fixture(scope="session")
-def deploy_strategy_manager(project, daddy):
-    def deploy_strategy_manager():
-        strategy_manager = daddy.deploy(
-            project.StrategyManager, daddy, ["0x2606a10b", "0xdf69b22a"]
-        )
-
-        return strategy_manager
-
-    yield deploy_strategy_manager
-
-
-@pytest.fixture(scope="session")
-def strategy_manager(deploy_strategy_manager):
-    strategy_manager = deploy_strategy_manager()
+def strategy_manager(project, yield_manager):
+    strategy_manager = project.StrategyManager.at(yield_manager.strategyManager())
 
     yield strategy_manager
 
 
 @pytest.fixture(scope="session")
-def deploy_yield_manager(project, daddy, strategy_manager):
+def deploy_yield_manager(project, daddy):
     def deploy_yield_manager():
-        yield_manager = daddy.deploy(project.YieldManager, daddy, strategy_manager)
+        yield_manager = daddy.deploy(
+            project.YieldManager, daddy, ["0x2606a10b", "0xdf69b22a"]
+        )
 
         return yield_manager