- Check that even a shared acquire() fails on a deleted lock
- Check that delete() fails on a lock you share (must own it or nothing)
These are assumptions I build on in future code, so better check for them.
Currently no code change is necessary for them to be valid.
Reviewed-by: iustinp
def testDelete(self):
self.sl.delete()
self.assertRaises(errors.LockError, self.sl.acquire)
def testDelete(self):
self.sl.delete()
self.assertRaises(errors.LockError, self.sl.acquire)
+ self.assertRaises(errors.LockError, self.sl.acquire, shared=1)
self.assertRaises(errors.LockError, self.sl.delete)
self.assertRaises(errors.LockError, self.sl.delete)
+ def testNoDeleteIfSharer(self):
+ self.sl.acquire(shared=1)
+ self.assertRaises(AssertionError, self.sl.delete)
+
def testDeletePendingSharersExclusiveDelete(self):
self.sl.acquire()
Thread(target=self._doItSharer).start()
def testDeletePendingSharersExclusiveDelete(self):
self.sl.acquire()
Thread(target=self._doItSharer).start()