diff options
| author | Kali Kaneko <kali@leap.se> | 2013-03-15 03:51:30 +0900 | 
|---|---|---|
| committer | Kali Kaneko <kali@leap.se> | 2013-03-15 03:51:30 +0900 | 
| commit | de1cc5864e652b7d06eda4f15143cd57d074180a (patch) | |
| tree | aa806fa8abc618a854466a12d389f52fa4b14095 | |
| parent | 8bc80ce66e182bfcbde9d98d9425fcc99ac450e6 (diff) | |
misc fixes (spacing, capitalization)
| -rw-r--r-- | src/leap/common/testing/basetest.py | 11 | ||||
| -rw-r--r-- | src/leap/common/testing/test_basetest.py | 24 | 
2 files changed, 19 insertions, 16 deletions
| diff --git a/src/leap/common/testing/basetest.py b/src/leap/common/testing/basetest.py index aa90367..65e23a9 100644 --- a/src/leap/common/testing/basetest.py +++ b/src/leap/common/testing/basetest.py @@ -41,8 +41,10 @@ class BaseLeapTest(unittest.TestCase):      @classmethod      def setUpClass(cls):          """ -        sets up common facilities -        for testing this TestCase +        Sets up common facilities for testing this TestCase: +        - custom PATH and HOME environmental variables +        - creates a temporal folder to which those point. +        It saves the old path and home vars so they can be restored later.          """          cls.old_path = os.environ['PATH']          cls.old_home = os.environ['HOME'] @@ -57,8 +59,9 @@ class BaseLeapTest(unittest.TestCase):      @classmethod      def tearDownClass(cls):          """ -        cleanup common facilities used -        for testing this TestCase +        Cleanup common facilities used for testing this TestCase: +        - restores the default PATH and HOME variables +        - removes the temporal folder          """          os.environ["PATH"] = cls.old_path          os.environ["HOME"] = cls.old_home diff --git a/src/leap/common/testing/test_basetest.py b/src/leap/common/testing/test_basetest.py index d8af1e7..c4636df 100644 --- a/src/leap/common/testing/test_basetest.py +++ b/src/leap/common/testing/test_basetest.py @@ -1,5 +1,5 @@  # -*- coding: utf-8 -*- -# leap.common.testing.test_basetest +# test_basetest.py  # Copyright (C) 2013 LEAP  #  # This program is free software: you can redistribute it and/or modify @@ -15,8 +15,7 @@  # You should have received a copy of the GNU General Public License  # along with this program.  If not, see <http://www.gnu.org/licenses/>.  """ -Unittests for base test -...becase it's oh so meta +Unittests for BaseLeapTest ...becase it's oh so meta  """  try:      import unittest2 as unittest @@ -33,12 +32,12 @@ _tempdir = None  # global for tempdir checking  class _TestCaseRunner(object):      """ -    TestCaseRunner used to run -    BaseLeapTest +    TestCaseRunner used to run BaseLeapTest      """      def run_testcase(self, testcase=None):          """ -        Runs a given testcase +        Runs a given TestCase +          @param testcase: the testcase          @type testcase: unittest.TestCase          """ @@ -59,8 +58,9 @@ class TestAbstractBaseLeapTest(unittest.TestCase, _TestCaseRunner):      TestCase for BaseLeapTest abs      """      def test_abstract_base_class(self): -        """test errors raised when setup/teardown not -        overloaded""" +        """ +        Test errors raised when setup/teardown not overloaded +        """          class _BaseTest(BaseLeapTest):              def test_dummy_method(self):                  pass @@ -79,7 +79,7 @@ class TestAbstractBaseLeapTest(unittest.TestCase, _TestCaseRunner):  class TestInitBaseLeapTest(BaseLeapTest):      """ -    testcase for testing initialization of BaseLeapTest +    TestCase for testing initialization of BaseLeapTest      """      def setUp(self): @@ -102,13 +102,13 @@ class TestInitBaseLeapTest(BaseLeapTest):  class TestCleanedBaseLeapTest(unittest.TestCase, _TestCaseRunner):      """ -    testcase for testing tempdir creation and cleanup +    TestCase for testing tempdir creation and cleanup      """      def test_tempdir_is_cleaned_after_tests(self):          """ -        test if a TestCase derived from BaseLeapTest -        creates and cleans the temporal dir +        test if a TestCase derived from BaseLeapTest creates and cleans the +        temporal dir          """          class _BaseTest(BaseLeapTest):              def setUp(self): | 
