entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 32 Releases Activity

manager: add link to role in api_client details (#76473) #245

Merged
yweber merged 1 commits from wip/76473-api-client-roles-links into main 2024-01-30 15:19:47 +01:00
Conversation 0 Commits 1 Files Changed 2 +13 -1
2 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/authentic2/manager/templates/authentic2/manager/api_client_detail.html
View File

@ -27,7 +27,7 @@
{% if api_client.apiclient_roles.count %}
<li>{% trans "Roles:" %}
<ul>
{% for role in api_client.apiclient_roles.all %}<li>{{ role.name }}</li>{% endfor %}
{% for role in api_client.apiclient_roles.all %}<li><a href="{% url 'a2-manager-role-members' pk=role.pk %}">{{ role.name }}</a></li>{% endfor %}
</ul>
</li>
{% else %}

12
tests/test_manager_apiclient.py
View File

@ -262,6 +262,18 @@ def test_detail(superuser, app, phone_activated_authn):
assert delete_button.text() == 'Delete'
def test_manager_apiclient_roles_list(app, superuser):
login(app, superuser, '/')
role = Role.objects.create(name='parent', slug='parent', ou=get_default_ou())
yweber marked this conversation as resolved Outdated
vdeniaud commented 2024-01-29 18:16:11 +01:00
Outdated
Review
Copy Link

Idéalement le test devrait se contenter de créer un seul rôle et vérifier le lien, savoir si on affiche ou pas les rôles qui n’appartiennent pas au client ou les rôles hérités n'est pas l'affaire du patch correspondant :)

(je comprends mieux en voyant le ticket suivant mais du coup ces lignes seraient sûrement plus logiques si elles étaient ajoutées là bas)

Idéalement le test devrait se contenter de créer un seul rôle et vérifier le lien, savoir si on affiche ou pas les rôles qui n’appartiennent pas au client ou les rôles hérités n'est pas l'affaire du patch correspondant :) (je comprends mieux en voyant le ticket suivant mais du coup ces lignes seraient sûrement plus logiques si elles étaient ajoutées là bas)
api_client = APIClient.objects.create(name='foo')
api_client.apiclient_roles.set([role.pk])
resp = app.get(reverse('a2-manager-api-client-detail', kwargs={'pk': api_client.pk}))
assert 'href="%s"' % (reverse('a2-manager-role-members', kwargs={'pk': role.pk})) in resp.text
def test_edit(superuser, app, ou1, ou2):
preferred_color = Attribute.objects.create(
name='preferred_color',