fixed flake8 and mypy errors

freqtrade/rpc/rpc.py

@@ -245,7 +245,7 @@ class RPC:
                         fiat_profit_sum = fiat_profit if isnan(fiat_profit_sum) \
                             else fiat_profit_sum + fiat_profit
                 trades_list.append([
-                    trade.id + f' {direction_str}',
+                    str(trade.id) + f' {direction_str}',
                     trade.pair + ('*' if (trade.open_order_id is not None
                                           and trade.close_rate_requested is None) else '')
                                + ('**' if (trade.close_rate_requested is not None) else ''),

freqtrade/rpc/telegram.py

@@ -272,7 +272,7 @@ class Telegram(RPCHandler):
         else:
             msg['profit_extra'] = ''
         is_fill = msg['type'] == RPCMessageType.SELL_FILL
-        message = [
+        message = "".join([
             f"{msg['emoji']} *{msg['exchange']}:* ",
             f"{'Exited' if is_fill else 'Exiting'} {msg['pair']} (#{msg['trade_id']})\n",
             f"*{'Profit' if is_fill else 'Unrealized Profit'}:* ",
@@ -285,8 +285,7 @@ class Telegram(RPCHandler):
             msg.get('leverage', None) is not None else "",
             f"*Amount:* `{msg['amount']:.8f}`\n",
             f"*Open Rate:* `{msg['open_rate']:.8f}`\n"
-        ]
-        message = "".join(message)
+        ])
         if msg['type'] == RPCMessageType.SELL:
             message += (f"*Current Rate:* `{msg['current_rate']:.8f}`\n"
                         f"*Close Rate:* `{msg['limit']:.8f}`")

tests/rpc/test_rpc_telegram.py

@@ -317,7 +317,8 @@ def test_status_table_handle(default_conf, update, ticker, fee, mocker) -> None:
     fields = re.sub('[ ]+', ' ', line[2].strip()).split(' ')
 
     assert int(fields[0]) == 1
-    assert 'ETH/BTC' in fields[1]
+    assert 'L' in fields[1]
+    assert 'ETH/BTC' in fields[2]
     assert msg_mock.call_count == 1
 
 

tests/test_freqtradebot.py

@@ -3190,6 +3190,7 @@ def test_execute_trade_exit_with_stoploss_on_exchange(
     assert cancel_order.call_count == 1
     assert rpc_mock.call_count == 3
 
+
 # TODO-lev: add short, RPC short, short fill
 def test_may_execute_trade_exit_after_stoploss_on_exchange_hit(default_conf_usdt, ticker_usdt, fee,
                                                                mocker) -> None: