Add unit test for config file validation 69/55169/2
authorKonrad Djimeli <konraddjimeli@gmail.com>
Sun, 8 Apr 2018 05:20:45 +0000 (06:20 +0100)
committerKonrad Djimeli <konraddjimeli@gmail.com>
Mon, 9 Apr 2018 04:39:57 +0000 (05:39 +0100)
Change-Id: I170de36f9641fce253acdd169061476443b86f81
Signed-off-by: Konrad Djimeli <konraddjimeli@gmail.com>
functest_kubernetes/k8stest.py
functest_kubernetes/test_k8stest.py

index d851d9e..7d9400e 100644 (file)
@@ -90,7 +90,7 @@ class K8sTesting(testcase.TestCase):
 
         if not os.path.isfile(self.config):
             self.__logger.error(
-                "Cannot run k8s testcases. Config file not found ")
+                "Cannot run k8s testcases. Config file not found")
             return self.EX_RUN_ERROR
 
         self.start_time = time.time()
index eb46ac8..230b6e0 100644 (file)
@@ -14,6 +14,9 @@ import logging
 import os
 import unittest
 
+import mock
+from xtesting.core import testcase
+
 from functest_kubernetes import k8stest
 
 
@@ -27,6 +30,8 @@ class K8sTests(unittest.TestCase):
         os.environ["KUBE_MASTER_URL"] = "https://127.0.0.1:6443"
         os.environ["KUBERNETES_PROVIDER"] = "local"
 
+        self.k8stesting = k8stest.K8sTesting()
+
     def _test_no_env_var(self, var):
         del os.environ[var]
         with self.assertRaises(Exception):
@@ -44,6 +49,18 @@ class K8sTests(unittest.TestCase):
     def test_no_kubernetes_provider(self):
         self._test_no_env_var("KUBERNETES_PROVIDER")
 
+    @mock.patch('functest_kubernetes.k8stest.os.path.isfile',
+                return_value=False)
+    def test_run_missing_config_file(self, mock_func):
+        self.k8stesting.config = 'not_file'
+        with mock.patch.object(self.k8stesting,
+                               '_K8sTesting__logger') as mock_logger:
+            self.assertEquals(self.k8stesting.run(),
+                              testcase.TestCase.EX_RUN_ERROR)
+            mock_logger.error.assert_called_with(
+                "Cannot run k8s testcases. Config file not found")
+        mock_func.assert_called_with('not_file')
+
 
 if __name__ == "__main__":
     logging.disable(logging.CRITICAL)