Permet l'ajout de nouveaux scopes sur les HabilitationType avec des demandes

Author: jbfeldis

app/components/molecules/admin/scopes_editor_component.html.erb

@@ -7,7 +7,7 @@
     </div>
   <% end %>
 
-  <% unless structural_fields_locked? %>
+  <% unless disabled? %>
     <script type="text/x-template" data-nested-form-target="template">
       <div class="nested-fields fr-grid-row fr-grid-row--gutters fr-mb-2w" data-new-record="true">
         <div class="fr-col-4">
@@ -57,6 +57,9 @@
           <div class="fr-input-group <%= scope_field_error_class(index, :value) %>">
             <label class="fr-label" for="scope-<%= index %>-value"><%= t('admin.habilitation_types.form.scopes_editor.value') %> <span class="fr-ml-1w fr-text-error">*</span></label>
             <input type="text" id="scope-<%= index %>-value" name="habilitation_type[scopes][][value]" value="<%= scope['value'] %>" class="fr-input" required aria-describedby="scope-<%= index %>-value-error"<%= ' disabled' if structural_fields_locked? %>>
+            <% if structural_fields_locked? %>
+              <input type="hidden" name="habilitation_type[scopes][][value]" value="<%= scope['value'] %>">
+            <% end %>
             <% if scope_field_errors(index, :value).any? %>
               <p class="fr-messages-group" id="scope-<%= index %>-value-error">
                 <% scope_field_errors(index, :value).each do |msg| %>
@@ -72,18 +75,18 @@
             <input type="text" id="scope-<%= index %>-group" name="habilitation_type[scopes][][group]" value="<%= scope['group'] %>" class="fr-input"<%= ' disabled' if disabled? %>>
           </div>
         </div>
-        <% unless structural_fields_locked? %>
-          <div class="fr-col-2 fr-mt-4w">
+        <div class="fr-col-2 fr-mt-4w">
+          <% unless structural_fields_locked? %>
             <button type="button" class="fr-btn fr-btn--tertiary fr-btn--sm" data-action="click->nested-form#remove">
               <%= t('admin.habilitation_types.form.scopes_editor.remove') %>
             </button>
-          </div>
-        <% end %>
+          <% end %>
+        </div>
       </div>
     <% end %>
   </div>
 
-  <% unless structural_fields_locked? %>
+  <% unless disabled? %>
     <button type="button" class="fr-btn fr-btn--secondary fr-btn--sm" data-action="click->nested-form#add">
       <%= t('admin.habilitation_types.form.scopes_editor.add') %>
     </button>

app/controllers/admin/habilitation_types_controller.rb

@@ -110,9 +110,21 @@ def sanitize_locked_scopes(submitted_scopes)
     existing = @habilitation_type.scopes || []
     return existing if submitted_scopes.blank?
 
-    existing.each_with_index.map do |scope, i|
-      submitted = (submitted_scopes[i] || {}).stringify_keys
-      scope.merge(submitted.slice(*HabilitationType::EDITORIAL_SCOPE_PARAMS))
+    update_existing_scopes(existing, submitted_scopes) +
+      extract_new_scopes(existing, submitted_scopes)
+  end
+
+  def update_existing_scopes(existing, submitted_scopes)
+    existing.map do |scope|
+      submitted = submitted_scopes.find { |s| s[:value] == scope['value'] } || {}
+      scope.merge(submitted.stringify_keys.slice(*HabilitationType::EDITORIAL_SCOPE_PARAMS))
     end
   end
+
+  def extract_new_scopes(existing, submitted_scopes)
+    existing_values = existing.to_set { |s| s['value'] }
+    submitted_scopes
+      .reject { |s| existing_values.include?(s[:value]) }
+      .map(&:stringify_keys)
+  end
 end

features/admin/habilitation_types.feature

@@ -143,3 +143,20 @@ Fonctionnalité: Espace admin: types d'habilitation
     Alors le champ "Fournisseur" est désactivé
     Et le champ radio "API" est désactivé
 
+  Scénario: La page de consultation n'affiche pas le bouton d'ajout de scope et les champs sont désactivés
+    Sachant que je me connecte
+    Et qu'un type d'habilitation "API Protégée" avec des demandes liées et des scopes existe
+    Quand je me rends sur le chemin "/admin/types-habilitation/api-protegee"
+    Alors le champ du scope 1 "name" est désactivé
+    Et la page ne contient pas "Ajouter un scope"
+
+  @javascript
+  Scénario: Je peux ajouter un scope à un type avec des demandes liées
+    Sachant que je me connecte
+    Et qu'un type d'habilitation "API Protégée" avec des demandes liées et des scopes existe
+    Quand je me rends sur le chemin "/admin/types-habilitation/api-protegee/edit"
+    Et que je clique sur "Ajouter un scope"
+    Et que je remplis le scope 2 avec nom "Nouveau scope" valeur "new_scope" groupe "Groupe"
+    Et que je clique sur "Enregistrer"
+    Alors il y a un message de succès contenant "mis à jour"
+

features/step_definitions/habilitation_types_steps.rb

@@ -12,6 +12,21 @@
   )
 end
 
+Sachantque("un type d'habilitation {string} avec des demandes liées et des scopes existe") do |name|
+  data_provider = DataProvider.first || FactoryBot.create(:data_provider)
+  habilitation_type = FactoryBot.create(
+    :habilitation_type,
+    name:,
+    data_provider:,
+    blocks: [{ 'name' => 'basic_infos' }, { 'name' => 'scopes' }],
+    scopes: [{ 'name' => 'Revenu fiscal', 'value' => 'rfr', 'group' => 'Revenus' }]
+  )
+  FactoryBot.create(
+    :authorization_request,
+    type: habilitation_type.authorization_request_type
+  )
+end
+
 Quand('je choisis le type {string}') do |kind|
   choose kind, allow_label_click: true
 end
@@ -32,3 +47,7 @@
 Alors('le champ radio {string} est désactivé') do |label|
   expect(page).to have_field(label, disabled: true, visible: :all)
 end
+
+Alors('le champ du scope {int} {string} est désactivé') do |index, field|
+  expect(page).to have_css("#scope-#{index - 1}-#{field}[disabled]")
+end