From 585681c8aeed55d6d1c2f3efb6443a7def4ce0d6 Mon Sep 17 00:00:00 2001 From: Jean-Baptiste LE STANG Date: Sat, 30 Dec 2017 00:18:29 +0100 Subject: [PATCH] fix flake8 code review --- freqtrade/main.py | 4 ++-- freqtrade/tests/test_main.py | 10 +++++----- freqtrade/tests/test_rpc_telegram.py | 16 ++++++++-------- 3 files changed, 15 insertions(+), 15 deletions(-) diff --git a/freqtrade/main.py b/freqtrade/main.py index 620ca6171..7b91720d7 100755 --- a/freqtrade/main.py +++ b/freqtrade/main.py @@ -75,10 +75,10 @@ def _process(dynamic_whitelist: Optional[int] = 0) -> bool: balance = exchange.get_balance(_CONF['stake_currency']) stake_amount = float(_CONF['stake_amount']) if _CONF.get('experimental', {}).get('auto_stake_amount', False): - #Numbers of trading slots available + # Numbers of trading slots available nb_trades_left = float(_CONF['max_open_trades'] - len (trades)) print("nb_trades_left "+str(nb_trades_left)) - #stake_amount in the conf is being used as the max value authorized + # Stake_amount in the conf is being used as the max value authorized stake_amount = min(_CONF['stake_amount'], balance / nb_trades_left) print("stake amount "+str(_CONF['stake_amount'])+" "+str(balance)) # Create entity and execute trade diff --git a/freqtrade/tests/test_main.py b/freqtrade/tests/test_main.py index 420825392..ab4f6e33e 100644 --- a/freqtrade/tests/test_main.py +++ b/freqtrade/tests/test_main.py @@ -116,7 +116,7 @@ def test_create_trade(default_conf, ticker, limit_buy_order, mocker): whitelist = copy.deepcopy(default_conf['exchange']['pair_whitelist']) init(default_conf, create_engine('sqlite://')) - create_trade(0.001,1) + create_trade(0.001, 1) trade = Trade.query.first() assert trade is not None @@ -250,7 +250,7 @@ def test_handle_trade(default_conf, limit_buy_order, limit_sell_order, mocker): ticker=MagicMock(return_value={'price_usd': 15000.0}), _cache_symbols=MagicMock(return_value={'BTC': 1})) init(default_conf, create_engine('sqlite://')) - create_trade(0.001,1) + create_trade(0.001, 1) trade = Trade.query.first() assert trade @@ -283,7 +283,7 @@ def test_handle_trade_roi(default_conf, ticker, limit_buy_order, mocker, caplog) mocker.patch('freqtrade.main.min_roi_reached', return_value=True) init(default_conf, create_engine('sqlite://')) - create_trade(0.001,1) + create_trade(0.001, 1) trade = Trade.query.first() trade.is_open = True @@ -315,7 +315,7 @@ def test_handle_trade_experimental(default_conf, ticker, limit_buy_order, mocker mocker.patch('freqtrade.main.min_roi_reached', return_value=False) init(default_conf, create_engine('sqlite://')) - create_trade(0.001,1) + create_trade(0.001, 1) trade = Trade.query.first() trade.is_open = True @@ -341,7 +341,7 @@ def test_close_trade(default_conf, ticker, limit_buy_order, limit_sell_order, mo # Create trade and sell it init(default_conf, create_engine('sqlite://')) - create_trade(0.001,1) + create_trade(0.001, 1) trade = Trade.query.first() assert trade diff --git a/freqtrade/tests/test_rpc_telegram.py b/freqtrade/tests/test_rpc_telegram.py index 8a701b29b..3b37968f2 100644 --- a/freqtrade/tests/test_rpc_telegram.py +++ b/freqtrade/tests/test_rpc_telegram.py @@ -102,7 +102,7 @@ def test_status_handle(default_conf, update, ticker, mocker): msg_mock.reset_mock() # Create some test data - create_trade(0.001,1) + create_trade(0.001, 1) # Trigger status while we have a fulfilled order for the open trade _status(bot=MagicMock(), update=update) @@ -138,7 +138,7 @@ def test_status_table_handle(default_conf, update, ticker, mocker): msg_mock.reset_mock() # Create some test data - create_trade(15.0,15.0) + create_trade(15.0, 15.0) _status_table(bot=MagicMock(), update=update) @@ -175,7 +175,7 @@ def test_profit_handle( msg_mock.reset_mock() # Create some test data - create_trade(0.001,1) + create_trade(0.001, 1) trade = Trade.query.first() # Simulate fulfilled LIMIT_BUY order for trade @@ -224,7 +224,7 @@ def test_forcesell_handle(default_conf, update, ticker, ticker_sell_up, mocker): init(default_conf, create_engine('sqlite://')) # Create some test data - create_trade(0.001,1) + create_trade(0.001, 1) trade = Trade.query.first() assert trade @@ -261,7 +261,7 @@ def test_forcesell_down_handle(default_conf, update, ticker, ticker_sell_down, m init(default_conf, create_engine('sqlite://')) # Create some test data - create_trade(0.001,1) + create_trade(0.001, 1) # Decrease the price and sell it mocker.patch.multiple('freqtrade.main.exchange', @@ -388,7 +388,7 @@ def test_performance_handle( init(default_conf, create_engine('sqlite://')) # Create some test data - create_trade(0.001,1) + create_trade(0.001, 1) trade = Trade.query.first() assert trade @@ -426,7 +426,7 @@ def test_daily_handle( init(default_conf, create_engine('sqlite://')) # Create some test data - create_trade(0.001,1) + create_trade(0.001, 1) trade = Trade.query.first() assert trade @@ -479,7 +479,7 @@ def test_count_handle(default_conf, update, ticker, mocker): update_state(State.RUNNING) # Create some test data - create_trade(0.001,1) + create_trade(0.001, 1) msg_mock.reset_mock() _count(bot=MagicMock(), update=update)