return dict from _rpc_status and handle rendering in module impl

freqtrade/rpc/telegram.py

@@ -136,8 +136,22 @@ class Telegram(RPC):
             return
 
         try:
-            for trade_msg in self._rpc_trade_status():
+            results = self._rpc_trade_status()
-                self._send_msg(trade_msg, bot=bot)
+            messages = [
+                "*Trade ID:* `{trade_id}`\n"
+                "*Current Pair:* [{pair}]({market_url})\n"
+                "*Open Since:* `{date}`\n"
+                "*Amount:* `{amount}`\n"
+                "*Open Rate:* `{open_rate:.8f}`\n"
+                "*Close Rate:* `{close_rate}`\n"
+                "*Current Rate:* `{current_rate:.8f}`\n"
+                "*Close Profit:* `{close_profit}`\n"
+                "*Current Profit:* `{current_profit:.2f}%`\n"
+                "*Open Order:* `{open_order}`".format(**result)
+                for result in results
+            ]
+            for msg in messages:
+                self._send_msg(msg, bot=bot)
         except RPCException as e:
             self._send_msg(str(e), bot=bot)
 

freqtrade/tests/rpc/test_rpc.py

@@ -52,24 +52,21 @@ def test_rpc_trade_status(default_conf, ticker, fee, markets, mocker) -> None:
         rpc._rpc_trade_status()
 
     freqtradebot.create_trade()
-    trades = rpc._rpc_trade_status()
+    results = rpc._rpc_trade_status()
-    trade = trades[0]
 
-    result_message = [
+    assert {
-        '*Trade ID:* `1`\n'
+        'trade_id': 1,
-        '*Current Pair:* '
+        'pair': 'ETH/BTC',
-        '[ETH/BTC](https://bittrex.com/Market/Index?MarketName=BTC-ETH)\n'
+        'market_url': 'https://bittrex.com/Market/Index?MarketName=BTC-ETH',
-        '*Open Since:* `just now`\n'
+        'date': 'just now',
-        '*Amount:* `90.99181074`\n'
+        'open_rate': 1.099e-05,
-        '*Open Rate:* `0.00001099`\n'
+        'close_rate': None,
-        '*Close Rate:* `None`\n'
+        'current_rate': 1.098e-05,
-        '*Current Rate:* `0.00001098`\n'
+        'amount': 90.99181074,
-        '*Close Profit:* `None`\n'
+        'close_profit': None,
-        '*Current Profit:* `-0.59%`\n'
+        'current_profit': -0.59,
-        '*Open Order:* `(limit buy rem=0.00000000)`'
+        'open_order': '(limit buy rem=0.00000000)'
-    ]
+    } == results[0]
-    assert trades == result_message
-    assert trade.find('[ETH/BTC]') >= 0
 
 
 def test_rpc_status_table(default_conf, ticker, fee, markets, mocker) -> None:

freqtrade/tests/rpc/test_rpc_telegram.py

@@ -209,7 +209,19 @@ def test_status(default_conf, update, mocker, fee, ticker, markets) -> None:
     mocker.patch.multiple(
         'freqtrade.rpc.telegram.Telegram',
         _init=MagicMock(),
-        _rpc_trade_status=MagicMock(return_value=[1, 2, 3]),
+        _rpc_trade_status=MagicMock(return_value=[{
+            'trade_id': 1,
+            'pair': 'ETH/BTC',
+            'market_url': 'https://bittrex.com/Market/Index?MarketName=BTC-ETH',
+            'date': 'just now',
+            'open_rate': 1.099e-05,
+            'close_rate': None,
+            'current_rate': 1.098e-05,
+            'amount': 90.99181074,
+            'close_profit': None,
+            'current_profit': -0.59,
+            'open_order': '(limit buy rem=0.00000000)'
+        }]),
         _status_table=status_table,
         _send_msg=msg_mock
     )
@@ -223,7 +235,7 @@ def test_status(default_conf, update, mocker, fee, ticker, markets) -> None:
         freqtradebot.create_trade()
 
     telegram._status(bot=MagicMock(), update=update)
-    assert msg_mock.call_count == 3
+    assert msg_mock.call_count == 1
 
     update.message.text = MagicMock()
     update.message.text.replace = MagicMock(return_value='table 2 3')