Merge "Improve the pylint score of functest-core"

[functest.git] / functest / tests / unit / ci / test_tier_handler.py

diff --git a/functest/tests/unit/ci/test_tier_handler.py b/functest/tests/unit/ci/test_tier_handler.py
index 01d99d7..871220d 100644 (file)
--- a/functest/tests/unit/ci/test_tier_handler.py
+++ b/functest/tests/unit/ci/test_tier_handler.py
@@ -5,6 +5,8 @@
 # which accompanies this distribution, and is available at
 # http://www.apache.org/licenses/LICENSE-2.0
 
+# pylint: disable=missing-docstring
+
 import logging
 import unittest
 
@@ -15,8 +17,6 @@ from functest.ci import tier_handler
 
 class TierHandlerTesting(unittest.TestCase):
 
-    logging.disable(logging.CRITICAL)
-
     def setUp(self):
         self.test = mock.Mock()
         attrs = {'get_name.return_value': 'test_name'}
@@ -32,15 +32,24 @@ class TierHandlerTesting(unittest.TestCase):
                                       'test_ci_loop',
                                       description='test_desc')
         self.testcase = tier_handler.TestCase('test_name',
+                                              'true',
                                               self.mock_depend,
                                               'test_criteria',
                                               'test_blocking',
-                                              'test_clean_flag',
                                               description='test_desc')
 
         self.dependency = tier_handler.Dependency('test_installer',
                                                   'test_scenario')
 
+        self.testcase.str = self.testcase.__str__()
+        self.dependency.str = self.dependency.__str__()
+        self.tier.str = self.tier.__str__()
+
+    def test_split_text(self):
+        test_str = 'this is for testing'
+        self.assertEqual(tier_handler.split_text(test_str, 10),
+                         ['this is ', 'for ', 'testing '])
+
     def test_add_test(self):
         self.tier.add_test(self.test)
         self.assertEqual(self.tier.tests_array,
@@ -82,11 +91,11 @@ class TierHandlerTesting(unittest.TestCase):
         self.assertEqual(self.tier.get_ci_loop(),
                          'test_ci_loop')
 
-    def test_testcase_is_none_present_item(self):
+    def test_testcase_is_none_in_item(self):
         self.assertEqual(tier_handler.TestCase.is_none("item"),
                          False)
 
-    def test_testcase_is_none_missing_item(self):
+    def test_testcase_is_none_no_item(self):
         self.assertEqual(tier_handler.TestCase.is_none(None),
                          True)
 
@@ -95,7 +104,7 @@ class TierHandlerTesting(unittest.TestCase):
                                                      'test_scenario'),
                          True)
 
-    def test_testcase_is_compatible_missing_installer_scenario(self):
+    def test_testcase_is_compatible_2(self):
         self.assertEqual(self.testcase.is_compatible('missing_installer',
                                                      'test_scenario'),
                          False)
@@ -107,6 +116,10 @@ class TierHandlerTesting(unittest.TestCase):
         self.assertEqual(self.tier.get_name(),
                          'test_tier')
 
+    def test_testcase_is_enabled(self):
+        self.assertEqual(self.testcase.is_enabled(),
+                         'true')
+
     def test_testcase_get_criteria(self):
         self.assertEqual(self.tier.get_order(),
                          'test_order')
@@ -125,4 +138,5 @@ class TierHandlerTesting(unittest.TestCase):
 
 
 if __name__ == "__main__":
+    logging.disable(logging.CRITICAL)
     unittest.main(verbosity=2)