Add tests for GETting views in control/

src/pretix/control/views/orders.py

@@ -203,6 +203,9 @@ class OrderComment(OrderView):
             messages.error(self.request, _('Could not update the comment.'))
         return redirect(self.get_order_url())
 
+    def get(self, *args, **kwargs):
+        return HttpResponseNotAllowed(['POST'])
+
 
 class OrderTransition(OrderView):
     permission = 'can_change_orders'

src/tests/control/test_views.py

@@ -0,0 +1,139 @@
+import datetime
+
+import pytest
+from django.utils.timezone import now
+
+from pretix.base.models import (
+    Event, EventPermission, Item, ItemCategory, Order, OrderPosition,
+    Organizer, OrganizerPermission, Question, Quota, User, Voucher,
+)
+
+
+@pytest.fixture
+def event():
+    orga = Organizer.objects.create(name='CCC', slug='ccc')
+    return Event.objects.create(
+        organizer=orga, name='30C3', slug='30c3',
+        date_from=datetime.datetime(2013, 12, 26, tzinfo=datetime.timezone.utc),
+        plugins='pretix.plugins.banktransfer,tests.testdummy',
+    )
+
+
+@pytest.fixture
+def item(event):
+    return Item.objects.create(name='Test item', event=event, default_price=13)
+
+
+@pytest.fixture
+def item_category(event):
+    return ItemCategory.objects.create(event=event)
+
+
+@pytest.fixture
+def order(item):
+    o = Order.objects.create(event=item.event, status=Order.STATUS_PENDING,
+                             expires=now() + datetime.timedelta(hours=1),
+                             total=13, code='DUMMY', email='dummy@dummy.test',
+                             datetime=now(), payment_provider='banktransfer')
+    OrderPosition.objects.create(order=o, item=item, price=13)
+    return o
+
+
+@pytest.fixture
+def question(event):
+    return Question.objects.create(event=event, question="What is your shoe size?", type="N", required=True)
+
+
+@pytest.fixture
+def quota(event):
+    return Quota.objects.create(name="Test", size=2, event=event)
+
+
+@pytest.fixture
+def voucher(quota):
+    return Voucher.objects.create(event=quota.event, quota=quota)
+
+
+@pytest.fixture
+def logged_in_client(client, event):
+    user = User.objects.create_superuser('dummy@dummy.dummy', 'dummy')
+    OrganizerPermission.objects.create(organizer=event.organizer, user=user, can_create_events=True)
+    EventPermission.objects.create(event=event, user=user, can_change_items=True,
+                                   can_change_settings=True, can_change_orders=True, can_view_orders=True)
+    client.force_login(user)
+    return client
+
+
+@pytest.mark.parametrize('url,expected', [
+    ('/control/', 200),
+    ('/control/settings/2fa/', 302),
+    ('/control/settings/history/', 200),
+
+    ('/control/organizers/', 200),
+    ('/control/organizers/add', 200),
+    ('/control/organizer/{orga}/edit', 200),
+
+    ('/control/events/', 200),
+    ('/control/events/add', 200),
+
+    ('/control/event/{orga}/{event}/', 200),
+    ('/control/event/{orga}/{event}/live/', 200),
+    ('/control/event/{orga}/{event}/settings/', 200),
+    ('/control/event/{orga}/{event}/settings/plugins', 200),
+    ('/control/event/{orga}/{event}/settings/permissions', 200),
+    ('/control/event/{orga}/{event}/settings/payment', 200),
+    ('/control/event/{orga}/{event}/settings/tickets', 200),
+    # ('/control/event/{orga}/{event}/settings/tickets/preview/(?P<output>[^/]+)', 200),
+    ('/control/event/{orga}/{event}/settings/email', 200),
+    ('/control/event/{orga}/{event}/settings/invoice', 200),
+    ('/control/event/{orga}/{event}/settings/invoice/preview', 200),
+    ('/control/event/{orga}/{event}/settings/display', 200),
+    ('/control/event/{orga}/{event}/items/', 200),
+    ('/control/event/{orga}/{event}/items/add', 200),
+    ('/control/event/{orga}/{event}/items/{item}/', 200),
+    ('/control/event/{orga}/{event}/items/{item}/variations', 200),
+    ('/control/event/{orga}/{event}/items/{item}/delete', 200),
+    ('/control/event/{orga}/{event}/categories/', 200),
+    ('/control/event/{orga}/{event}/categories/{category}/delete', 200),
+    ('/control/event/{orga}/{event}/categories/{category}/', 200),
+    ('/control/event/{orga}/{event}/categories/add', 200),
+    ('/control/event/{orga}/{event}/questions/', 200),
+    ('/control/event/{orga}/{event}/questions/{question}/delete', 200),
+    ('/control/event/{orga}/{event}/questions/{question}/', 200),
+    ('/control/event/{orga}/{event}/questions/{question}/change', 200),
+    ('/control/event/{orga}/{event}/questions/add', 200),
+    ('/control/event/{orga}/{event}/quotas/', 200),
+    ('/control/event/{orga}/{event}/quotas/{quota}/', 200),
+    ('/control/event/{orga}/{event}/quotas/{quota}/change', 200),
+    ('/control/event/{orga}/{event}/quotas/{quota}/delete', 200),
+    ('/control/event/{orga}/{event}/quotas/add', 200),
+    ('/control/event/{orga}/{event}/vouchers/', 200),
+    ('/control/event/{orga}/{event}/vouchers/tags/', 200),
+    ('/control/event/{orga}/{event}/vouchers/rng', 200),
+    ('/control/event/{orga}/{event}/vouchers/{voucher}/', 200),
+    ('/control/event/{orga}/{event}/vouchers/{voucher}/delete', 200),
+    ('/control/event/{orga}/{event}/vouchers/add', 200),
+    ('/control/event/{orga}/{event}/vouchers/bulk_add', 200),
+    ('/control/event/{orga}/{event}/orders/{order_code}/extend', 200),
+    ('/control/event/{orga}/{event}/orders/{order_code}/contact', 200),
+    ('/control/event/{orga}/{event}/orders/{order_code}/comment', 405),
+    ('/control/event/{orga}/{event}/orders/{order_code}/change', 200),
+    ('/control/event/{orga}/{event}/orders/{order_code}/', 200),
+    ('/control/event/{orga}/{event}/orders/overview/', 200),
+    ('/control/event/{orga}/{event}/orders/export/', 200),
+    ('/control/event/{orga}/{event}/orders/go', 302),
+    ('/control/event/{orga}/{event}/orders/', 200),
+])
+@pytest.mark.django_db
+def test_one_view(logged_in_client, url, expected, event, item, item_category, order, question, quota, voucher):
+    url = url.format(
+        event=event.slug, orga=event.organizer.slug,
+        category=item_category.pk,
+        item=item.pk,
+        order_code=order.code,
+        question=question.pk,
+        quota=quota.pk,
+        voucher=voucher.pk,
+    )
+    response = logged_in_client.get(url)
+    assert response.status_code == expected