From e251f18777a190f05127ae587caf4810be0d0228 Mon Sep 17 00:00:00 2001 From: Jean-Paul Calderone <exarkun@twistedmatrix.com> Date: Fri, 15 Nov 2019 13:06:13 -0500 Subject: [PATCH] Heck if I know how to name things --- src/_zkapauthorizer/controller.py | 9 +++++++-- src/_zkapauthorizer/tests/test_controller.py | 4 ++-- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/src/_zkapauthorizer/controller.py b/src/_zkapauthorizer/controller.py index d852606..5c387d4 100644 --- a/src/_zkapauthorizer/controller.py +++ b/src/_zkapauthorizer/controller.py @@ -163,11 +163,15 @@ class NonRedeemer(object): @implementer(IRedeemer) @attr.s -class DoubleSpentRedeemer(object): +class DoubleSpendRedeemer(object): """ - A ``DoubleSpentRedeemer`` pretends to try to redeem vouchers for ZKAPs but + A ``DoubleSpendRedeemer`` pretends to try to redeem vouchers for ZKAPs but always fails with an error indicating the voucher has already been spent. """ + @classmethod + def make(cls, section_name, node_config, announcement, reactor): + return cls() + def random_tokens_for_voucher(self, voucher, count): return dummy_random_tokens(voucher, count) @@ -495,5 +499,6 @@ def get_redeemer(plugin_name, node_config, announcement, reactor): _REDEEMERS = { u"non": NonRedeemer.make, u"dummy": DummyRedeemer.make, + u"double-spend": DoubleSpendRedeemer.make, u"ristretto": RistrettoRedeemer.make, } diff --git a/src/_zkapauthorizer/tests/test_controller.py b/src/_zkapauthorizer/tests/test_controller.py index bccd896..c357ca4 100644 --- a/src/_zkapauthorizer/tests/test_controller.py +++ b/src/_zkapauthorizer/tests/test_controller.py @@ -96,7 +96,7 @@ from ..controller import ( IRedeemer, NonRedeemer, DummyRedeemer, - DoubleSpentRedeemer, + DoubleSpendRedeemer, RistrettoRedeemer, PaymentController, AlreadySpent, @@ -191,7 +191,7 @@ class PaymentControllerTests(TestCase): ) controller = PaymentController( store, - DoubleSpentRedeemer(), + DoubleSpendRedeemer(), ) controller.redeem(voucher) -- GitLab