feat: Site status message

ietf/status/tests.py

@@ -98,7 +98,6 @@ def test_status_page(self):
         status.save()
 
         r = self.client.get(f'/status/{slug}/')
-        # with a Status it should redirect
         self.assertEqual(r.status_code, 200)
 
         status.delete()
@@ -115,7 +114,6 @@ def test_status_page(self):
         status.save()
 
         r = self.client.get(f'/status/{slug}/')
-        # with a Status it should redirect
         self.assertEqual(r.status_code, 200)
         self.assertContains(r, test_string)
 

ietf/status/views.py

@@ -1,15 +1,15 @@
 # Copyright The IETF Trust 2016-2024, All Rights Reserved
 # -*- coding: utf-8 -*-
 
-import json
-from django.http import HttpResponse, HttpResponseRedirect, HttpResponseNotFound
+from django.urls import reverse as urlreverse
+from django.http import HttpResponseRedirect, HttpResponseNotFound, JsonResponse
 from ietf.utils import markdown
 from django.shortcuts import render, get_object_or_404
 from ietf.status.models import Status
 
 import debug                            # pyflakes:ignore
 
-def get_context_data():
+def get_last_active_status():
     status = Status.objects.filter(active=True).order_by("-date").first()
     if status is None:
         return { "hasMessage": False }
@@ -20,13 +20,13 @@ def get_context_data():
         "slug": status.slug,
         "title": status.title,
         "body": status.body,
-        "url": f"/status/{status.slug}",
+        "url": urlreverse('ietf.status.views.status_page', kwargs={ 'slug': status.slug }),
         "date": status.date.isoformat()
     }
     return context
 
 def status_latest_html(request):
-    return render(request, "status/latest.html", context=get_context_data())
+    return render(request, "status/latest.html", context=get_last_active_status())
 
 def status_page(request, slug):
     sanitised_slug = slug.rstrip("/")
@@ -37,10 +37,10 @@ def status_page(request, slug):
     })
 
 def status_latest_json(request):
-    return HttpResponse(json.dumps(get_context_data()), status=200, content_type='application/json')
+    return JsonResponse(get_last_active_status())
 
 def status_latest_redirect(request):
-    context = get_context_data()
+    context = get_last_active_status()
     if context["hasMessage"] == True:
         return HttpResponseRedirect(context["url"])
     return HttpResponseNotFound()

ietf/utils/test_runner.py

@@ -566,6 +566,14 @@ def ignore_pattern(regex, pattern):
                         covered.add(regex)
                         break
 
+            uncovered = set()
+            for regex, compiled, obj in patterns:
+                if regex not in covered:
+                    uncovered.add(regex)
+
+            if(len(uncovered) > 0):
+                print("Uncovered URLs", uncovered)
+
             self.runner.coverage_data["url"] = {
                 "coverage": 1.0*len(covered)/len(patterns),
                 "covered": dict( (k, (o.lookup_str, k in covered)) for k,p,o in patterns ),