From a0c214012afe9dde6bffe655e4d3eca11e47a254 Mon Sep 17 00:00:00 2001
From: Jean-Paul Calderone <exarkun@twistedmatrix.com>
Date: Fri, 1 May 2020 15:29:46 -0400
Subject: [PATCH] Add some more leases to the share stat test
This makes the test a little more challenging by adding some more bookkeeping
overhead that needs to be accounted for.
---
src/_zkapauthorizer/tests/test_storage_protocol.py | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/src/_zkapauthorizer/tests/test_storage_protocol.py b/src/_zkapauthorizer/tests/test_storage_protocol.py
index f2b9b68..55a0fb9 100644
--- a/src/_zkapauthorizer/tests/test_storage_protocol.py
+++ b/src/_zkapauthorizer/tests/test_storage_protocol.py
@@ -360,8 +360,9 @@ class ShareTests(TestCase):
sharenum=sharenums(),
size=sizes(),
clock=clocks(),
+ leases=lists(lease_renew_secrets(), unique=True),
)
- def test_stat_shares_immutable(self, storage_index, renew_secret, cancel_secret, sharenum, size, clock):
+ def test_stat_shares_immutable(self, storage_index, renew_secret, cancel_secret, sharenum, size, clock, leases):
"""
Size and lease information about immutable shares can be retrieved from a
storage server.
@@ -385,6 +386,14 @@ class ShareTests(TestCase):
size,
canary=self.canary,
)
+ # Perhaps put some more leases on it. Leases might impact our
+ # ability to determine share data size.
+ for renew_secret in leases:
+ self.anonymous_storage_server.remote_add_lease(
+ storage_index,
+ renew_secret,
+ b"",
+ )
finally:
patch.cleanUp()
--
GitLab