Visibilite d'une fiche #265
Conversation
alanzirek
force-pushed
the
visibilite
branch
3 times, most recently
from
4131d41
to
84663b4
Compare
alanzirek
force-pushed
the
visibilite
branch
4 times, most recently
from
44b2ff3
to
02c51b4
Compare
| def _fiche_detection_bakery(): | ||
| etat = Etat.objects.get(id=Etat.get_etat_initial()) | ||
| return baker.make(FicheDetection, _fill_optional=True, etat=etat) | ||
| return baker.make( | ||
| FicheDetection, _fill_optional=True, etat=etat, createur=mocked_authentification_user.agent.structure |
There was a problem hiding this comment.
Cela va impliquer par défaut que toutes les fiches que l'on utilise vont être créé par le user de test, est ce que c'est vraiment ce que l'on veut ? Pourquoi ne pas personaliser le comportement au niveau du test voulu directement ?
There was a problem hiding this comment.
J'ai opté pour cette solution car avec l'ajout de la visibilité d'une fiche, cela impact beaucoup de tests.
Étant donné que dans la plupart des tests une fiche détection est en visibilité brouillon (valeur par défaut), il faut que le user soit de la même structure pour y accéder et donc exécuter les tests.
J'ai préféré faire l'inverse (modifier la structure du user si besoin) pour avoir moins de tests à modifier.
Est-ce ok pour toi ?
Oui je suis d'accord, je m'en suis rendu compte tardivement, mais ceci étant les 2 scopes sont liés. |
alanzirek
force-pushed
the
visibilite
branch
2 times, most recently
from
813ea84
to
2eda949
Compare
alanzirek
force-pushed
the
visibilite
branch
5 times, most recently
from
33cccf0
to
20754ae
Compare
Modèle : - Ajout d'unTextChoices Visibilite pour gérer les valeurs : brouillon, local, et national. (core/models.py) - Ajout du mixinAllowVisibiliteMixin pour gérer et ajouter à tout type de fiche la logique de visibilité. (core/mixins.py) - Ajout de méthodes dans le modèle FicheDetection pour gérer la mise à jour de la visibilité ou l'accès à une fiche détection. (can_update_visibilite et can_user_access dans sv/models.py/FicheDetection/) Form : - Ajout d'un formulaire FicheDetectionVisibiliteUpdateForm pour permettre aux agents de mettre à jour la visibilité d'une fiche détection. (sv/forms.py) Templates : - Ajout Radio Button DSFR: Ajout de la classe DSFRRadioButton et du template correspondant pour intégrer le radio button du DSFR. (core/fields.py, core/templates/forms/dsfr_radio_btn.html) - Badge: Affichage du statut de visibilité sous forme de badge sur la page de détail du formulaire de détection. (sv/templates/sv/fichedetection_detail.html) - Modal: Ajout d'une modale pour mettre à jour la visibilité dans la page de détail d'une fiche détection. (sv/templates/sv/_fichedetection_update_visibilite_modal.html) - Filtre de template: Ajout d'un filtre de template get_item pour récupérer la description d'une visibilité à partir de sa valeur (core/templatetags/get_item.py) Migration de la base de données: - Ajout d'un script de migration pour inclure le nouveau champ visibilite dans le modèle FicheDetection et définir des valeurs par défaut (brouillon) pour les enregistrements existants. (sv/migrations/0022_fichedetection_visibilite.py)
Cette PR ajoute la notion de visibilité d'une fiche (brouillon, local, national).
Modèle :
TextChoicesVisibilitepour gérer les valeurs :brouillon,local, etnational. (core/models.py)AllowVisibiliteMixinpour gérer et ajouter à tout type de fiche la logique de visibilité. (core/mixins.py)FicheDetectionpour gérer la mise à jour de la visibilité ou l'accès à une fiche détection. (can_update_visibiliteetcan_user_accessdanssv/models.py/FicheDetection/)Form :
FicheDetectionVisibiliteUpdateFormpour permettre aux agents de mettre à jour la visibilité d'une fiche détection. (sv/forms.py)Templates :
DSFRRadioButtonet du template correspondant pour intégrer le radio button du DSFR. (core/fields.py,core/templates/forms/dsfr_radio_btn.html)sv/templates/sv/fichedetection_detail.html)sv/templates/sv/_fichedetection_update_visibilite_modal.html)get_itempour récupérer la description d'une visibilité à partir de sa valeur (core/templatetags/get_item.py)Tests :
fiche_detection_bakerypour que le créateur de chaque fiche détection soit lié au user connecté et donc puisse accéder à la fiche (cf. commentaire)Migration de la base de données :
visibilitedans le modèleFicheDetectionet définir des valeurs par défaut (brouillon) pour les enregistrements existants. (sv/migrations/0022_fichedetection_visibilite.py)