Afficher le scope AEEH sur les formulaires éditeur enfance - #1692
Open
jbfeldis wants to merge 1 commit into
Open
Afficher le scope AEEH sur les formulaires éditeur enfance#1692jbfeldis wants to merge 1 commit into
jbfeldis wants to merge 1 commit into
Conversation
Ajoute cnav_allocation_enfant_handicape à scopes_config.displayed des 27 cas d'usage tarification_municipale_enfance ayant un service_provider_id. Le scope est ajouté à displayed uniquement : ni à disabled, qui verrouillerait la case, ni à initialize_with.scopes, qui la pré cocherait. abelium hérite du scopes_config de familea via la clé de fusion YAML, une seule édition couvre les deux. Les 5 formulaires single page n'ont pas de liste displayed et affichaient déjà le scope décoché. available_scopes de familea attendait 4 scopes et en attend 5 : le test vérifie que la config displayed est respectée, pas le nombre de scopes.
Member
|
tl;dr de ce qu'il faut faire:
> grep "use_case: tarification_municipale_enfance" config/authorization_request_forms/api_particulier.yml | wc -l
32Il n'y a que 26 lignes d'ajoutées sur le fichier, je pense qu'il en manque 6. |
skelz0r
self-requested a review
July 15, 2026 08:41
skelz0r
removed their request for review
July 15, 2026 08:42
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contexte
DP-1844 — sur les cas d’usage « Tarification sociale des services municipaux à l’enfance », le scope
cnav_allocation_enfant_handicape(Statut AEEH) doit apparaître sans être pré coché.Critère retenu : cas d’usage
tarification_municipale_enfanceavec unservice_provider_idrempli (= les formulaires éditeur). Le critère tombe juste : sur les 33 cas d’usagetarification_municipale_enfance, 32 ont unservice_provider_id, et le seul sans est le formulaire génériqueapi-particulier-tarification-municipale-enfance… qui avait déjà le scope dans l’état demandé. Il a servi de référence, pas de cible.Ce que fait la PR
config/authorization_request_forms/api_particulier.yml: +26 lignes, 26 blocsscopes_config.displayedmodifiés, couvrant 27 cas d’usage (abeliumhérite de tout lescopes_configdefamileavia la clé de fusion<<:, donc une seule édition couvre les deux). Aucune autre ligne touchée.displayeduniquement — ni dansdisabled(qui verrouillerait la case), ni dansinitialize_with.scopes(qui la pré cocherait).config/authorization_definitions/api_particulier.yml, sansfeature_flagniincluded: true.Cas d’usage modifiés (27)
Scopes cochables — 11
La case apparaît décochée et le demandeur peut la cocher : effet fonctionnel réel.
api-particulier-famileaapi-particulier-abeliumfamileavia<<:)api-particulier-agedi-proxima-enfapi-particulier-andyvieapi-particulier-carte-plusapi-particulier-city-family-mushroom-softwareapi-particulier-jcdeveloppement-familyclicapi-particulier-nfiapi-particulier-noethysapi-particulier-resagendaapi-particulier-ypok-ykidzScopes figés — 16
La case apparaît décochée mais n’est pas cochable — voir « Points à confirmer » ci-dessous.
api-particulier-3d-ouestapi-particulier-achetezaapi-particulier-agora-plusapi-particulier-aigaapi-particulier-amiciel-maliceapi-particulier-arpege-concertoapi-particulier-bl-enfance-berger-levraultapi-particulier-cantine-de-franceapi-particulier-civil-enfance-ciril-groupapi-particulier-cosoluce-fluoapi-particulier-e1osapi-particulier-ecorestauration-loyfeeyapi-particulier-entrouvert-publikapi-particulier-jvs-mairistemapi-particulier-technocarte-ileapi-particulier-waigeo-myperischoolNon modifiés car déjà conformes — 6
api-particulier-docaposte-fastdisplayed→ affiche déjà tous les scopes, AEEH compris, décochéapi-particulier-odyssee-informatique-pandoreapi-particulier-qiis-eticketapi-particulier-teamnet-axelapi-particulier-sigec-maelisapi-particulier-tarification-municipale-enfanceservice_provider_id) : avait déjà le scope27 modifiés + 6 déjà conformes = les 33 cas d’usage
tarification_municipale_enfance.Points à confirmer @miryad
1. Sur les 16 cas d’usage « scopes figés », la case AEEH restera non cochable.
Sur ces formulaires, le bloc
scopesest déclaré dansstatic_blocks(et non danssteps) : le décorateur le marqueeditable: falseet le form builder rend toutes les cases en lecture seule. La demande est donc littéralement satisfaite (la case apparaît, non cochée), mais sans effet fonctionnel : l’habilitation ne pourra jamais contenir l’AEEH sur ces 16 formulaires.Si le besoin est que le demandeur puisse réellement cocher l’AEEH chez ces éditeurs, il faut sortir
scopesdestatic_blocks→ cela change le contrat passé avec l’éditeur → à cadrer avec eux dans un ticket distinct. Dis-nous ce qui est attendu.2. Périmètre limité au cas d’usage enfance.
Lu au pied de la lettre, « tous les cas d’usage qui ont le
service_provider_idrempli » viserait 63 formulaires, tous cas d’usage confondus :tarification_eaje(14),tarification_transports(5),ccas(4),aides_facultatives(3),cantines-colleges-lycees-tarification(3),stationnement_residentiel(2). Mettre l’AEEH sur du stationnement résidentiel ou de la gestion RH n’a pas de sens, et le ticket dit « du cas d’usage enfance » → périmètre restreint àtarification_municipale_enfance. À confirmer, l’élargissement est trivial.Test plan
bundle exec rspec spec/acceptance spec/models→ 810 examples, 0 failuresbundle exec rubocop→ cleanspec/acceptance/authorizations_configs_spec.rb: sur les 32 formulaires éditeur du cas d’usage, l’AEEH est affiché et jamais pré coché (+ garde-fou « ensemble non vide » pour éviter un test qui passe à vide)spec/models/authorization_request_spec.rb:#available_scopesdefamileaattendait 4 scopes, en attend 5. Échec légitime — le test vérifie que la configdisplayedest respectée, pas quefamileaa 4 scopes. L’exemple voisin#disabled_scopespasse toujours avec le seulcnaf_quotient_familial, ce qui confirme que l’AEEH est affiché sans être verrouillédocs/ni feature cucumber ne référence ces formulaires🤖 Generated with Claude Code