Merge pull request #13 from cvxgrp/12-remove-obsolete-fixture

no portfolio fixture needed
cvxgrp · Jan 1, 2024 · 2befa91 · 2befa91
2 parents 16be850 + bd64741
commit 2befa91
Showing 1 changed file with 0 additions and 10 deletions.
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -6,10 +6,6 @@
 import pandas as pd
 import pytest
 
-from cvx.simulator.portfolio import EquityPortfolio
-
-# from cvx.simulator.portfolio import build_portfolio
-
 
 @pytest.fixture(scope="session", name="resource_dir")
 def resource_fixture():
@@ -22,9 +18,3 @@ def prices(resource_dir):
     return pd.read_csv(
         resource_dir / "price.csv", index_col=0, header=0, parse_dates=True
     )
-
-
-@pytest.fixture()
-def portfolio(prices):
-    positions = pd.DataFrame(index=prices.index, columns=prices.columns, data=1.0)
-    return EquityPortfolio(prices=prices, stocks=positions)