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
yweber commented 2024-01-29 13:35:25 +01:00
Owner
Copy Link
No description provided.
yweber force-pushed wip/76473-api-client-roles-links from 9065aa08ea to 7bd5fc2aa1 2024-01-29 14:05:10 +01:00 Compare
yweber force-pushed wip/76473-api-client-roles-links from 7bd5fc2aa1 to 94e4b14af9 2024-01-29 14:06:53 +01:00 Compare
yweber changed title from WIP: manager: add link to role in api_client details (#76473) to manager: add link to role in api_client details (#76473) 2024-01-29 14:37:04 +01:00
vdeniaud requested changes 2024-01-29 15:07:29 +01:00
tests/test_api_client.py Outdated
@ -199,1 +201,4 @@
assert User.objects.get(uuid=resp.json['uuid'])
def test_api_roles_list(app, api_client, superuser):
vdeniaud commented 2024-01-29 15:06:59 +01:00
Owner
Copy Link

Ce test irait mieux dans le fichier test_manager_apiclient.py je crois

Ce test irait mieux dans le fichier `test_manager_apiclient.py` je crois
yweber marked this conversation as resolved
yweber force-pushed wip/76473-api-client-roles-links from 94e4b14af9 to 4f1d5fc17d 2024-01-29 15:51:26 +01:00 Compare
yweber force-pushed wip/76473-api-client-roles-links from 4f1d5fc17d to eae4b34218 2024-01-29 15:58:10 +01:00 Compare
yweber force-pushed wip/76473-api-client-roles-links from eae4b34218 to c758a4fe7e 2024-01-29 16:07:45 +01:00 Compare
yweber requested review from vdeniaud 2024-01-29 18:19:05 +01:00
vdeniaud approved these changes 2024-01-29 18:28:49 +01:00
vdeniaud left a comment
Owner
Copy Link

Une remarque de détail à appliquer si tu veux

Une remarque de détail à appliquer si tu veux
tests/test_manager_apiclient.py Outdated
@ -265,0 +265,4 @@
def test_manager_apiclient_roles_list(app, superuser):
login(app, superuser, '/')
parent_role = Role.objects.create(name='parent', slug='parent', ou=get_default_ou())
vdeniaud commented 2024-01-29 18:16:11 +01:00
Owner
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)
yweber marked this conversation as resolved
yweber force-pushed wip/76473-api-client-roles-links from c758a4fe7e to 522a226d99 2024-01-30 09:21:19 +01:00 Compare
yweber force-pushed wip/76473-api-client-roles-links from 522a226d99 to 02fccc9a0b 2024-01-30 15:14:11 +01:00 Compare
yweber merged commit 02fccc9a0b into main 2024-01-30 15:19:47 +01:00
yweber deleted branch wip/76473-api-client-roles-links 2024-01-30 15:19:47 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/authentic#245
No description provided.
Delete Branch "wip/76473-api-client-roles-links"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?