fix pep8 on test_conf

Author: Yair Fried

tools/kcli/tests/test_conf.py

@@ -8,18 +8,23 @@
 
 @pytest.fixture()
 def our_cwd_setup(request):
+    """Change cwd to test_cwd dir. Revert to original dir on teardown. """
+
     bkp = os.getcwd()
+
     def our_cwd_teardown():
         os.chdir(bkp)
+
     request.addfinalizer(our_cwd_teardown)
     os.chdir(MYCWD)
 
 
-
 @pytest.yield_fixture
 def os_environ():
-    from kcli import conf
     """Backups env var from os.environ and restores it at teardown. """
+
+    from kcli import conf
+
     backup_flag = False
     if conf.ENV_VAR_NAME in os.environ:
         backup_flag = True
@@ -31,28 +36,6 @@ def os_environ():
 
 def test_get_config_dir(our_cwd_setup):
     from kcli import conf
-    file = conf.load_config_file()
-    assert os.path.abspath(file.get("DEFAULTS", "KHALEESI_DIR")) == \
-           os.path.abspath(MYCWD)
-
-    # # Negative - Missing input
-    # with pytest.raises(ValueError) as exc:
-    #     core.get_config_dir(fake_config_cli(config_dir=None))
-    # assert "Missing path" in str(exc.value)
-    #
-    # # Negative - Bad path
-    # with pytest.raises(ValueError) as exc:
-    #     core.get_config_dir(fake_config_cli(config_dir="/fake/path"))
-    # assert "Bad path" in str(exc.value)
-    # assert "/fake/path" in str(exc.value)
-    #
-    # # verify CLI
-    # assert core.get_config_dir(
-    #     fake_config_cli(config_dir=SETTINGS_DIR)) == SETTINGS_DIR
-
-    # Verify ENV
-
-    # # Verify CLI over ENV
-    # os.environ["KHALEESI_SETTINGS"] = "/fake/env/path"
-    # assert core.get_config_dir(
-    #     fake_config_cli(config_dir=SETTINGS_DIR)) == SETTINGS_DIR
+    conf_file = conf.load_config_file()
+    assert os.path.abspath(conf_file.get("DEFAULTS", "KHALEESI_DIR")) == \
+        os.path.abspath(MYCWD)