arthexis 0.1.18__py3-none-any.whl → 0.1.20__py3-none-any.whl

pages/tests.py

@@ -3,12 +3,13 @@ import os
 os.environ.setdefault("DJANGO_SETTINGS_MODULE", "config.settings")
 
 import django
+import pytest
 
 django.setup()
 
 from django.test import Client, RequestFactory, TestCase, SimpleTestCase, override_settings
 from django.test.utils import CaptureQueriesContext
-from django.urls import NoReverseMatch, reverse
+from django.urls import reverse
 from django.templatetags.static import static
 from urllib.parse import quote
 from django.contrib.auth import get_user_model
@@ -33,6 +34,8 @@ from pages.models import (
     UserManual,
     UserStory,
 )
+from django.http import FileResponse
+
 from pages.admin import (
     ApplicationAdmin,
     UserManualAdmin,
@@ -506,6 +509,7 @@ class InvitationTests(TestCase):
         lead = InviteLead.objects.get()
         self.assertEqual(lead.mac_address, "aa:bb:cc:dd:ee:ff")
 
+    @pytest.mark.feature("ap-router")
     @patch("pages.views.public_wifi.grant_public_access")
     @patch(
         "pages.views.public_wifi.resolve_mac_address",
@@ -690,23 +694,6 @@ class AdminDashboardAppListTests(TestCase):
         resp = self.client.get(reverse("admin:index"))
         self.assertContains(resp, "5. Horologia MODELS")
 
-    def test_dashboard_handles_missing_last_net_message_url(self):
-        from pages.templatetags import admin_extras
-
-        real_reverse = admin_extras.reverse
-
-        def fake_reverse(name, *args, **kwargs):
-            if name == "last-net-message":
-                raise NoReverseMatch("missing")
-            return real_reverse(name, *args, **kwargs)
-
-        with patch("pages.templatetags.admin_extras.reverse", side_effect=fake_reverse):
-            resp = self.client.get(reverse("admin:index"))
-
-        self.assertEqual(resp.status_code, 200)
-        self.assertNotIn(b"last-net-message", resp.content)
-
-
 class AdminSidebarTests(TestCase):
     def setUp(self):
         self.client = Client()
@@ -816,7 +803,8 @@ class ViewHistoryLoggingTests(TestCase):
         )
         landing = module.landings.get(path="/")
         landing.label = "Home Landing"
-        landing.save(update_fields=["label"])
+        landing.track_leads = True
+        landing.save(update_fields=["label", "track_leads"])
 
         resp = self.client.get(
             reverse("pages:index"), HTTP_REFERER="https://example.com/ref"
@@ -841,7 +829,8 @@ class ViewHistoryLoggingTests(TestCase):
         )
         landing = module.landings.get(path="/")
         landing.label = "No Celery"
-        landing.save(update_fields=["label"])
+        landing.track_leads = True
+        landing.save(update_fields=["label", "track_leads"])
 
         resp = self.client.get(reverse("pages:index"))
 
@@ -861,7 +850,8 @@ class ViewHistoryLoggingTests(TestCase):
         )
         landing = module.landings.get(path="/")
         landing.enabled = False
-        landing.save(update_fields=["enabled"])
+        landing.track_leads = True
+        landing.save(update_fields=["enabled", "track_leads"])
 
         resp = self.client.get(reverse("pages:index"))
 
@@ -1130,6 +1120,50 @@ class LogViewerAdminTests(SimpleTestCase):
         self.assertEqual(context["selected_log"], "selected.log")
         self.assertIn("hello world", context["log_content"])
 
