Merge pull request #3262 from freqtrade/humanize_show_trades_error

Humanize show-trades error when no database is specified

freqtrade/commands/list_commands.py

@@ -206,6 +206,11 @@ def start_show_trades(args: Dict[str, Any]) -> None:
     from freqtrade.persistence import init, Trade
     import json
     config = setup_utils_configuration(args, RunMode.UTIL_NO_EXCHANGE)
+
+    if 'db_url' not in config:
+        raise OperationalException("--db-url is required for this command.")
+
+    logger.info(f'Using DB: "{config["db_url"]}"')
     init(config['db_url'], clean_open_orders=False)
     tfilter = []
 

tests/commands/test_commands.py

@@ -1077,3 +1077,11 @@ def test_show_trades(mocker, fee, capsys, caplog):
     assert '"trade_id": 1' in captured.out
     assert '"trade_id": 2' in captured.out
     assert '"trade_id": 3' not in captured.out
+    args = [
+        "show-trades",
+    ]
+    pargs = get_args(args)
+    pargs['config'] = None
+
+    with pytest.raises(OperationalException, match=r"--db-url is required for this command."):
+        start_show_trades(pargs)