diff --git a/src/tests/api/test_orders.py b/src/tests/api/test_orders.py index eeee636962..1213ba052d 100644 --- a/src/tests/api/test_orders.py +++ b/src/tests/api/test_orders.py @@ -382,7 +382,10 @@ def test_payment_refund_fail(token_client, organizer, event, order, monkeypatch) def test_payment_refund_success(token_client, organizer, event, order, monkeypatch): def charge_retr(*args, **kwargs): def refund_create(amount): - pass + r = MockedCharge() + r.id = 'foo' + r.status = 'succeeded' + return r c = MockedCharge() c.refunds.create = refund_create diff --git a/src/tests/base/test_models.py b/src/tests/base/test_models.py index 5c859a8611..0145f4ecbd 100644 --- a/src/tests/base/test_models.py +++ b/src/tests/base/test_models.py @@ -1019,7 +1019,7 @@ class OrderTestCase(BaseQuotaTestCase): o = Order.annotate_overpayments(Order.objects.all()).first() assert not o.is_underpaid assert not o.is_overpaid - assert not o.is_pending_with_full_payment + assert o.is_pending_with_full_payment assert not o.has_pending_refund assert not o.has_external_refund diff --git a/src/tests/plugins/stripe/test_provider.py b/src/tests/plugins/stripe/test_provider.py index f796fa96dc..4de82efd03 100644 --- a/src/tests/plugins/stripe/test_provider.py +++ b/src/tests/plugins/stripe/test_provider.py @@ -206,7 +206,10 @@ def test_refund_success(env, factory, monkeypatch): def charge_retr(*args, **kwargs): def refund_create(amount): - pass + r = MockedCharge() + r.id = 'foo' + r.status = 'succeeded' + return r c = MockedCharge() c.refunds.create = refund_create