+    def test_log_viewer_applies_line_limit(self):
+        content = "\n".join(f"line {i}" for i in range(50))
+        self._create_log("limited.log", content)
+        response = self._render({"log": "limited.log", "limit": "20"})
+        context = response.context_data
+        self.assertEqual(context["log_limit_choice"], "20")
+        self.assertIn("line 49", context["log_content"])
+        self.assertIn("line 30", context["log_content"])
+        self.assertNotIn("line 29", context["log_content"])
+
+    def test_log_viewer_all_limit_returns_full_log(self):
+        content = "first\nsecond\nthird"
+        self._create_log("all.log", content)
+        response = self._render({"log": "all.log", "limit": "all"})
+        context = response.context_data
+        self.assertEqual(context["log_limit_choice"], "all")
+        self.assertIn("first", context["log_content"])
+        self.assertIn("second", context["log_content"])
+
+    def test_log_viewer_invalid_limit_defaults_to_20(self):
+        content = "\n".join(f"item {i}" for i in range(5))
+        self._create_log("invalid-limit.log", content)
+        response = self._render({"log": "invalid-limit.log", "limit": "oops"})
+        context = response.context_data
+        self.assertEqual(context["log_limit_choice"], "20")
+
+    def test_log_viewer_downloads_selected_log(self):
+        self._create_log("download.log", "downloadable content")
+        request = self._build_request({"log": "download.log", "download": "1"})
+        context = {
+            "site_title": "Constellation",
+            "site_header": "Constellation",
+            "site_url": "/",
+            "available_apps": [],
+        }
+        with patch("pages.admin.admin.site.each_context", return_value=context), patch(
+            "pages.context_processors.get_site", return_value=None
+        ):
+            response = log_viewer(request)
+        self.assertIsInstance(response, FileResponse)
+        self.assertIn("attachment", response["Content-Disposition"])
+        content = b"".join(response.streaming_content).decode()
+        self.assertIn("downloadable content", content)
+
     def test_log_viewer_reports_missing_log(self):
         response = self._render({"log": "missing.log"})
         self.assertIn("requested log could not be found", response.context_data["log_error"])
@@ -1328,6 +1362,7 @@ class SiteAdminRegisterCurrentTests(TestCase):
         self.assertEqual(site.name, "")
 
 
+@pytest.mark.feature("screenshot-poll")
 class SiteAdminScreenshotTests(TestCase):
     def setUp(self):
         self.client = Client()
@@ -1447,17 +1482,17 @@ class NavAppsTests(TestCase):
         )
         app = Application.objects.create(name="Readme")
         Module.objects.create(
-            node_role=role, application=app, path="/", is_default=True
+            node_role=role, application=app, path="/", is_default=True, menu="Cookbooks"
         )
 
     def test_nav_pill_renders(self):
         resp = self.client.get(reverse("pages:index"))
-        self.assertContains(resp, "README")
+        self.assertContains(resp, "COOKBOOKS")
         self.assertContains(resp, "badge rounded-pill")
 
     def test_nav_pill_renders_with_port(self):
         resp = self.client.get(reverse("pages:index"), HTTP_HOST="127.0.0.1:8000")
-        self.assertContains(resp, "README")
+        self.assertContains(resp, "COOKBOOKS")
 
     def test_nav_pill_uses_menu_field(self):
         site_app = Module.objects.get()
@@ -1465,7 +1500,7 @@ class NavAppsTests(TestCase):
         site_app.save()
         resp = self.client.get(reverse("pages:index"))
         self.assertContains(resp, 'badge rounded-pill text-bg-secondary">DOCS')
-        self.assertNotContains(resp, 'badge rounded-pill text-bg-secondary">README')
+        self.assertNotContains(resp, 'badge rounded-pill text-bg-secondary">COOKBOOKS')
 
     def test_app_without_root_url_excluded(self):
         role = NodeRole.objects.get(name="Terminal")
@@ -1530,20 +1565,22 @@ class RoleLandingRedirectTests(TestCase):
 
     def test_satellite_redirects_to_dashboard(self):
         target = self._configure_role_landing(
-            "Satellite", "/ocpp/", "CPMS Online Dashboard"
+            "Satellite", "/ocpp/cpms/dashboard/", "CPMS Online Dashboard"
         )
         resp = self.client.get(reverse("pages:index"))
         self.assertRedirects(resp, target, fetch_redirect_response=False)
 
     def test_control_redirects_to_rfid(self):
         target = self._configure_role_landing(
-            "Control", "/ocpp/rfid/", "RFID Tag Validator"
+            "Control", "/ocpp/rfid/validator/", "RFID Tag Validator"
         )
         resp = self.client.get(reverse("pages:index"))
         self.assertRedirects(resp, target, fetch_redirect_response=False)
 
     def test_security_group_redirect_takes_priority(self):
