From 98d7c2b638f73711a37042a7ff9e2e054342c55b Mon Sep 17 00:00:00 2001
From: Jean-Paul Calderone <exarkun@twistedmatrix.com>
Date: Mon, 8 Nov 2021 13:38:23 -0500
Subject: [PATCH] add a controller test for vouchers that should end up in the
error state
---
src/_zkapauthorizer/tests/test_controller.py | 39 ++++++++++++++++++++
1 file changed, 39 insertions(+)
diff --git a/src/_zkapauthorizer/tests/test_controller.py b/src/_zkapauthorizer/tests/test_controller.py
index c063eb0..e7eceb4 100644
--- a/src/_zkapauthorizer/tests/test_controller.py
+++ b/src/_zkapauthorizer/tests/test_controller.py
@@ -59,6 +59,7 @@ from zope.interface import implementer
from ..controller import (
AlreadySpent,
DoubleSpendRedeemer,
+ ErrorRedeemer,
DummyRedeemer,
IndexedRedeemer,
IRedeemer,
@@ -71,6 +72,7 @@ from ..controller import (
UnpaidRedeemer,
token_count_for_group,
)
+from ..model import Error as model_Error
from ..model import DoubleSpend as model_DoubleSpend
from ..model import Pending as model_Pending
from ..model import Redeemed as model_Redeemed
@@ -425,6 +427,43 @@ class PaymentControllerTests(TestCase):
),
)
+ @given(
+ tahoe_configs(),
+ datetimes(),
+ vouchers(),
+ )
+ def test_error_state(self, get_config, now, voucher):
+ """
+ If ``IRedeemer.redeem`` fails with an unrecognized exception then the
+ voucher is put into the error state.
+ """
+ details = u"these are the reasons it broke"
+ store = self.useFixture(TemporaryVoucherStore(get_config, lambda: now)).store
+ controller = PaymentController(
+ store,
+ ErrorRedeemer(details),
+ default_token_count=100,
+ allowed_public_keys=set(),
+ clock=Clock(),
+ )
+ self.assertThat(
+ controller.redeem(voucher),
+ succeeded(Always()),
+ )
+
+ persisted_voucher = controller.get_voucher(voucher)
+ self.assertThat(
+ persisted_voucher,
+ MatchesStructure(
+ state=Equals(
+ model_Error(
+ finished=now,
+ details=details,
+ )
+ ),
+ ),
+ )
+
@given(tahoe_configs(), datetimes(), vouchers())
def test_double_spent_after_double_spend(self, get_config, now, voucher):
"""
--
GitLab