From de1cc5864e652b7d06eda4f15143cd57d074180a Mon Sep 17 00:00:00 2001
From: Kali Kaneko <kali@leap.se>
Date: Fri, 15 Mar 2013 03:51:30 +0900
Subject: misc fixes (spacing, capitalization)

---
 src/leap/common/testing/basetest.py      | 11 +++++++----
 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):
-- 
cgit v1.2.3