-        self._configure_role_landing("Control", "/ocpp/rfid/", "RFID Tag Validator")
+        self._configure_role_landing(
+            "Control", "/ocpp/rfid/validator/", "RFID Tag Validator"
+        )
         role = self.node.role
         group = SecurityGroup.objects.create(name="Operators")
         group_landing = self._ensure_landing(role, "/ocpp/group/", "Group Landing")
@@ -1560,7 +1597,9 @@ class RoleLandingRedirectTests(TestCase):
         )
 
     def test_user_redirect_overrides_group_with_higher_priority(self):
-        self._configure_role_landing("Control", "/ocpp/rfid/", "RFID Tag Validator")
+        self._configure_role_landing(
+            "Control", "/ocpp/rfid/validator/", "RFID Tag Validator"
+        )
         role = self.node.role
         group = SecurityGroup.objects.create(name="Operators")
         group_landing = self._ensure_landing(role, "/ocpp/group/", "Group Landing")
@@ -1582,10 +1621,10 @@ class RoleLandingRedirectTests(TestCase):
         )
 
 
-class ConstellationNavTests(TestCase):
+class WatchtowerNavTests(TestCase):
     def setUp(self):
         self.client = Client()
-        role, _ = NodeRole.objects.get_or_create(name="Constellation")
+        role, _ = NodeRole.objects.get_or_create(name="Watchtower")
         Node.objects.update_or_create(
             mac_address=Node.get_current_mac(),
             defaults={
@@ -1595,38 +1634,56 @@ class ConstellationNavTests(TestCase):
             },
         )
         Site.objects.update_or_create(
-            id=1, defaults={"domain": "testserver", "name": ""}
+            id=1, defaults={"domain": "arthexis.com", "name": "Arthexis"}
         )
         fixtures = [
             Path(
                 settings.BASE_DIR,
                 "pages",
                 "fixtures",
-                "constellation__application_ocpp.json",
+                "default__application_pages.json",
+            ),
+            Path(
+                settings.BASE_DIR,
+                "pages",
+                "fixtures",
+                "watchtower__application_ocpp.json",
             ),
             Path(
                 settings.BASE_DIR,
                 "pages",
                 "fixtures",
-                "constellation__module_ocpp.json",
+                "watchtower__module_ocpp.json",
             ),
             Path(
                 settings.BASE_DIR,
                 "pages",
                 "fixtures",
-                "constellation__landing_ocpp_dashboard.json",
+                "watchtower__landing_ocpp_dashboard.json",
             ),
             Path(
                 settings.BASE_DIR,
                 "pages",
                 "fixtures",
-                "constellation__landing_ocpp_cp_simulator.json",
+                "watchtower__landing_ocpp_cp_simulator.json",
             ),
             Path(
                 settings.BASE_DIR,
                 "pages",
                 "fixtures",
-                "constellation__landing_ocpp_rfid.json",
+                "watchtower__landing_ocpp_rfid.json",
+            ),
+            Path(
+                settings.BASE_DIR,
+                "pages",
+                "fixtures",
+                "watchtower__module_readme.json",
+            ),
+            Path(
+                settings.BASE_DIR,
+                "pages",
+                "fixtures",
+                "watchtower__landing_readme.json",
             ),
         ]
         call_command("loaddata", *map(str, fixtures))
@@ -1639,13 +1696,13 @@ class ConstellationNavTests(TestCase):
         self.assertNotIn("RFID", nav_labels)
         self.assertTrue(
             Module.objects.filter(
-                path="/ocpp/", node_role__name="Constellation"
+                path="/ocpp/", node_role__name="Watchtower"
             ).exists()
         )
         self.assertFalse(
             Module.objects.filter(
                 path="/ocpp/rfid/",
-                node_role__name="Constellation",
+                node_role__name="Watchtower",
                 is_deleted=False,
             ).exists()
         )
