Skip to content

Commit c6294e0

Browse files
committed
test: changed names for new test's methods
1 parent 2878e67 commit c6294e0

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

tests/conftest.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -327,7 +327,7 @@ def with_bounds_params():
327327

328328

329329
@pytest.fixture(scope="module")
330-
def min_ratio_asset_left_to_max_cagr_when_none_params():
330+
def _min_ratio_asset_when_none_params():
331331
return dict(
332332
assets=["SPY.US", "GLD.US"],
333333
ccy="USD",
@@ -337,7 +337,7 @@ def min_ratio_asset_left_to_max_cagr_when_none_params():
337337

338338

339339
@pytest.fixture(scope="module")
340-
def min_ratio_asset_left_to_max_cagr_when_not_none_params():
340+
def _min_ratio_asset_when_not_none_params():
341341
return dict(
342342
assets=["SPY.US", "MCFTR.INDX"],
343343
ccy="RUB",
@@ -388,13 +388,13 @@ def init_frontier_with_bounds(with_bounds_params):
388388

389389

390390
@pytest.fixture(scope="module")
391-
def init_frontier_with_none(min_ratio_asset_left_to_max_cagr_when_none_params):
392-
return ok.EfficientFrontierReb(**min_ratio_asset_left_to_max_cagr_when_none_params)
391+
def init_frontier_with_none(_min_ratio_asset_when_none_params):
392+
return ok.EfficientFrontierReb(**_min_ratio_asset_when_none_params)
393393

394394

395395
@pytest.fixture(scope="module")
396-
def init_frontier_with_not_none(min_ratio_asset_left_to_max_cagr_when_not_none_params):
397-
return ok.EfficientFrontierReb(**min_ratio_asset_left_to_max_cagr_when_not_none_params)
396+
def init_frontier_with_not_none(_min_ratio_asset_when_not_none_params):
397+
return ok.EfficientFrontierReb(**_min_ratio_asset_when_not_none_params)
398398

399399

400400
@pytest.fixture(scope="module")

tests/test_frontier_reb.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -151,18 +151,18 @@ def test_minimize_risk_raises_error_when_no_solution(init_frontier_with_bounds):
151151

152152
@mark.rebalance
153153
@mark.frontier
154-
def test_min_ratio_asset_left_to_max_cagr_when_none(init_frontier_with_none):
154+
def test_min_ratio_asset_when_none(init_frontier_with_none):
155155
x = init_frontier_with_none
156-
result = x._min_ratio_asset_left_to_max_cagr
156+
result = x._min_ratio_asset
157157

158158
assert result == None
159159

160160

161161
@mark.rebalance
162162
@mark.frontier
163-
def test_min_ratio_asset_left_to_max_cagr_when_not_none(init_frontier_with_not_none):
163+
def test_min_ratio_asset_when_not_none(init_frontier_with_not_none):
164164
x = init_frontier_with_not_none
165-
result = x._min_ratio_asset_left_to_max_cagr
165+
result = x._min_ratio_asset
166166

167167
expected_result = {
168168
"min_asset_cagr": approx(0.1959425614987127, abs=1e-2),

0 commit comments

Comments
 (0)