From d628acc62a9a14350658cb3ba42afe47dfc02d9d Mon Sep 17 00:00:00 2001 From: Raphael Michel Date: Thu, 21 Jul 2022 09:14:38 +0200 Subject: [PATCH] Remove left-over debug statements --- src/pretix/api/serializers/orderchange.py | 1 - src/pretix/base/services/cart.py | 1 - src/pretix/control/views/orders.py | 2 -- src/tests/api/test_teams.py | 1 - src/tests/control/test_orders.py | 5 +---- src/tests/helpers/test_jsonlogic_boolalg.py | 1 - src/tests/plugins/paypal2/test_checkout.py | 1 - src/tests/plugins/sendmail/test_sendmail.py | 1 - src/tests/presale/test_event.py | 1 - 9 files changed, 1 insertion(+), 13 deletions(-) diff --git a/src/pretix/api/serializers/orderchange.py b/src/pretix/api/serializers/orderchange.py index fd66a2aea5..3574047396 100644 --- a/src/pretix/api/serializers/orderchange.py +++ b/src/pretix/api/serializers/orderchange.py @@ -396,7 +396,6 @@ class OrderChangeOperationSerializer(serializers.Serializer): def validate(self, data): seen_positions = set() for d in data.get('patch_positions', []): - print(d, seen_positions) if d['position'] in seen_positions: raise ValidationError({'patch_positions': ['You have specified the same object twice.']}) seen_positions.add(d['position']) diff --git a/src/pretix/base/services/cart.py b/src/pretix/base/services/cart.py index c945720863..965ec3886f 100644 --- a/src/pretix/base/services/cart.py +++ b/src/pretix/base/services/cart.py @@ -1241,7 +1241,6 @@ def add_items_to_cart(self, event: int, items: List[dict], cart_id: str=None, lo :raises CartError: On any error that occurred """ with language(locale): - print(locale) ia = False if invoice_address: try: diff --git a/src/pretix/control/views/orders.py b/src/pretix/control/views/orders.py index 0e4d27b83a..efb4eb8a5f 100644 --- a/src/pretix/control/views/orders.py +++ b/src/pretix/control/views/orders.py @@ -540,10 +540,8 @@ class OrderComment(OrderView): self.order.log_action('pretix.event.order.checkin_attention', user=self.request.user, data={ 'new_value': form.cleaned_data.get('checkin_attention') }) - print(self.order.custom_followup_at) self.order.save(update_fields=['checkin_attention', 'comment', 'custom_followup_at']) self.order.refresh_from_db() - print(self.order.custom_followup_at) messages.success(self.request, _('The comment has been updated.')) else: messages.error(self.request, _('Could not update the comment.')) diff --git a/src/tests/api/test_teams.py b/src/tests/api/test_teams.py index 4bb3beaf74..eb0d80d087 100644 --- a/src/tests/api/test_teams.py +++ b/src/tests/api/test_teams.py @@ -113,7 +113,6 @@ def test_team_update(token_client, organizer, event, second_team): }, format='json' ) - print(resp.data) assert resp.status_code == 400 diff --git a/src/tests/control/test_orders.py b/src/tests/control/test_orders.py index afae3acfa0..2a88e481b6 100644 --- a/src/tests/control/test_orders.py +++ b/src/tests/control/test_orders.py @@ -648,7 +648,6 @@ def test_order_reactivate(client, env): client.login(email='dummy@dummy.dummy', password='dummy') response = client.post('/control/event/dummy/dummy/orders/FOO/reactivate', { }, follow=True) - print(response.content.decode()) assert 'alert-success' in response.content.decode() with scopes_disabled(): o = Order.objects.get(id=env[2].id) @@ -1079,7 +1078,6 @@ def test_order_mark_paid_forced(client, env): 'amount': str(o.pending_sum), 'force': 'on' }, follow=True) - print(response.content.decode()) assert 'alert-success' in response.content.decode() with scopes_disabled(): o = Order.objects.get(id=env[2].id) @@ -1366,7 +1364,7 @@ class OrderChangeTests(SoupTest): date_end=self.event.date_from + timedelta(days=1), attendee_name_parts={'_scheme': 'full', 'full_name': 'John Doe'}, ) - r = self.client.post('/control/event/{}/{}/orders/{}/change'.format( + self.client.post('/control/event/{}/{}/orders/{}/change'.format( self.event.organizer.slug, self.event.slug, self.order.code ), { 'add-TOTAL_FORMS': '0', @@ -1377,7 +1375,6 @@ class OrderChangeTests(SoupTest): 'op-{}-used_membership'.format(self.op2.pk): str(m_correct1.pk), 'op-{}-used_membership'.format(self.op3.pk): str(m_correct1.pk), }, follow=True) - print(r.content) self.op1.refresh_from_db() self.order.refresh_from_db() assert self.op1.used_membership == m_correct1 diff --git a/src/tests/helpers/test_jsonlogic_boolalg.py b/src/tests/helpers/test_jsonlogic_boolalg.py index c559866717..db17bc7f14 100644 --- a/src/tests/helpers/test_jsonlogic_boolalg.py +++ b/src/tests/helpers/test_jsonlogic_boolalg.py @@ -63,7 +63,6 @@ def compare_ignoring_order(data1, data2): try: assert set(data1) == set(data2) except: - print(data1, data2) assert len(data1) == len(data2) and all(data1.count(i) == data2.count(i) for i in data1) elif isinstance(data1, dict) and isinstance(data2, dict): assert set(data1.keys()) == set(data2.keys()) diff --git a/src/tests/plugins/paypal2/test_checkout.py b/src/tests/plugins/paypal2/test_checkout.py index a27c91e096..41b71345a4 100644 --- a/src/tests/plugins/paypal2/test_checkout.py +++ b/src/tests/plugins/paypal2/test_checkout.py @@ -137,5 +137,4 @@ def test_payment(env, monkeypatch): 'payment_paypal_wallet_oid': '04F89033701558004', 'payment_paypal_wallet_payer': 'Q739JNKWH67HE', }) - print(response.content.decode()) assert response['Location'] == '/ccc/30c3/checkout/confirm/' diff --git a/src/tests/plugins/sendmail/test_sendmail.py b/src/tests/plugins/sendmail/test_sendmail.py index 0a6857e80a..cd25e0248e 100644 --- a/src/tests/plugins/sendmail/test_sendmail.py +++ b/src/tests/plugins/sendmail/test_sendmail.py @@ -440,7 +440,6 @@ def test_sendmail_attendee_checkin_filter(logged_in_client, sendmail_url, event, }, follow=True) assert response.status_code == 200 - print(response.rendered_content) assert 'alert-success' in response.rendered_content assert len(djmail.outbox) == 1 assert djmail.outbox[0].to == ['attendee1@dummy.test'] diff --git a/src/tests/presale/test_event.py b/src/tests/presale/test_event.py index 8c716c6932..9346ad38a8 100644 --- a/src/tests/presale/test_event.py +++ b/src/tests/presale/test_event.py @@ -328,7 +328,6 @@ class ItemDisplayTest(EventTestMixin, SoupTest): self.event.subevents.create(name='Foo SE2', date_from=now() + datetime.timedelta(days=12), active=True) resp = self.client.get('/%s/%s/' % (self.orga.slug, self.event.slug)) - print(resp.rendered_content) self.assertIn("Foo SE2", resp.rendered_content) self.assertNotIn("Foo SE1", resp.rendered_content) resp = self.client.get('/%s/%s/?date=%d-W%d' % (self.orga.slug, self.event.slug, se1.date_from.isocalendar()[0], se1.date_from.isocalendar()[1]))