@@ -1657,9 +1714,16 @@ class ConstellationNavTests(TestCase):
         landing_labels = [landing.label for landing in ocpp_module.enabled_landings]
         self.assertIn("RFID Tag Validator", landing_labels)
 
+    @override_settings(ALLOWED_HOSTS=["testserver", "arthexis.com"])
+    def test_cookbooks_pill_visible_for_arthexis(self):
+        resp = self.client.get(
+            reverse("pages:index"), HTTP_HOST="arthexis.com"
+        )
+        self.assertContains(resp, 'badge rounded-pill text-bg-secondary">COOKBOOKS')
+
     def test_ocpp_dashboard_visible(self):
         resp = self.client.get(reverse("pages:index"))
-        self.assertContains(resp, 'href="/ocpp/"')
+        self.assertContains(resp, 'href="/ocpp/cpms/dashboard/"')
 
 
 class ReleaseModuleNavTests(TestCase):
@@ -1801,7 +1865,7 @@ class ControlNavTests(TestCase):
         self.client.force_login(user)
         resp = self.client.get(reverse("pages:index"))
         self.assertEqual(resp.status_code, 200)
-        self.assertContains(resp, 'href="/ocpp/"')
+        self.assertContains(resp, 'href="/ocpp/cpms/dashboard/"')
         self.assertContains(
             resp, 'badge rounded-pill text-bg-secondary">CHARGERS'
         )
@@ -1833,10 +1897,76 @@ class ControlNavTests(TestCase):
         self.assertFalse(resp.context["header_references"])
         self.assertNotContains(resp, "https://example.com/hidden")
 
+    def test_header_link_hidden_when_only_site_matches(self):
+        terminal_role, _ = NodeRole.objects.get_or_create(name="Terminal")
+        site = Site.objects.get(domain="testserver")
+        reference = Reference.objects.create(
+            alt_text="Restricted",
+            value="https://example.com/restricted",
+            show_in_header=True,
+        )
+        reference.roles.add(terminal_role)
+        reference.sites.add(site)
+
+        resp = self.client.get(reverse("pages:index"))
+
+        self.assertIn("header_references", resp.context)
+        self.assertFalse(resp.context["header_references"])
+        self.assertNotContains(resp, "https://example.com/restricted")
+
     def test_readme_pill_visible(self):
         resp = self.client.get(reverse("pages:readme"))
-        self.assertContains(resp, 'href="/readme/"')
-        self.assertContains(resp, 'badge rounded-pill text-bg-secondary">README')
+        self.assertContains(resp, 'href="/read/"')
+        self.assertContains(resp, 'badge rounded-pill text-bg-secondary">COOKBOOKS')
+
+    def test_cookbook_pill_has_no_dropdown(self):
+        module = Module.objects.get(node_role__name="Control", path="/read/")
+        Landing.objects.create(
+            module=module,
+            path="/man/",
+            label="Manuals",
+            enabled=True,
+        )
+
+        resp = self.client.get(reverse("pages:readme"))
+
+        self.assertContains(
+            resp,
+            '<a class="nav-link" href="/read/"><span class="badge rounded-pill text-bg-secondary">COOKBOOKS</span></a>',
+            html=True,
+        )
+        self.assertNotContains(resp, 'dropdown-item" href="/man/"')
+
+    def test_readme_page_includes_qr_share(self):
+        resp = self.client.get(reverse("pages:readme"), {"section": "intro"})
+        self.assertContains(resp, 'id="reader-qr"')
+        self.assertContains(
+            resp,
+            'data-url="http://testserver/read/?section=intro"',
+        )
+        self.assertNotContains(resp, "Scan this page")
+        self.assertNotContains(
+            resp, 'class="small text-break text-muted mt-3 mb-0"'
+        )
+
+    def test_readme_document_by_name(self):
+        resp = self.client.get(reverse("pages:readme-document", args=["AGENTS.md"]))
+        self.assertEqual(resp.status_code, 200)
+        self.assertContains(resp, "Agent Guidelines")
+
+    def test_readme_document_by_relative_path(self):
+        resp = self.client.get(
+            reverse(
+                "pages:readme-document",
+                args=["docs/development/maintenance-roadmap.md"],
+            )
+        )
+        self.assertEqual(resp.status_code, 200)
+        self.assertContains(resp, "Maintenance Improvement Proposals")
+
+    def test_readme_document_rejects_traversal(self):
+        resp = self.client.get("/read/../../SECRET.md")
+        self.assertEqual(resp.status_code, 404)
 
 
 class SatelliteNavTests(TestCase):
