From e4682b78c5f0e9d3ea90f3038371c9d7530ea082 Mon Sep 17 00:00:00 2001
From: Sam Germain <sam.germain@usask.ca>
Date: Mon, 18 Oct 2021 00:16:49 -0600
Subject: [PATCH] updates suggested on github

---
 freqtrade/freqtradebot.py      | 14 +++++---------
 tests/plugins/test_pairlist.py |  1 -
 2 files changed, 5 insertions(+), 10 deletions(-)

diff --git a/freqtrade/freqtradebot.py b/freqtrade/freqtradebot.py
index dafb3b106..a84e64898 100644
--- a/freqtrade/freqtradebot.py
+++ b/freqtrade/freqtradebot.py
@@ -479,11 +479,7 @@ class FreqtradeBot(LoggingMixin):
             bid_check_dom = self.config.get('bid_strategy', {}).get('check_depth_of_market', {})
             if ((bid_check_dom.get('enabled', False)) and
                     (bid_check_dom.get('bids_to_ask_delta', 0) > 0)):
-                if self._check_depth_of_market(
-                    pair,
-                    bid_check_dom,
-                    side=signal
-                ):
+                if self._check_depth_of_market(pair, bid_check_dom, side=signal):
                     return self.execute_entry(
                         pair,
                         stake_amount,
@@ -629,9 +625,10 @@ class FreqtradeBot(LoggingMixin):
         if not stake_amount:
             return False
 
-        log_type = f"{name} signal found"
-        logger.info(f"{log_type}: about create a new trade for {pair} with stake_amount: "
-                    f"{stake_amount} ...")
+        logger.info(
+            f"{name} signal found: about create a new trade for {pair} with stake_amount: "
+            f"{stake_amount} ..."
+        )
 
         amount = (stake_amount / enter_limit_requested) * leverage
         order_type = self.strategy.order_types['buy']
@@ -1280,7 +1277,6 @@ class FreqtradeBot(LoggingMixin):
         :param trade: Trade instance
         :param limit: limit rate for the sell order
         :param sell_reason: Reason the sell was triggered
-        :param side: "buy" or "sell"
         :return: True if it succeeds (supported) False (not supported)
         """
         exit_type = 'sell'  # TODO-lev: Update to exit
diff --git a/tests/plugins/test_pairlist.py b/tests/plugins/test_pairlist.py
index fc8b20f02..93eebde82 100644
--- a/tests/plugins/test_pairlist.py
+++ b/tests/plugins/test_pairlist.py
@@ -665,7 +665,6 @@ def test_PerformanceFilter_error(mocker, whitelist_conf, caplog) -> None:
 
 
 @pytest.mark.usefixtures("init_persistence")
-# TODO-lev: @pytest.mark.parametrize('is_short', [True, False])
 def test_PerformanceFilter_lookback(mocker, whitelist_conf, fee, caplog) -> None:
     whitelist_conf['exchange']['pair_whitelist'].append('XRP/BTC')
     whitelist_conf['pairlists'] = [