aboutsummaryrefslogtreecommitdiffstats
path: root/functest/core
diff options
context:
space:
mode:
authorCédric Ollivier <cedric.ollivier@orange.com>2017-03-27 14:47:16 +0200
committerCédric Ollivier <cedric.ollivier@orange.com>2017-03-27 14:47:16 +0200
commitfbca55174c6698855c8564c45d0018e746bd7643 (patch)
tree3f075703c0e285a3c75626406f3691e2c99b56f8 /functest/core
parent69e51470d8e9f51e02d5f0bd6e957c4e24c603c6 (diff)
Add docstrings in testcase.py
testcase.py is now rated at 10.00/10 by pylint. Change-Id: Ia90e53db7a25aac3827cd29bb9fd6baaa46cd487 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
Diffstat (limited to 'functest/core')
-rw-r--r--functest/core/testcase.py51
1 files changed, 51 insertions, 0 deletions
diff --git a/functest/core/testcase.py b/functest/core/testcase.py
index 85d1fc74..b540cfb5 100644
--- a/functest/core/testcase.py
+++ b/functest/core/testcase.py
@@ -7,13 +7,18 @@
# which accompanies this distribution, and is available at
# http://www.apache.org/licenses/LICENSE-2.0
+"""Define the parent class of Functest TestCase."""
+
import os
import functest.utils.functest_logger as ft_logger
import functest.utils.functest_utils as ft_utils
+__author__ = "Cedric Ollivier <cedric.ollivier@orange.com>"
+
class TestCase(object):
+ """Parent class of Functest TestCase."""
EX_OK = os.EX_OK
EX_RUN_ERROR = os.EX_SOFTWARE
@@ -31,6 +36,17 @@ class TestCase(object):
self.stop_time = ""
def check_criteria(self):
+ """Interpret the results of TestCase.
+
+ It allows getting the results of TestCase. It completes run()
+ which only returns the execution status.
+
+ It can be overriden if checking criteria is not suitable.
+
+ Returns:
+ TestCase.EX_OK if criteria is 'PASS'.
+ TestCase.EX_TESTCASE_FAILED otherwise.
+ """
try:
assert self.criteria
if self.criteria == 'PASS':
@@ -40,11 +56,46 @@ class TestCase(object):
return TestCase.EX_TESTCASE_FAILED
def run(self, **kwargs):
+ """Run TestCase.
+
+ It allows running TestCase and getting its execution
+ status.
+
+ The subclasses must override the default implementation which
+ is false on purpose. The only prerequisite is to set the
+ following attributes to push the results to DB:
+ * case_name,
+ * criteria,
+ * start_time,
+ * stop_time.
+
+ Args:
+ **kwargs: Arbitrary keyword arguments.
+
+ Returns:
+ TestCase.EX_RUN_ERROR.
+ """
# pylint: disable=unused-argument
self.logger.error("Run must be implemented")
return TestCase.EX_RUN_ERROR
def push_to_db(self):
+ """Push the results of TestCase to the DB.
+
+ It allows publishing the results and to check the status.
+
+ It could be overriden if the common implementation is not
+ suitable. The following attributes must be set before pushing
+ the results to DB:
+ * case_name,
+ * criteria,
+ * start_time,
+ * stop_time.
+
+ Returns:
+ TestCase.EX_OK if results were pushed to DB.
+ TestCase.EX_PUSH_TO_DB_ERROR otherwise.
+ """
try:
assert self.project_name
assert self.case_name