Provide a fallback draw query parameter to datatables views

swh/web/auth/tests/test_views.py

@@ -170,6 +170,12 @@ def test_oidc_list_bearer_tokens(client, keycloak_oidc):
         assert oidc_token.creation_date.isoformat() == tokens_data[i]["creation_date"]
 
 
+@pytest.mark.django_db
+def test_oidc_list_bearer_tokens_without_query_params(client, keycloak_oidc):
+    _generate_and_test_bearer_token(client, keycloak_oidc)
+    check_http_get_response(client, reverse("oidc-list-bearer-tokens"), status_code=200)
+
+
 def test_oidc_get_bearer_token_anonymous_user(client):
     """
     Anonymous user should be refused access with forbidden response.

swh/web/auth/views.py

@@ -83,7 +83,7 @@ def oidc_list_bearer_tokens(request: HttpRequest) -> HttpResponse:
 
     table_data: Dict[str, Any] = {}
     table_data["recordsTotal"] = len(tokens_data)
-    table_data["draw"] = int(request.GET["draw"])
+    table_data["draw"] = int(request.GET.get("draw", 1))
     table_data["data"] = tokens_data
     table_data["recordsFiltered"] = len(tokens_data)
     return JsonResponse(table_data)

swh/web/save_code_now/admin_views.py

@@ -52,7 +52,7 @@ def _datatables_origin_urls_response(request, urls_query_set):
     urls_query_set = urls_query_set.order_by(field_order)
 
     table_data = {}
-    table_data["draw"] = int(request.GET["draw"])
+    table_data["draw"] = int(request.GET.get("draw", 1))
     table_data["recordsTotal"] = urls_query_set.count()
     table_data["recordsFiltered"] = urls_query_set.count()
     length, page = datatables_pagination_params(request)

swh/web/save_code_now/tests/test_origin_save_views.py

@@ -172,3 +172,20 @@ def test_admin_origin_save_requests_csv_dump_staff(client, staff_user):
     client.force_login(staff_user)
     url = reverse("admin-origin-save-requests-csv")
     check_http_get_response(client, url, status_code=200)
+
+
+@pytest.mark.django_db
+def test_save_origin_requests_list_without_query_params(client):
+    check_http_get_response(
+        client,
+        reverse("origin-save-requests-list", url_args={"status": "all"}),
+        status_code=200,
+        content_type="application/json",
+    )
+
+
+@pytest.mark.django_db
+def test_admin_origin_save_authorized_urls_without_query_params(client, staff_user):
+    client.force_login(staff_user)
+    url = reverse("admin-origin-save-authorized-urls-list")
+    check_http_get_response(client, url, status_code=200)

swh/web/save_code_now/views.py

@@ -54,7 +54,7 @@ def origin_save_requests_list(request, status):
 
     table_data = {}
     table_data["recordsTotal"] = save_requests.count()
-    table_data["draw"] = int(request.GET["draw"])
+    table_data["draw"] = int(request.GET.get("draw", 1))
 
     search_value = request.GET.get("search[value]")