@@ -1908,8 +2038,8 @@ class SatelliteNavTests(TestCase):
 
     def test_readme_pill_visible(self):
         resp = self.client.get(reverse("pages:readme"))
-        self.assertContains(resp, 'href="/readme/"')
-        self.assertContains(resp, 'badge rounded-pill text-bg-secondary">README')
+        self.assertContains(resp, 'href="/read/"')
+        self.assertContains(resp, 'badge rounded-pill text-bg-secondary">COOKBOOKS')
 
 
 class PowerNavTests(TestCase):
@@ -1944,9 +2074,9 @@ class PowerNavTests(TestCase):
                 power_module = module
                 break
         self.assertIsNotNone(power_module)
-        self.assertEqual(power_module.menu_label.upper(), "CALCULATE")
+        self.assertEqual(power_module.menu_label.upper(), "CALCULATORS")
         landing_labels = {landing.label for landing in power_module.enabled_landings}
-        self.assertIn("AWG Calculator", landing_labels)
+        self.assertIn("AWG Cable Calculator", landing_labels)
 
     def test_manual_pill_label(self):
         resp = self.client.get(reverse("pages:index"))
@@ -1970,9 +2100,26 @@ class PowerNavTests(TestCase):
                 break
         self.assertIsNotNone(power_module)
         landing_labels = {landing.label for landing in power_module.enabled_landings}
-        self.assertIn("AWG Calculator", landing_labels)
+        self.assertIn("AWG Cable Calculator", landing_labels)
         self.assertIn("Energy Tariff Calculator", landing_labels)
 
+    def test_locked_landing_shows_lock_icon(self):
+        resp = self.client.get(reverse("pages:index"))
+        html = resp.content.decode()
+        energy_index = html.find("Energy Tariff Calculator")
+        self.assertGreaterEqual(energy_index, 0)
+        icon_index = html.find("dropdown-lock-icon", energy_index, energy_index + 300)
+        self.assertGreaterEqual(icon_index, 0)
+
+    def test_lock_icon_disappears_after_login(self):
+        self.client.force_login(self.user)
+        resp = self.client.get(reverse("pages:index"))
+        html = resp.content.decode()
+        energy_index = html.find("Energy Tariff Calculator")
+        self.assertGreaterEqual(energy_index, 0)
+        icon_index = html.find("dropdown-lock-icon", energy_index, energy_index + 300)
+        self.assertEqual(icon_index, -1)
+
 
 class StaffNavVisibilityTests(TestCase):
     def setUp(self):
@@ -1994,12 +2141,12 @@ class StaffNavVisibilityTests(TestCase):
     def test_nonstaff_pill_hidden(self):
         self.client.login(username="user", password="pw")
         resp = self.client.get(reverse("pages:index"))
-        self.assertContains(resp, 'href="/ocpp/"')
+        self.assertContains(resp, 'href="/ocpp/cpms/dashboard/"')
 
     def test_staff_sees_pill(self):
         self.client.login(username="staff", password="pw")
         resp = self.client.get(reverse("pages:index"))
-        self.assertContains(resp, 'href="/ocpp/"')
+        self.assertContains(resp, 'href="/ocpp/cpms/dashboard/"')
 
 
 class ModuleAdminReloadActionTests(TestCase):
