Development #34133
pas de rôle technique "administrateur des services"
0%
Description
Et donc pas possibilité de donner accès à cet écran sans également mettre les gens superadmins.
Fichiers
Demandes liées
Révisions associées
update translation (#34133)
tests: add tests on a2_rbac post_migrate handlers (#34133)
adapt manager tests (#34133)
Historique
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Fichier 0001-a2_rbac-add-default-role-of-service-administrator-34.patch 0001-a2_rbac-add-default-role-of-service-administrator-34.patch ajouté
- Fichier 0002-update-translation-34133.patch 0002-update-translation-34133.patch ajouté
- Fichier 0003-tests-add-tests-on-a2_rbac-post_migrate-handlers-341.patch 0003-tests-add-tests-on-a2_rbac-post_migrate-handlers-341.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Paul Marillonnet il y a presque 5 ans
Les tests sont en échec (quelques adaptations nécessaires dans les tests de test_manager.py ; et la méthode emit_post_migrate_signal qui en django 1.8 prend un premier paramètre supplémentaire created_models).
Mis à jour par Paul Marillonnet il y a plus de 4 ans
- Statut changé de Solution proposée à En cours
- Assigné à changé de Benjamin Dauvergne à Paul Marillonnet
Je reprends sur la base de ce qu'a fait Benj, puisque ce ticket semble maintenant critique.
Mis à jour par Paul Marillonnet il y a plus de 4 ans
- Fichier 0003-tests-add-tests-on-a2_rbac-post_migrate-handlers-341.patch 0003-tests-add-tests-on-a2_rbac-post_migrate-handlers-341.patch ajouté
- Statut changé de En cours à Solution proposée
Des erreurs bâteau dans les tests, corrigé ici.
Mis à jour par Paul Marillonnet il y a plus de 4 ans
- Statut changé de Solution proposée à En cours
- Assigné à changé de Paul Marillonnet à Benjamin Dauvergne
Et non, raté, encore des choses à adapter dans les tests hors de ce fichier test_a2_rbac.py
Mis à jour par Paul Marillonnet il y a plus de 4 ans
- Assigné à changé de Benjamin Dauvergne à Paul Marillonnet
(Fausse manip')
Mis à jour par Paul Marillonnet il y a plus de 4 ans
- Fichier 0003-tests-add-tests-on-a2_rbac-post_migrate-handlers-341.patch 0003-tests-add-tests-on-a2_rbac-post_migrate-handlers-341.patch ajouté
- Fichier 0004-adapt-manager-tests-34133.patch 0004-adapt-manager-tests-34133.patch ajouté
- Statut changé de En cours à Solution proposée
Voilà, quelques modifications nécessaires dans le patch de tests écrit par Benj, et un quatrième patch d'adaptation des tests du backoffice.
Mis à jour par Paul Marillonnet il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Et comme je ne suis pas l'auteur du patch, ack.
Mis à jour par Paul Marillonnet il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 2349e21a0188e64de7e4f085d43f2448791f944c
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Tue Sep 24 17:13:12 2019 +0200
adapt manager tests (#34133)
commit f93c71a58bfdfd60950143c5bc440369c65c8fbc
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Jun 28 12:26:56 2019 +0200
tests: add tests on a2_rbac post_migrate handlers (#34133)
commit a313152f3a25ffbfebfb967cfc13ecbc8e58cc02
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Jun 27 19:16:47 2019 +0200
update translation (#34133)
commit 968a6e25758799273a0230389cc6756f12f3d0d3
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Jun 27 18:59:32 2019 +0200
a2_rbac: add default role of service administrator (#34133)
Mis à jour par Frédéric Péters il y a plus de 4 ans
Ça a amené une erreur côté tests hobo,
# Two services loaded roles.json, so there must be 4 roles associated
# to their "ou" excluding their superuser role, 2 admin roles for users
# and roles, a 2 loaded roles, petite enfance and état-civil
> assert Role.objects.filter(ou__isnull=False,
service__isnull=True).count() == 8
E AssertionError: assert 10 == 8
Mis à jour par Paul Marillonnet il y a plus de 4 ans
- Lié à Bug #36615: Le rôle par défaut d'administration des services casse les tests authentic ajouté
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
a2_rbac: add default role of service administrator (#34133)