summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKali Kaneko <kali@leap.se>2013-03-15 03:51:30 +0900
committerKali Kaneko <kali@leap.se>2013-03-15 03:51:30 +0900
commitde1cc5864e652b7d06eda4f15143cd57d074180a (patch)
treeaa806fa8abc618a854466a12d389f52fa4b14095
parent8bc80ce66e182bfcbde9d98d9425fcc99ac450e6 (diff)
misc fixes (spacing, capitalization)
-rw-r--r--src/leap/common/testing/basetest.py11
-rw-r--r--src/leap/common/testing/test_basetest.py24
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):