From 5fd0e5020f0ecad0753883938b3cf8b5b56c63fc Mon Sep 17 00:00:00 2001 From: di1ara Date: Thu, 20 Jul 2023 13:38:33 -0400 Subject: [PATCH] changed the function name and added a doc string explaining --- tests/conftest.py | 4 ++-- tests/test_configs.py | 7 ++++++- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/tests/conftest.py b/tests/conftest.py index 10be9ef6..6f170615 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -180,8 +180,8 @@ def tmpconfdir( # sessions by default: # https://docs.pytest.org/en/6.2.x/tmpdir.html#the-default-base-temporary-directory # BUT, if we wanted to always wipe conf dir and all contained files, - from shutil import rmtree - rmtree(str(tmp_path)) + # from shutil import rmtree + # rmtree(str(tmp_path)) @pytest.fixture diff --git a/tests/test_configs.py b/tests/test_configs.py index 231e3471..530c282b 100644 --- a/tests/test_configs.py +++ b/tests/test_configs.py @@ -1,9 +1,14 @@ +''' + testing the creation of brokers.toml if it doesn't exist, + issue was found when trying to run piker chart on windows. + +''' from pathlib import Path from piker import config -def test_root_conf_networking_section(): +def test_brokers_created_from_template_on_load_accounts(): # load account names from ``brokers.toml`` accounts_def = config.load_accounts( providers=['binance'],