From 2f0775fa1b6381b163ecd55ba1a0b684a9bbfea8 Mon Sep 17 00:00:00 2001
From: Matthias <xmatthias@outlook.com>
Date: Wed, 29 Jan 2020 21:30:29 +0100
Subject: [PATCH] Extract build-config tests to new file

---
 tests/commands/test_build_config.py | 42 +++++++++++++++++++++++++++++
 tests/commands/test_commands.py     | 40 ++-------------------------
 2 files changed, 44 insertions(+), 38 deletions(-)
 create mode 100644 tests/commands/test_build_config.py

diff --git a/tests/commands/test_build_config.py b/tests/commands/test_build_config.py
new file mode 100644
index 000000000..4114ff489
--- /dev/null
+++ b/tests/commands/test_build_config.py
@@ -0,0 +1,42 @@
+import json
+from pathlib import Path
+from unittest.mock import MagicMock
+
+import pytest
+
+from freqtrade.commands import start_new_config
+from tests.conftest import get_args, log_has_re
+
+
+@pytest.mark.parametrize('exchange', ['bittrex', 'binance', 'kraken', 'ftx'])
+def test_start_new_config(mocker, caplog, exchange):
+    wt_mock = mocker.patch.object(Path, "write_text", MagicMock())
+    mocker.patch.object(Path, "exists", MagicMock(return_value=False))
+    sample_selections = {
+        'max_open_trades': 3,
+        'stake_currency': 'USDT',
+        'stake_amount': 100,
+        'fiat_display_currency': 'EUR',
+        'ticker_interval': '15m',
+        'dry_run': True,
+        'exchange_name': exchange,
+        'exchange_key': 'sampleKey',
+        'exchange_secret': 'Samplesecret',
+        'telegram': False,
+        'telegram_token': 'asdf1244',
+        'telegram_chat_id': '1144444',
+    }
+    mocker.patch('freqtrade.commands.build_config_commands.ask_user_config',
+                 return_value=sample_selections)
+    args = [
+        "new-config",
+        "--config",
+        "coolconfig.json"
+    ]
+    start_new_config(get_args(args))
+
+    assert log_has_re("Writing config to .*", caplog)
+    assert wt_mock.call_count == 1
+    result = json.loads(wt_mock.call_args_list[0][0][0])
+    assert result['exchange']['name'] == exchange
+    assert result['ticker_interval'] == '15m'
diff --git a/tests/commands/test_commands.py b/tests/commands/test_commands.py
index 51b69449d..65d7f6eaf 100644
--- a/tests/commands/test_commands.py
+++ b/tests/commands/test_commands.py
@@ -1,4 +1,3 @@
-import json
 import re
 from pathlib import Path
 from unittest.mock import MagicMock, PropertyMock
@@ -9,9 +8,8 @@ from freqtrade.commands import (start_create_userdir, start_download_data,
                                 start_hyperopt_list, start_hyperopt_show,
                                 start_list_exchanges, start_list_markets,
                                 start_list_strategies, start_list_timeframes,
-                                start_new_config, start_new_hyperopt,
-                                start_new_strategy, start_test_pairlist,
-                                start_trading)
+                                start_new_hyperopt, start_new_strategy,
+                                start_test_pairlist, start_trading)
 from freqtrade.configuration import setup_utils_configuration
 from freqtrade.exceptions import OperationalException
 from freqtrade.state import RunMode
@@ -539,40 +537,6 @@ def test_start_new_hyperopt_no_arg(mocker, caplog):
         start_new_hyperopt(get_args(args))
 
 
-@pytest.mark.parametrize('exchange', ['bittrex', 'binance', 'kraken', 'ftx'])
-def test_start_new_config(mocker, caplog, exchange):
-    wt_mock = mocker.patch.object(Path, "write_text", MagicMock())
-    mocker.patch.object(Path, "exists", MagicMock(return_value=False))
-    sample_selections = {
-        'max_open_trades': 3,
-        'stake_currency': 'USDT',
-        'stake_amount': 100,
-        'fiat_display_currency': 'EUR',
-        'ticker_interval': '15m',
-        'dry_run': True,
-        'exchange_name': exchange,
-        'exchange_key': 'sampleKey',
-        'exchange_secret': 'Samplesecret',
-        'telegram': False,
-        'telegram_token': 'asdf1244',
-        'telegram_chat_id': '1144444',
-    }
-    mocker.patch('freqtrade.commands.build_config_commands.ask_user_config',
-                 return_value=sample_selections)
-    args = [
-        "new-config",
-        "--config",
-        "coolconfig.json"
-    ]
-    start_new_config(get_args(args))
-
-    assert log_has_re("Writing config to .*", caplog)
-    assert wt_mock.call_count == 1
-    result = json.loads(wt_mock.call_args_list[0][0][0])
-    assert result['exchange']['name'] == exchange
-    assert result['ticker_interval'] == '15m'
-
-
 def test_download_data_keyboardInterrupt(mocker, caplog, markets):
     dl_mock = mocker.patch('freqtrade.commands.data_commands.refresh_backtest_ohlcv_data',
                            MagicMock(side_effect=KeyboardInterrupt))