From c3a9db6488db2acff97d00c4e1fe20fc989fd8b4 Mon Sep 17 00:00:00 2001 From: hroff-1902 Date: Thu, 11 Apr 2019 22:22:33 +0300 Subject: [PATCH] change comments to docstrings --- freqtrade/tests/test_configuration.py | 45 +++++++++++++++++---------- 1 file changed, 28 insertions(+), 17 deletions(-) diff --git a/freqtrade/tests/test_configuration.py b/freqtrade/tests/test_configuration.py index edc6111da..dde9ea271 100644 --- a/freqtrade/tests/test_configuration.py +++ b/freqtrade/tests/test_configuration.py @@ -619,9 +619,11 @@ def test_validate_tsl(default_conf): configuration._validate_config_consistency(default_conf) -# config['exchange'] subtree has required options in it -# so it cannot be omitted in the config def test_load_config_default_exchange(all_conf) -> None: + """ + config['exchange'] subtree has required options in it + so it cannot be omitted in the config + """ del all_conf['exchange'] assert 'exchange' not in all_conf @@ -630,12 +632,13 @@ def test_load_config_default_exchange(all_conf) -> None: match=r'\'exchange\' is a required property'): configuration = Configuration(Namespace()) configuration._validate_config_schema(all_conf) -### assert 'exchange' in all_conf -# config['exchange']['name'] option is required -# so it cannot be omitted in the config def test_load_config_default_exchange_name(all_conf) -> None: + """ + config['exchange']['name'] option is required + so it cannot be omitted in the config + """ del all_conf['exchange']['name'] assert 'name' not in all_conf['exchange'] @@ -646,10 +649,12 @@ def test_load_config_default_exchange_name(all_conf) -> None: configuration._validate_config_schema(all_conf) -# config['exchange']['sandbox'] option has default value: False -# so it can be omitted in the config and the default value -# should be present in the config as the option value def test_load_config_default_exchange_sandbox(all_conf) -> None: + """ + config['exchange']['sandbox'] option has default value: False + so it can be omitted in the config and the default value + should be present in the config as the option value + """ del all_conf['exchange']['sandbox'] assert 'sandbox' not in all_conf['exchange'] @@ -660,10 +665,12 @@ def test_load_config_default_exchange_sandbox(all_conf) -> None: assert all_conf['exchange']['sandbox'] is False -# config['exchange']['key'] option has default value: '' -# so it can be omitted in the config and the default value -# should be present in the config as the option value def test_load_config_default_exchange_key(all_conf) -> None: + """ + config['exchange']['key'] option has default value: '' + so it can be omitted in the config and the default value + should be present in the config as the option value + """ del all_conf['exchange']['key'] assert 'key' not in all_conf['exchange'] @@ -674,10 +681,12 @@ def test_load_config_default_exchange_key(all_conf) -> None: assert all_conf['exchange']['key'] == '' -# config['exchange']['secret'] option has default value: '' -# so it can be omitted in the config and the default value -# should be present in the config as the option value def test_load_config_default_exchange_secret(all_conf) -> None: + """ + config['exchange']['secret'] option has default value: '' + so it can be omitted in the config and the default value + should be present in the config as the option value + """ del all_conf['exchange']['secret'] assert 'secret' not in all_conf['exchange'] @@ -688,10 +697,12 @@ def test_load_config_default_exchange_secret(all_conf) -> None: assert all_conf['exchange']['secret'] == '' -# config['exchange']['password'] option has default value: '' -# so it can be omitted in the config and the default value -# should be present in the config as the option value def test_load_config_default_exchange_password(all_conf) -> None: + """ + config['exchange']['password'] option has default value: '' + so it can be omitted in the config and the default value + should be present in the config as the option value + """ del all_conf['exchange']['password'] assert 'password' not in all_conf['exchange']