aboutsummaryrefslogtreecommitdiffstats
path: root/functest_kubernetes
diff options
context:
space:
mode:
Diffstat (limited to 'functest_kubernetes')
-rw-r--r--functest_kubernetes/k8stest.py2
-rw-r--r--functest_kubernetes/test_k8stest.py14
2 files changed, 16 insertions, 0 deletions
diff --git a/functest_kubernetes/k8stest.py b/functest_kubernetes/k8stest.py
index 7d9400ed..2977bb81 100644
--- a/functest_kubernetes/k8stest.py
+++ b/functest_kubernetes/k8stest.py
@@ -51,6 +51,8 @@ class K8sTesting(testcase.TestCase):
lines = output.split('\n')
i = 0
while i < len(lines):
+ if 'Error' in lines[i]:
+ self.__logger.error(lines[i])
if '[k8s.io]' in lines[i]:
if i != 0 and 'seconds' in lines[i - 1]:
self.__logger.debug(lines[i - 1])
diff --git a/functest_kubernetes/test_k8stest.py b/functest_kubernetes/test_k8stest.py
index 05f40e12..9db14d5a 100644
--- a/functest_kubernetes/test_k8stest.py
+++ b/functest_kubernetes/test_k8stest.py
@@ -67,6 +67,20 @@ class K8sTests(unittest.TestCase):
self.k8stesting.run_kubetest()
@mock.patch('functest_kubernetes.k8stest.os.path.isfile')
+ def test_error_logging(self, mock_isfile):
+ # pylint: disable=unused-argument
+ with mock.patch('functest_kubernetes.k8stest.'
+ 'subprocess.Popen') as mock_popen, \
+ mock.patch.object(self.k8stesting,
+ '_K8sTesting__logger') as mock_logger:
+ mock_stdout = mock.Mock()
+ attrs = {'stdout.read.return_value': 'Error loading client'}
+ mock_stdout.configure_mock(**attrs)
+ mock_popen.return_value = mock_stdout
+ self.k8stesting.run_kubetest()
+ mock_logger.error.assert_called_with("Error loading client")
+
+ @mock.patch('functest_kubernetes.k8stest.os.path.isfile')
@mock.patch('functest_kubernetes.k8stest.subprocess.Popen')
def test_run(self, mock_open, mock_isfile):
self.assertEquals(self.k8stesting.run(),