lychiyu/freqtrade_origin
1
0
Fork 0
mirror of https://github.com/freqtrade/freqtrade.git synced 2024-11-12 19:23:55 +00:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Merge pull request #10810 from xzmeng/delete-market-change
Some checks failed
Build Documentation / Deploy Docs through mike (push) Waiting to run
Details
Devcontainer Pre-Build / build-and-push (push) Has been cancelled
Details

Browse Source 
fix: delete market change file when deleting backtest result
This commit is contained in:
Matthias 2024-10-19 13:37:43 +02:00 committed by GitHub
commit 9b16cc6266
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
freqtrade/data/btanalysis.py
View File

@ -244,9 +244,10 @@ def delete_backtest_result(file_abs: Path):
"""
# *.meta.json
logger.info(f"Deleting backtest result file: {file_abs.name}")
file_abs_meta = file_abs.with_suffix(".meta.json")
file_abs.unlink()
file_abs_meta.unlink()
for file in file_abs.parent.glob(f"{file_abs.stem}*"):
logger.info(f"Deleting file: {file}")
file.unlink()
def update_backtest_metadata(filename: Path, strategy: str, content: dict[str, Any]):

3
tests/rpc/test_rpc_apiserver.py
View File

@ -2598,6 +2598,8 @@ def test_api_delete_backtest_history_entry(botclient, tmp_path: Path):
file_path.touch()
meta_path = file_path.with_suffix(".meta.json")
meta_path.touch()
market_change_path = file_path.with_name(file_path.stem + "_market_change.feather")
market_change_path.touch()
rc = client_delete(client, f"{BASE_URI}/backtest/history/randomFile.json")
assert_response(rc, 503)
@ -2614,6 +2616,7 @@ def test_api_delete_backtest_history_entry(botclient, tmp_path: Path):
assert not file_path.exists()
assert not meta_path.exists()
assert not market_change_path.exists()
def test_api_patch_backtest_history_entry(botclient, tmp_path: Path):