diff options
| author | Scott Gasch <[email protected]> | 2022-08-20 19:06:09 -0700 |
|---|---|---|
| committer | Scott Gasch <[email protected]> | 2022-08-20 19:06:09 -0700 |
| commit | f38b4e71714093d2804c18a963f05e85a75a8ffe (patch) | |
| tree | 7198a35299fec80131f01734020d45bbe80115c0 /tests | |
| parent | a36aa2f6f29ee07198e07b8d2beadc2183d4ea16 (diff) | |
Make release return a bool. Improve tests and logging.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/zookeeper_test.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/zookeeper_test.py b/tests/zookeeper_test.py index 6c00314..05749c1 100644 --- a/tests/zookeeper_test.py +++ b/tests/zookeeper_test.py @@ -20,9 +20,9 @@ class TestZookeeper(unittest.TestCase): @zookeeper.obtain_lease(also_pass_lease=True, duration=datetime.timedelta(minutes=1)) def test_release_lease(self, lease: zookeeper.RenewableReleasableLease): self.assertTrue(lease) - lease.release() + self.assertTrue(lease.release()) self.assertFalse(lease) - lease.release() + self.assertFalse(lease.release()) self.assertFalse(lease) @zookeeper.obtain_lease(also_pass_lease=True, duration=datetime.timedelta(minutes=1)) @@ -30,7 +30,7 @@ class TestZookeeper(unittest.TestCase): self.assertTrue(lease) self.assertTrue(lease.try_renew(datetime.timedelta(minutes=2))) self.assertTrue(lease) - lease.release() + self.assertTrue(lease.release()) @zookeeper.obtain_lease( also_pass_lease=True, @@ -38,7 +38,7 @@ class TestZookeeper(unittest.TestCase): ) def test_cant_renew_lease_after_released(self, lease: zookeeper.RenewableReleasableLease): self.assertTrue(lease) - lease.release() + self.assertTrue(lease.release()) self.assertFalse(lease) self.assertFalse(lease.try_renew(datetime.timedelta(minutes=2))) |