@@ -2012,7 +2159,7 @@ class ModuleAdminReloadActionTests(TestCase):
             password="pw",
         )
         self.client.force_login(self.superuser)
-        self.role, _ = NodeRole.objects.get_or_create(name="Constellation")
+        self.role, _ = NodeRole.objects.get_or_create(name="Watchtower")
         Application.objects.get_or_create(name="ocpp")
         Application.objects.get_or_create(name="awg")
         Site.objects.update_or_create(
@@ -2050,7 +2197,11 @@ class ModuleAdminReloadActionTests(TestCase):
         )
         self.assertSetEqual(
             charger_landings,
-            {"/ocpp/", "/ocpp/simulator/", "/ocpp/rfid/"},
+            {
+                "/ocpp/cpms/dashboard/",
+                "/ocpp/evcs/simulator/",
+                "/ocpp/rfid/validator/",
+            },
         )
 
         calculator_landings = set(
@@ -2188,6 +2339,47 @@ class UserManualAdminFormTests(TestCase):
         self.assertEqual(form.cleaned_data["content_pdf"], self.manual.content_pdf)
 
 
+class UserManualModelTests(TestCase):
+    def _build_manual(self, **overrides):
+        defaults = {
+            "slug": "manual-model-test",
+            "title": "Manual Model",
+            "description": "Manual description",
+            "languages": "en",
+            "content_html": "<p>Manual</p>",
+            "content_pdf": base64.b64encode(b"initial").decode("ascii"),
+        }
+        defaults.update(overrides)
+        return UserManual(**defaults)
+
+    def test_save_encodes_uploaded_file(self):
+        upload = SimpleUploadedFile("manual.pdf", b"PDF data")
+        manual = self._build_manual(slug="manual-upload", content_pdf=upload)
+        manual.save()
+        manual.refresh_from_db()
+        self.assertEqual(
+            manual.content_pdf,
+            base64.b64encode(b"PDF data").decode("ascii"),
+        )
+
+    def test_save_encodes_raw_bytes(self):
+        manual = self._build_manual(slug="manual-bytes", content_pdf=b"PDF raw")
+        manual.save()
+        manual.refresh_from_db()
+        self.assertEqual(
+            manual.content_pdf,
+            base64.b64encode(b"PDF raw").decode("ascii"),
+        )
+
+    def test_save_strips_data_uri_prefix(self):
+        encoded = base64.b64encode(b"PDF data").decode("ascii")
+        data_uri = f"data:application/pdf;base64,{encoded}"
+        manual = self._build_manual(slug="manual-data-uri", content_pdf=data_uri)
+        manual.save()
+        manual.refresh_from_db()
+        self.assertEqual(manual.content_pdf, encoded)
+
+
 class LandingCreationTests(TestCase):
     def setUp(self):
         role, _ = NodeRole.objects.get_or_create(name="Terminal")
@@ -2209,12 +2401,12 @@ class LandingCreationTests(TestCase):
 
 
 class LandingFixtureTests(TestCase):
-    def test_constellation_fixture_loads_without_duplicates(self):
+    def test_watchtower_fixture_loads_without_duplicates(self):
         from glob import glob
 
-        NodeRole.objects.get_or_create(name="Constellation")
+        NodeRole.objects.get_or_create(name="Watchtower")
         fixtures = glob(
-            str(Path(settings.BASE_DIR, "pages", "fixtures", "constellation__*.json"))
+            str(Path(settings.BASE_DIR, "pages", "fixtures", "watchtower__*.json"))
         )
         fixtures = sorted(
             fixtures,
@@ -2224,9 +2416,11 @@ class LandingFixtureTests(TestCase):
         )
         call_command("loaddata", *fixtures)
         call_command("loaddata", *fixtures)
-        module = Module.objects.get(path="/ocpp/", node_role__name="Constellation")
+        module = Module.objects.get(path="/ocpp/", node_role__name="Watchtower")
         module.create_landings()
-        self.assertEqual(module.landings.filter(path="/ocpp/rfid/").count(), 1)
+        self.assertEqual(
+            module.landings.filter(path="/ocpp/rfid/validator/").count(), 1
+        )
 
 
 class AllowedHostSubnetTests(TestCase):
@@ -2379,9 +2573,9 @@ class FaviconTests(TestCase):
             )
             self.assertContains(resp, b64)
 
