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): |