From 76a3e97e05616677871d2b4f32fc7cdce614834a Mon Sep 17 00:00:00 2001 From: Matthias Date: Fri, 19 Aug 2022 06:39:51 +0200 Subject: [PATCH] Add migrations end message closes #7251 --- freqtrade/persistence/migrations.py | 6 ++++++ tests/test_persistence.py | 1 + 2 files changed, 7 insertions(+) diff --git a/freqtrade/persistence/migrations.py b/freqtrade/persistence/migrations.py index e54675f16..311554359 100644 --- a/freqtrade/persistence/migrations.py +++ b/freqtrade/persistence/migrations.py @@ -307,7 +307,9 @@ def check_migrate(engine, decl_base, previous_tables) -> None: # Migrates both trades and orders table! # if ('orders' not in previous_tables # or not has_column(cols_orders, 'stop_price')): + migrating = False if not has_column(cols_trades, 'precision_mode'): + migrating = True logger.info(f"Running database migration for trades - " f"backup: {table_back_name}, {order_table_bak_name}") migrate_trades_and_orders_table( @@ -315,6 +317,7 @@ def check_migrate(engine, decl_base, previous_tables) -> None: order_table_bak_name, cols_orders) if not has_column(cols_pairlocks, 'side'): + migrating = True logger.info(f"Running database migration for pairlocks - " f"backup: {pairlock_table_bak_name}") @@ -329,3 +332,6 @@ def check_migrate(engine, decl_base, previous_tables) -> None: set_sqlite_to_wal(engine) fix_old_dry_orders(engine) + + if migrating: + logger.info("Database migration finished.") diff --git a/tests/test_persistence.py b/tests/test_persistence.py index f68791b72..bfba58706 100644 --- a/tests/test_persistence.py +++ b/tests/test_persistence.py @@ -1387,6 +1387,7 @@ def test_migrate_new(mocker, default_conf, fee, caplog): assert log_has("trying trades_bak2", caplog) assert log_has("Running database migration for trades - backup: trades_bak2, orders_bak0", caplog) + assert log_has("Database migration finished.", caplog) assert pytest.approx(trade.open_trade_value) == trade._calc_open_trade_value( trade.amount, trade.open_rate) assert trade.close_profit_abs is None