From 838985f6a06d7f1851282c30ef202eb8eeb214c3 Mon Sep 17 00:00:00 2001 From: Matthias Date: Fri, 21 Aug 2020 07:13:13 +0200 Subject: [PATCH] Don't reset open-order-id just yet it's needed to get the fees --- freqtrade/freqtradebot.py | 1 - tests/test_freqtradebot.py | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/freqtrade/freqtradebot.py b/freqtrade/freqtradebot.py index 0dac4d888..fd5847a94 100644 --- a/freqtrade/freqtradebot.py +++ b/freqtrade/freqtradebot.py @@ -606,7 +606,6 @@ class FreqtradeBot: stake_amount = order['cost'] amount = safe_value_fallback(order, 'filled', 'amount') buy_limit_filled_price = safe_value_fallback(order, 'average', 'price') - order_id = None # in case of FOK the order may be filled immediately and fully elif order_status == 'closed': diff --git a/tests/test_freqtradebot.py b/tests/test_freqtradebot.py index a907ce5c2..9a3dcbd78 100644 --- a/tests/test_freqtradebot.py +++ b/tests/test_freqtradebot.py @@ -1057,7 +1057,7 @@ def test_execute_buy(mocker, default_conf, fee, limit_buy_order, limit_buy_order assert freqtrade.execute_buy(pair, stake_amount) trade = Trade.query.all()[3] assert trade - assert trade.open_order_id is None + assert trade.open_order_id == '555' assert trade.open_rate == 0.5 assert trade.stake_amount == 40.495905365