From: Cédric Ollivier <cedric.ollivier@orange.com>
Date: Fri, 9 Feb 2018 19:46:41 +0000 (+0100)
Subject: Ensure logging is disabled when unit testing feature
X-Git-Tag: 0.2~65
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=commitdiff_plain;h=6432b7db47fe05f8b5b4c55d658537bf71a881c6;p=functest-xtesting.git

Ensure logging is disabled when unit testing feature

Change-Id: Ibe08477a8b50a6c9b71ab4e5b0ebaaab36692b08
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
---

diff --git a/functest/tests/unit/core/test_feature.py b/functest/tests/unit/core/test_feature.py
index 8c73bb5d..3219c726 100644
--- a/functest/tests/unit/core/test_feature.py
+++ b/functest/tests/unit/core/test_feature.py
@@ -55,6 +55,9 @@ class FeatureTestingBase(unittest.TestCase):
 class FeatureTesting(FeatureTestingBase):
 
     def setUp(self):
+        # logging must be disabled else it calls time.time()
+        # what will break these unit tests.
+        logging.disable(logging.CRITICAL)
         with mock.patch('six.moves.builtins.open'):
             self.feature = feature.Feature(
                 project_name=self._project_name, case_name=self._case_name)
@@ -74,6 +77,9 @@ class FeatureTesting(FeatureTestingBase):
 class BashFeatureTesting(FeatureTestingBase):
 
     def setUp(self):
+        # logging must be disabled else it calls time.time()
+        # what will break these unit tests.
+        logging.disable(logging.CRITICAL)
         with mock.patch('six.moves.builtins.open'):
             self.feature = feature.BashFeature(
                 project_name=self._project_name, case_name=self._case_name)
@@ -108,7 +114,4 @@ class BashFeatureTesting(FeatureTestingBase):
 
 
 if __name__ == "__main__":
-    # logging must be disabled else it calls time.time()
-    # what will break these unit tests.
-    logging.disable(logging.CRITICAL)
     unittest.main(verbosity=2)