lockutils: add a failing unit test

Add a unit test which fails because of #1107950 - in this case,
lockutils is deleting the lock directory every time the lock is dropped
even though another process might have acquired a lock using that
directory.

Change-Id: Ic82409f9462e570bc102ab469334c53aafc6d7ac

1 files changed, 22 insertions, 3 deletions

diff --git a/tests/unit/test_lockutils.py b/tests/unit/test_lockutils.py
index e1f0375..ae5dca7 100644
--- a/tests/unit/test_lockutils.py
+++ b/tests/unit/test_lockutils.py
@@ -125,10 +125,8 @@ class LockTestCase(utils.BaseTestCase):
             if os.path.exists(tempdir):
                 shutil.rmtree(tempdir)
 
-    def test_synchronized_externally(self):
+    def _do_test_synchronized_externally(self):
         """We can lock across multiple processes"""
-        lock_dir = tempfile.mkdtemp()
-        self.config(lock_path=lock_dir)
 
         @lockutils.synchronized('external', 'test-', external=True)
         def lock_files(handles_dir):
@@ -176,5 +174,26 @@ class LockTestCase(utils.BaseTestCase):
         finally:
             if os.path.exists(handles_dir):
                 shutil.rmtree(handles_dir, ignore_errors=True)
+
+    def test_synchronized_externally(self):
+        lock_dir = tempfile.mkdtemp()
+        self.config(lock_path=lock_dir)
+
+        try:
+            self._do_test_synchronized_externally()
+        finally:
+            if os.path.exists(lock_dir):
+                shutil.rmtree(lock_dir, ignore_errors=True)
+
+    def test_synchronized_externally_lock_dir_not_exist(self):
+        self.skipTest('bug #1107950')
+
+        lock_dir = tempfile.mkdtemp()
+        os.rmdir(lock_dir)
+        self.config(lock_path=lock_dir)
+
+        try:
+            self._do_test_synchronized_externally()
+        finally:
             if os.path.exists(lock_dir):
                 shutil.rmtree(lock_dir, ignore_errors=True)