Changeset 5739

Timestamp:

12 Feb 2011, 15:33:43 (14 years ago)

Author:

uli

Message:

Add tests for removeFileOrDirectory. Here we also use unittests
instead of doctests in docstrings, because it would not be very
interesting to developers to see how file removal works. We can tell
that this function removes files and developers will know what that
means even without examples.

The unittest, however, checks that the funtion really removes files
when called.

Test coverage for waeup.sirp.utils.helpers now at 100% :)

File:

• main/waeup.sirp/trunk/src/waeup/sirp/utils/tests/test_helpers.py (modified) (2 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/utils/tests/test_helpers.py

@@ -33 +33 @@
 class FakeObject(object):
     implements(IFakeObject)
+
+class RemoveFileOrDirectoryTestCase(unittest.TestCase):
+
+    def setUp(self):
+        self.dirpath = tempfile.mkdtemp()
+        self.filepath = os.path.join(self.dirpath, 'somefile')
+        self.non_file = os.path.join(self.dirpath, 'nonfile')
+        open(self.filepath, 'wb').write('Hi!')
+        return
+
+    def tearDown(self):
+        if os.path.exists(self.dirpath):
+            shutil.rmtree(self.dirpath)
+        return
+
+    def test_handle_not_existing_path(self):
+        result = helpers.removeFileOrDirectory(self.non_file)
+        self.assertTrue(result is None)
+        return
+
+    def test_handle_dir(self):
+        helpers.removeFileOrDirectory(self.dirpath)
+        self.assertFalse(
+            os.path.exists(self.dirpath)
+            )
+        return
+
+    def test_handle_file(self):
+        helpers.removeFileOrDirectory(self.filepath)
+        self.assertFalse(
+            os.path.exists(self.filepath)
+            )
+        return
 
 class CopyFileSystemTreeTestCase(unittest.TestCase):
@@ -117 +150 @@
         FactoryBaseTestCase,
         CopyFileSystemTreeTestCase,
+        RemoveFileOrDirectoryTestCase,
         ]:
         suite.addTests(