From 488c00c37866702efca6266c66eff45f19c609e7 Mon Sep 17 00:00:00 2001
From: Valerij Maljulin <vmaljuli@redhat.com>
Date: Wed, 11 Oct 2023 11:35:23 +0200
Subject: [PATCH] fixup! Permissions as HTML page

---
 tests/test_api_v10.py               | 12 ++++++++++++
 waiverdb/templates/permissions.html | 23 ++++++++++++++++-------
 2 files changed, 28 insertions(+), 7 deletions(-)

diff --git a/tests/test_api_v10.py b/tests/test_api_v10.py
index 0e1445c0..766815ed 100644
--- a/tests/test_api_v10.py
+++ b/tests/test_api_v10.py
@@ -892,6 +892,18 @@ def test_permissions_endpoint(client):
         assert r.status_code == 200
         assert r.json == config['PERMISSIONS'][0:1]
 
+        r = client.get('/api/v1.0/permissions?html=1')
+        assert r.status_code == 200
+        assert r.content_type == 'text/html'
+        assert f"<td>{config['PERMISSIONS'][0]['name']}</td>" in r.text
+        assert f"<td>{config['PERMISSIONS'][1]['name']}</td>" in r.text
+
+        r = client.get('/api/v1.0/permissions?testcase=greenwave-tests.test1&html=1')
+        assert r.status_code == 200
+        assert r.content_type == 'text/html'
+        assert f"<td>{config['PERMISSIONS'][0]['name']}</td>" not in r.text
+        assert f"<td>{config['PERMISSIONS'][1]['name']}</td>" in r.text
+
 
 def test_config_endpoint_superusers(client):
     config = {
diff --git a/waiverdb/templates/permissions.html b/waiverdb/templates/permissions.html
index f43201c0..b8c1c068 100644
--- a/waiverdb/templates/permissions.html
+++ b/waiverdb/templates/permissions.html
@@ -25,26 +25,35 @@ <h1>
                 <img src="{{ url_for('favicon') }}" style="height: 64px" alt="Waiver DB" />
                 {{ self.title() }}
             </h1>
-            <table class="table table-striped text-nowrap">
+            <table class="table table-striped">
                 <thead>
                     <tr>
                         <th>Name</th>
                         <th>Description</th>
                         <th>Maintainers</th>
                         <th>Testcases</th>
-                        <th>User groups</th>
+                        <th>Users/Groups</th>
                     </tr>
                 </thead>
                 <tbody>
-                    {% for p in permissions %}
+                    {% for p in permissions -%}
                     <tr>
                         <td>{{ p["name"] }}</td>
                         <td>{{ p["description"] }}</td>
-                        <td>{{ p["maintainers"] | map("urlize") | join("<br />") | safe }}</td>
-                        <td>{{ p["testcases"] | join("<br />") | safe }}</td>
-                        <td>{{ p["groups"] | join("<br />") | safe }}</td>
+                        <td>{{ p["maintainers"] | map("urlize") | safe | join("<br />") | safe }}</td>
+                        <td>{{ p["testcases"] | map("replace('-', '&#8209;')") | join("<br />") | safe }}</td>
+                        <td>
+                            {% if p["groups"] | length > 0 -%}
+                            <span class="font-weight-bold">Groups:</span><br />
+                            {{ p["groups"] | map("replace('-', '&#8209;')") | join("<br />") | safe }}
+                            {%- endif %}
+                            {% if p["users"] | length > 0 -%}
+                            <span class="font-weight-bold">Users:</span><br />
+                            {{ p["users"] | map("replace('-', '&#8209;')") | join("<br />") | safe }}
+                            {%- endif %}
+                        </td>
                     </tr>
-                    {% endfor %}
+                    {%- endfor %}
                 </tbody>
             </table>
         </div>