-    def test_constellation_nodes_use_goldenrod_favicon(self):
+    def test_watchtower_nodes_use_goldenrod_favicon(self):
         with override_settings(MEDIA_ROOT=self.tmpdir):
-            role, _ = NodeRole.objects.get_or_create(name="Constellation")
+            role, _ = NodeRole.objects.get_or_create(name="Watchtower")
             Node.objects.update_or_create(
                 mac_address=Node.get_current_mac(),
                 defaults={
@@ -2396,7 +2590,7 @@ class FaviconTests(TestCase):
             resp = self.client.get(reverse("pages:index"))
             b64 = (
                 Path(settings.BASE_DIR)
-                .joinpath("pages", "fixtures", "data", "favicon_constellation.txt")
+                .joinpath("pages", "fixtures", "data", "favicon_watchtower.txt")
                 .read_text()
                 .strip()
             )
@@ -2547,20 +2741,41 @@ class FavoriteTests(TestCase):
         self.assertContains(resp, f'aria-label="{badge_label}"')
 
     def test_dashboard_shows_charge_point_availability_badge(self):
-        Charger.objects.create(charger_id="CP-001", last_status="Available")
         Charger.objects.create(
             charger_id="CP-001", connector_id=1, last_status="Available"
         )
+        Charger.objects.create(charger_id="CP-002", last_status="Available")
+        Charger.objects.create(
+            charger_id="CP-003", connector_id=1, last_status="Unavailable"
+        )
+
+        resp = self.client.get(reverse("admin:index"))
+
+        expected = "1 / 2"
+        badge_label = gettext(
+            "%(available)s chargers reporting Available status with a CP number, out of %(total)s total Available chargers. %(missing)s Available chargers are missing a connector number."
+        ) % {"available": 1, "total": 2, "missing": 1}
+
+        self.assertContains(resp, expected)
+        self.assertContains(resp, 'class="charger-availability-badge"')
+        self.assertContains(resp, f'title="{badge_label}"')
+        self.assertContains(resp, f'aria-label="{badge_label}"')
+
+    def test_dashboard_charge_point_badge_ignores_aggregator(self):
+        Charger.objects.create(charger_id="CP-AGG", last_status="Available")
+        Charger.objects.create(
+            charger_id="CP-AGG", connector_id=1, last_status="Available"
+        )
         Charger.objects.create(
-            charger_id="CP-002", connector_id=1, last_status="Unavailable"
+            charger_id="CP-AGG", connector_id=2, last_status="Available"
         )
 
         resp = self.client.get(reverse("admin:index"))
 
-        expected = "2 / 1"
+        expected = "2 / 2"
         badge_label = gettext(
-            "%(total)s chargers reporting Available status, including %(with_cp)s with a CP number"
-        ) % {"total": 2, "with_cp": 1}
+            "%(available)s chargers reporting Available status with a CP number."
+        ) % {"available": 2}
 
         self.assertContains(resp, expected)
         self.assertContains(resp, 'class="charger-availability-badge"')

pages/urls.py

@@ -6,7 +6,9 @@ app_name = "pages"
 
 urlpatterns = [
     path("", views.index, name="index"),
-    path("readme/", views.readme, name="readme"),
+    path("read/<path:doc>/edit/", views.readme_edit, name="readme-edit"),
+    path("read/", views.readme, name="readme"),
+    path("read/<path:doc>", views.readme, name="readme-document"),
     path("sitemap.xml", views.sitemap, name="pages-sitemap"),
     path("release/", views.release_admin_redirect, name="release-admin"),
     path("client-report/", views.client_report, name="client-report"),