Merge pull request #186 from alfredodeza/env_var

improve the tearDown method as it was contaminating the global config

1 files changed, 10 insertions, 0 deletions

diff --git a/pecan/tests/test_conf.py b/pecan/tests/test_conf.py
index 772b466..ae590c7 100644
--- a/pecan/tests/test_conf.py
+++ b/pecan/tests/test_conf.py
@@ -284,6 +284,11 @@ class TestGlobalConfig(TestCase):
 
 
 class TestConfFromEnv(TestCase):
+    #
+    # Note that there is a good chance of pollution if ``tearDown`` does not
+    # reset the configuration like this class does. If implementing new classes
+    # for configuration this tearDown **needs to be implemented**
+    #
 
     def setUp(self):
         self.conf_from_env = self.get_conf_from_env()
@@ -291,6 +296,11 @@ class TestConfFromEnv(TestCase):
 
     def tearDown(self):
         os.environ['PECAN_CONFIG'] = ''
+        from pecan import configuration
+        configuration.set_config(
+            dict(configuration.initconf()),
+            overwrite=True
+        )
 
     def get_conf_from_env(self):
         from pecan import configuration