From 9de95d174f8840bf1057f9db4685cbcb6107d2cf Mon Sep 17 00:00:00 2001
From: Zack Cerza <zack@cerza.org>
Date: Fri, 20 Sep 2013 15:11:13 -0500
Subject: [PATCH] Add a basic test for teuthology.config

---
 teuthology/config.py           | 17 ++++++++++-------
 teuthology/test/test_config.py | 17 +++++++++++++++++
 2 files changed, 27 insertions(+), 7 deletions(-)
 create mode 100644 teuthology/test/test_config.py

diff --git a/teuthology/config.py b/teuthology/config.py
index b69e64cc326..e11901a7eaf 100644
--- a/teuthology/config.py
+++ b/teuthology/config.py
@@ -2,22 +2,25 @@ import os
 import yaml
 import logging
 
-CONF_FILE = os.path.join(os.environ['HOME'], '.teuthology.yaml')
-
 log = logging.getLogger(__name__)
 
 
-class _Config(object):
+class Config(object):
     """
     This class is intended to unify teuthology's many configuration files and
     objects. Currently it serves as a convenient interface to
     ~/.teuthology.yaml and nothing else.
     """
+    teuthology_yaml = os.path.join(os.environ['HOME'], '.teuthology.yaml')
+
     def __init__(self):
-        if os.path.exists(CONF_FILE):
-            self.__conf = yaml.safe_load(file(CONF_FILE))
+        self.load_files()
+
+    def load_files(self):
+        if os.path.exists(self.teuthology_yaml):
+            self.__conf = yaml.safe_load(file(self.teuthology_yaml))
         else:
-            log.debug("%s not found", CONF_FILE)
+            log.debug("%s not found", self.teuthology_yaml)
             self.__conf = {}
 
     # This property declaration exists mainly as an example; it is not
@@ -48,4 +51,4 @@ class _Config(object):
     def __getattr__(self, name):
         return self.__conf.get(name)
 
-config = _Config()
+config = Config()
diff --git a/teuthology/test/test_config.py b/teuthology/test/test_config.py
new file mode 100644
index 00000000000..5742c58b147
--- /dev/null
+++ b/teuthology/test/test_config.py
@@ -0,0 +1,17 @@
+from .. import config
+
+
+class TestConfig(object):
+    def setup(self):
+        pass
+
+    def teardown(self):
+        pass
+
+    def test_get_and_set(self):
+        conf_obj = config.Config()
+        conf_obj.teuthology_yaml = ''
+        conf_obj.load_files()
+        assert conf_obj.ceph_git_base_url == "https://github.com/ceph/"
+        conf_obj._Config__conf['ceph_git_base_url'] = "git://ceph.com/"
+        assert conf_obj.ceph_git_base_url == "git://ceph.com/"