Add some test-coverage
This commit is contained in:
parent
f3de74f817
commit
15bb0af1b3
@ -51,6 +51,9 @@ def test_get_latest_hyperopt_file(testdatadir, mocker):
|
|||||||
res = get_latest_hyperopt_file(testdatadir.parent)
|
res = get_latest_hyperopt_file(testdatadir.parent)
|
||||||
assert res == testdatadir.parent / "hyperopt_results.pickle"
|
assert res == testdatadir.parent / "hyperopt_results.pickle"
|
||||||
|
|
||||||
|
res = get_latest_hyperopt_file(str(testdatadir.parent))
|
||||||
|
assert res == testdatadir.parent / "hyperopt_results.pickle"
|
||||||
|
|
||||||
|
|
||||||
def test_load_backtest_data_old_format(testdatadir):
|
def test_load_backtest_data_old_format(testdatadir):
|
||||||
|
|
||||||
|
@ -5,7 +5,7 @@ from datetime import datetime
|
|||||||
from pathlib import Path
|
from pathlib import Path
|
||||||
from copy import deepcopy
|
from copy import deepcopy
|
||||||
from typing import Dict, List
|
from typing import Dict, List
|
||||||
from unittest.mock import MagicMock, PropertyMock
|
from unittest.mock import MagicMock
|
||||||
|
|
||||||
import pandas as pd
|
import pandas as pd
|
||||||
import pytest
|
import pytest
|
||||||
@ -81,14 +81,14 @@ def create_results(mocker, hyperopt, testdatadir) -> List[Dict]:
|
|||||||
|
|
||||||
mocker.patch.object(Path, "is_file", MagicMock(return_value=False))
|
mocker.patch.object(Path, "is_file", MagicMock(return_value=False))
|
||||||
stat_mock = MagicMock()
|
stat_mock = MagicMock()
|
||||||
stat_mock.st_size = PropertyMock(return_value=1)
|
stat_mock.st_size = 1
|
||||||
mocker.patch.object(Path, "stat", MagicMock(return_value=False))
|
mocker.patch.object(Path, "stat", MagicMock(return_value=stat_mock))
|
||||||
|
|
||||||
mocker.patch.object(Path, "unlink", MagicMock(return_value=True))
|
mocker.patch.object(Path, "unlink", MagicMock(return_value=True))
|
||||||
mocker.patch('freqtrade.optimize.hyperopt.dump', return_value=None)
|
mocker.patch('freqtrade.optimize.hyperopt.dump', return_value=None)
|
||||||
mocker.patch('freqtrade.optimize.hyperopt.file_dump_json')
|
mocker.patch('freqtrade.optimize.hyperopt.file_dump_json')
|
||||||
|
|
||||||
return [{'loss': 1, 'result': 'foo', 'params': {}}]
|
return [{'loss': 1, 'result': 'foo', 'params': {}, 'is_best': True}]
|
||||||
|
|
||||||
|
|
||||||
def test_setup_hyperopt_configuration_without_arguments(mocker, default_conf, caplog) -> None:
|
def test_setup_hyperopt_configuration_without_arguments(mocker, default_conf, caplog) -> None:
|
||||||
@ -524,6 +524,28 @@ def test_read_results_returns_epochs(mocker, hyperopt, testdatadir, caplog) -> N
|
|||||||
mock_load.assert_called_once()
|
mock_load.assert_called_once()
|
||||||
|
|
||||||
|
|
||||||
|
def test_load_previous_results(mocker, hyperopt, testdatadir, caplog) -> None:
|
||||||
|
epochs = create_results(mocker, hyperopt, testdatadir)
|
||||||
|
mock_load = mocker.patch('freqtrade.optimize.hyperopt.load', return_value=epochs)
|
||||||
|
mocker.patch.object(Path, 'is_file', MagicMock(return_value=True))
|
||||||
|
statmock = MagicMock()
|
||||||
|
statmock.st_size = 5
|
||||||
|
# mocker.patch.object(Path, 'stat', MagicMock(return_value=statmock))
|
||||||
|
|
||||||
|
results_file = testdatadir / 'optimize' / 'ut_results.pickle'
|
||||||
|
|
||||||
|
hyperopt_epochs = hyperopt.load_previous_results(results_file)
|
||||||
|
|
||||||
|
assert hyperopt_epochs == epochs
|
||||||
|
mock_load.assert_called_once()
|
||||||
|
|
||||||
|
del epochs[0]['is_best']
|
||||||
|
mock_load = mocker.patch('freqtrade.optimize.hyperopt.load', return_value=epochs)
|
||||||
|
|
||||||
|
with pytest.raises(OperationalException):
|
||||||
|
hyperopt.load_previous_results(results_file)
|
||||||
|
|
||||||
|
|
||||||
def test_roi_table_generation(hyperopt) -> None:
|
def test_roi_table_generation(hyperopt) -> None:
|
||||||
params = {
|
params = {
|
||||||
'roi_t1': 5,
|
'roi_t1': 5,
|
||||||
|
Loading…
Reference in New Issue
Block a user