Merge pull request #9986 from demarches-simplifiees/feat/4408

ETQ Usager / Instructeur, je souhaite ajouter plusieurs Pj d'un coup dans la messagerie

app/assets/stylesheets/attachment.scss

@@ -49,9 +49,8 @@
   }
 }
 
-.attachment-multiple.fr-downloads-group.destroyable {
-  ul {
+.attachment-multiple.fr-downloads-group.destroyable ul,
+ul[data-file-input-reset-target='fileList'] {
   list-style-type: none;
   padding-inline-start: 0;
-  }
 }

app/components/attachment/edit_component.rb

@@ -2,6 +2,7 @@
 class Attachment::EditComponent < ApplicationComponent
   attr_reader :champ
   attr_reader :attachment
+  attr_reader :attachments
   attr_reader :user_can_destroy
   alias user_can_destroy? user_can_destroy
   attr_reader :as_multiple
@@ -9,24 +10,23 @@ class Attachment::EditComponent < ApplicationComponent
 
   EXTENSIONS_ORDER = ['jpeg', 'png', 'pdf', 'zip'].freeze
 
-  def initialize(champ: nil, auto_attach_url: nil, attached_file:, direct_upload: true, index: 0, as_multiple: false, view_as: :link, user_can_destroy: true, **kwargs)
-    @as_multiple = as_multiple
-    @attached_file = attached_file
-    @auto_attach_url = auto_attach_url
+  def initialize(champ: nil, auto_attach_url: nil, attached_file:, direct_upload: true, index: 0, as_multiple: false, view_as: :link, user_can_destroy: true, user_can_replace: false, attachments: [], **kwargs)
     @champ = champ
+    @attached_file = attached_file
     @direct_upload = direct_upload
     @index = index
     @view_as = view_as
     @user_can_destroy = user_can_destroy
+    @user_can_replace = user_can_replace
+    @as_multiple = as_multiple
+    @auto_attach_url = auto_attach_url
+    # Adaptation pour la gestion des pièces jointes multiples
+    @attachments = attachments.presence || (kwargs.key?(:attachment) ? [kwargs.delete(:attachment)] : [])
+    @attachments << attached_file.attachment if attached_file.respond_to?(:attachment) && @attachments.empty?
+    @attachments.compact!
 
-    # attachment passed by kwarg because we don't want a default (nil) value.
-    @attachment = if kwargs.key?(:attachment)
-      kwargs.delete(:attachment)
-    elsif attached_file.respond_to?(:attachment)
-      attached_file.attachment
-    else
-      fail ArgumentError, "You must pass an `attachment` kwarg when not using as single attachment like in #{attached_file.name}. Set it to nil for a new attachment."
-    end
+    # Utilisation du premier attachement comme référence pour la rétrocompatibilité
+    @attachment = @attachments.first
 
     # When parent form has nested attributes, pass the form builder object_name
     # to correctly infer the input attribute name.
@@ -63,7 +63,7 @@ class Attachment::EditComponent < ApplicationComponent
 
   def file_field_options
     track_issue_with_missing_validators if missing_validators?
-    {
+    options = {
       class: class_names("fr-upload attachment-input": true, "#{attachment_input_class}": true, "hidden": persisted?),
       direct_upload: @direct_upload,
       id: input_id,
@@ -71,8 +71,13 @@ class Attachment::EditComponent < ApplicationComponent
       data: {
         auto_attach_url:,
         turbo_force: :server
-      }.merge(has_file_size_validator? ? { max_file_size: } : {})
-    }.merge(has_content_type_validator? ? { accept: accept_content_type } : {})
+      }.merge(has_file_size_validator? ? { max_file_size: max_file_size } : {})
+    }
+
+    options.merge!(has_content_type_validator? ? { accept: accept_content_type } : {})
+    options[:multiple] = true if as_multiple?
+
+    options
   end
 
   def poll_url
@@ -90,7 +95,8 @@ class Attachment::EditComponent < ApplicationComponent
   end
 
   def field_name(object_name = nil, method_name = nil, *method_names, multiple: false, index: nil)
-    helpers.field_name(@form_object_name || ActiveModel::Naming.param_key(@attached_file.record), attribute_name)
+    field_name = @form_object_name || ActiveModel::Naming.param_key(@attached_file.record)
+    "#{field_name}[#{attribute_name}]#{'[]' if as_multiple?}"
   end
 
   def attribute_name
@@ -126,24 +132,24 @@ class Attachment::EditComponent < ApplicationComponent
     !!attachment&.persisted?
   end
 
-  def downloadable?
+  def downloadable?(attachment)
     return false unless @view_as == :download
 
-    viewable?
+    viewable?(attachment)
   end
 
-  def viewable?
+  def viewable?(attachment)
     return false if attachment.virus_scanner_error?
     return false if attachment.watermark_pending?
 
     true
   end
 
-  def error?
+  def error?(attachment)
     attachment.virus_scanner_error?
   end
 
-  def error_message
+  def error_message(attachment)
     case
     when attachment.virus_scanner.infected?
       t(".errors.virus_infected")

app/components/attachment/edit_component/edit_component.en.yml

@@ -5,6 +5,7 @@ en:
   retry: Retry
   delete: Delete
   delete_file: Delete file %{filename}
+  multiple_files: Multiple files possible.
   replace: Replace
   replace_file: Replace file %{filename}
   open_file: Open file %{filename}

app/components/attachment/edit_component/edit_component.fr.yml

@@ -5,6 +5,7 @@ fr:
   retry: Réessayer
   delete: Supprimer
   delete_file: Supprimer le fichier %{filename}
+  multiple_files: Plusieurs fichiers possibles.
   replace: Remplacer
   replace_file: Remplacer le fichier %{filename}
   open_file: Ouvrir le fichier %{filename}

app/components/attachment/edit_component/edit_component.html.haml

@@ -1,4 +1,24 @@
-.attachment.fr-upload-group{ { id: attachment ? dom_id(attachment, :edit) : nil, class: class_names("fr-mb-1w": !(as_multiple? && downloadable?)) }.compact }
+.attachment.fr-upload-group{ id: (attachment ? dom_id(attachment, :edit) : nil), class: class_names("fr-mb-1w": !(as_multiple? && attachments.any?(&:persisted?))) }
+  - if as_multiple?
+    - attachments.each do |attachment|
+      - if attachment.persisted?
+        %div{ id: dom_id(attachment, :persisted_row) }
+          .flex.flex-gap-2{ class: class_names("attachment-error": attachment.virus_scanner_error?) }
+            - if user_can_destroy?
+              = render NestedForms::OwnedButtonComponent.new(formaction: destroy_attachment_path, http_method: :delete, opt: {class: "fr-btn fr-btn--tertiary fr-btn--sm fr-icon-delete-line", title: t(".delete_file", filename: attachment.filename)}) do
+                = t('.delete')
+
+              - if downloadable?(attachment)
+                = render Dsfr::DownloadComponent.new(attachment: attachment)
+            - else
+              .fr-py-1v
+                %span.attachment-filename.fr-mr-1w= link_to_if(viewable?(attachment), attachment.filename.to_s, helpers.url_for(attachment.blob), title: t(".open_file", filename: attachment.filename), **helpers.external_link_attributes)
+
+              = render Attachment::ProgressComponent.new(attachment: attachment, ignore_antivirus: true)
+
+            - if error?(attachment)
+              %p.fr-error-text= error_message(attachment)
+  - else
     - if persisted?
       %div{ id: dom_id(attachment, :persisted_row) }
         .flex.flex-gap-2{ class: class_names("attachment-error": attachment.virus_scanner_error?) }
@@ -6,30 +26,32 @@
             = render NestedForms::OwnedButtonComponent.new(formaction: destroy_attachment_path, http_method: :delete, opt: {class: "fr-btn fr-btn--tertiary fr-btn--sm fr-icon-delete-line", title: t(".delete_file", filename: attachment.filename)}) do
               = t('.delete')
 
-        - if downloadable?
+          - if downloadable?(attachment)
             = render Dsfr::DownloadComponent.new(attachment:)
           - else
             .fr-py-1v
-            %span.attachment-filename.fr-mr-1w= link_to_if(viewable?, attachment.filename.to_s, helpers.url_for(attachment.blob), title: t(".open_file", filename: attachment.filename), **helpers.external_link_attributes)
+              %span.attachment-filename.fr-mr-1w= link_to_if(viewable?(attachment), attachment.filename.to_s, helpers.url_for(attachment.blob), title: t(".open_file", filename: attachment.filename), **helpers.external_link_attributes)
 
             = render Attachment::ProgressComponent.new(attachment: attachment, ignore_antivirus: true)
 
-      - if error?
-        %p.fr-error-text= error_message
+        - if error?(attachment)
+          %p.fr-error-text= error_message(attachment)
 
-  - elsif first?
+  - if first? && !persisted?
     %p.fr-hint-text.fr-mb-1w
       - if max_file_size.present?
         = t('.max_file_size', max_file_size: number_to_human_size(max_file_size))
       - if allowed_formats.present?
         = t('.allowed_formats', formats: allowed_formats.join(', '))
+      - if as_multiple?
+        = t('.multiple_files')
 
-
-  - if !as_multiple?
+  - if !persisted? || champ.present? && champ.titre_identite?
     = file_field(champ, field_name, **file_field_options)
 
-    - if persisted?
-      - Attachment::PendingPollComponent.new(attachment: attachment, poll_url:, context: poll_context).then do |component|
+    - attachments.filter(&:persisted?).each do |attachment|
+      - if attachment.persisted?
+        - Attachment::PendingPollComponent.new(attachment: attachment, poll_url: poll_url, context: poll_context).then do |component|
           .fr-mt-2w
             = render component
 

app/components/attachment/multiple_component.rb

@@ -15,7 +15,7 @@ class Attachment::MultipleComponent < ApplicationComponent
 
   delegate :count, :empty?, to: :attachments, prefix: true
 
-  def initialize(champ:, attached_file:, form_object_name: nil, view_as: :link, user_can_destroy: true, max: nil)
+  def initialize(champ: nil, attached_file:, form_object_name: nil, view_as: :link, user_can_destroy: true, user_can_replace: false, max: nil)
     @champ = champ
     @attached_file = attached_file
     @form_object_name = form_object_name
@@ -35,11 +35,11 @@ class Attachment::MultipleComponent < ApplicationComponent
   end
 
   def empty_component_id
-    "attachment-multiple-empty-#{champ.public_id}"
+    champ.present? ? "attachment-multiple-empty-#{champ.public_id}" : "attachment-multiple-empty-generic"
   end
 
   def auto_attach_url
-    helpers.auto_attach_url(champ)
+    champ.present? ? helpers.auto_attach_url(champ) : '#'
   end
   alias poll_url auto_attach_url
 

app/components/attachment/multiple_component/multiple_component.html.haml

@@ -5,10 +5,10 @@
     %ul.fr-my-1v
       - each_attachment do |attachment, index|
         %li{ id: dom_id(attachment) }
-          = render Attachment::EditComponent.new(champ:, attached_file:, attachment:, index:, as_multiple: true, view_as:, user_can_destroy:, form_object_name:)
+          = render Attachment::EditComponent.new(champ:, attached_file:, attachment:, index:, view_as:, user_can_destroy:, form_object_name:)
 
   %div{ id: empty_component_id, class: class_names("hidden": !can_attach_next?), data: { turbo_force: :server } }
-    = render Attachment::EditComponent.new(champ:, attached_file:, attachment: nil, index: attachments_count, user_can_destroy:, form_object_name:)
+    = render Attachment::EditComponent.new(champ:, as_multiple: champ.nil?, attached_file:, attachment: nil, index: attachments_count, user_can_destroy:, form_object_name:)
 
   // single poll and refresh message for all attachments
   = render Attachment::PendingPollComponent.new(attachments: attachments, poll_url:, context: poll_context)

app/components/dossiers/message_component/message_component.html.haml

@@ -27,7 +27,9 @@
 
     - if groupe_gestionnaire.nil? && commentaire.piece_jointe.attached?
       .fr-ml-2w
-        = render Attachment::ShowComponent.new(attachment: commentaire.piece_jointe.attachment, new_tab: true)
+        - commentaire.piece_jointe.each do |attachment|
+          = render Attachment::ShowComponent.new(attachment: attachment, new_tab: true)
+
 
     - if show_reply_button?
       = button_tag type: 'button', class: 'fr-btn fr-btn--sm fr-btn--secondary fr-icon-arrow-go-back-line fr-btn--icon-left', onclick: 'document.querySelector("#commentaire_body").focus()' do

app/controllers/experts/avis_controller.rb

@@ -234,7 +234,7 @@ module Experts
     end
 
     def commentaire_params
-      params.require(:commentaire).permit(:body, :piece_jointe)
+      params.require(:commentaire).permit(:body, piece_jointe: [])
     end
   end
 end

app/controllers/instructeurs/dossiers_controller.rb

@@ -256,6 +256,7 @@ module Instructeurs
         flash.notice = "Message envoyé"
         redirect_to messagerie_instructeur_dossier_path(procedure, dossier)
       else
+        @commentaire.piece_jointe.purge.reload
         flash.alert = @commentaire.errors.full_messages
         render :messagerie
       end
@@ -393,7 +394,7 @@ module Instructeurs
     end
 
     def commentaire_params
-      params.require(:commentaire).permit(:body, :piece_jointe)
+      params.require(:commentaire).permit(:body, piece_jointe: [])
     end
 
     def champs_private_params

app/controllers/users/dossiers_controller.rb

@@ -621,7 +621,7 @@ module Users
     end
 
     def commentaire_params
-      params.require(:commentaire).permit(:body, :piece_jointe)
+      params.require(:commentaire).permit(:body, piece_jointe: [])
     end
   end
 end

app/graphql/types/message_type.rb

@@ -5,10 +5,10 @@ module Types
     field :body, String, null: false
     field :created_at, GraphQL::Types::ISO8601DateTime, null: false
     field :attachment, Types::File, null: true, deprecation_reason: "Utilisez le champ `attachments` à la place.", extensions: [
-      { Extensions::Attachment => { attachment: :piece_jointe } }
+      { Extensions::Attachment => { attachments: :piece_jointe, as: :single } }
     ]
     field :attachments, [Types::File], null: false, extensions: [
-      { Extensions::Attachment => { attachment: :piece_jointe, as: :multiple } }
+      { Extensions::Attachment => { attachments: :piece_jointe } }
     ]
     field :correction, CorrectionType, null: true
 

app/javascript/controllers/file_input_reset_controller.ts

@@ -1,37 +1,79 @@
 import { ApplicationController } from './application_controller';
-import { hide, show } from '@utils';
-
 export class FileInputResetController extends ApplicationController {
-  static targets = ['reset'];
-
-  declare readonly resetTarget: HTMLElement;
+  static targets = ['fileList'];
+  declare fileListTarget: HTMLElement;
 
   connect() {
-    this.on('change', (event) => {
-      if (event.target == this.fileInput) {
-        this.showResetButton();
+    super.connect();
+    this.updateFileList();
+    this.element.addEventListener('change', (event) => {
+      if (
+        event.target instanceof HTMLInputElement &&
+        event.target.type === 'file'
+      ) {
+        this.updateFileList();
       }
     });
   }
 
-  reset(event: Event) {
+  updateFileList() {
+    const files = this.fileInput?.files ?? [];
+    this.fileListTarget.innerHTML = '';
+
+    const deleteLabel =
+      this.element.getAttribute('data-delete-label') || 'Delete';
+
+    Array.from(files).forEach((file, index) => {
+      const container = document.createElement('li');
+      container.classList.add('flex', 'flex-gap-2', 'fr-mb-1w');
+
+      const deleteButton = this.createDeleteButton(deleteLabel, index);
+      container.appendChild(deleteButton);
+
+      const listItem = document.createElement('div');
+      listItem.textContent = file.name;
+
+      container.appendChild(listItem);
+      this.fileListTarget.appendChild(container);
+    });
+  }
+
+  createDeleteButton(deleteLabel: string, index: number) {
+    const button = document.createElement('button');
+    button.textContent = deleteLabel;
+    button.classList.add(
+      'fr-btn',
+      'fr-btn--tertiary',
+      'fr-btn--sm',
+      'fr-icon-delete-line'
+    );
+
+    button.addEventListener('click', (event) => {
       event.preventDefault();
-    this.fileInput.value = '';
-    hide(this.resetTarget);
+      this.removeFile(index);
+    });
+
+    return button;
   }
 
-  showResetButton() {
-    show(this.resetTarget);
+  removeFile(index: number) {
+    const files = this.fileInput?.files;
+    if (!files) return;
+
+    const dataTransfer = new DataTransfer();
+    Array.from(files).forEach((file, i) => {
+      if (index !== i) {
+        dataTransfer.items.add(file);
+      }
+    });
+
+    if (this.fileInput) this.fileInput.files = dataTransfer.files;
+    this.updateFileList();
   }
 
-  private get fileInput() {
-    const inputs =
-      this.element.querySelectorAll<HTMLInputElement>('input[type="file"]');
-    if (inputs.length == 0) {
-      throw new Error('No file input found');
-    } else if (inputs.length > 1) {
-      throw new Error('Multiple file inputs found');
-    }
-    return inputs[0];
+  private get fileInput(): HTMLInputElement | null {
+    return this.element.querySelector(
+      'input[type="file"]'
+    ) as HTMLInputElement | null;
   }
 }

app/lib/recovery/exporter.rb

@@ -10,7 +10,7 @@ module Recovery
                  :invites,
                  :traitements,
                  :transfer_logs,
-                 commentaires: { piece_jointe_attachment: :blob },
+                 commentaires: { piece_jointe_attachments: :blob },
                  avis: { introduction_file_attachment: :blob, piece_justificative_file_attachment: :blob },
                  dossier_operation_logs: { serialized_attachment: :blob },
                  attestation: { pdf_attachment: :blob },

app/lib/recovery/importer.rb

@@ -112,9 +112,12 @@ module Recovery
       end
     end
 
-    def import(pj)
+    def import(pjs)
+      attachments = pjs.respond_to?(:each) ? pjs : [pjs]
+      attachments.each do |pj|
         ActiveStorage::Blob.insert(pj.blob.attributes)
         ActiveStorage::Attachment.insert(pj.attributes)
       end
     end
+  end
 end

app/models/commentaire.rb

@@ -7,7 +7,7 @@ class Commentaire < ApplicationRecord
 
   validate :messagerie_available?, on: :create, unless: -> { dossier.brouillon? }
 
-  has_one_attached :piece_jointe
+  has_many_attached :piece_jointe
 
   validates :body, presence: { message: "ne peut être vide" }, unless: :discarded?
 
@@ -67,12 +67,6 @@ class Commentaire < ApplicationRecord
     sent_by?(connected_user) && (sent_by_instructeur? || sent_by_expert?) && !discarded?
   end
 
-  def file_url
-    if piece_jointe.attached? && piece_jointe.virus_scanner.safe?
-      Rails.application.routes.url_helpers.url_for(piece_jointe)
-    end
-  end
-
   def soft_delete!
     transaction do
       discard!
@@ -80,7 +74,7 @@ class Commentaire < ApplicationRecord
       update! body: ''
     end
 
-    piece_jointe.purge_later if piece_jointe.attached?
+    piece_jointe.each(&:purge_later) if piece_jointe.attached?
   end
 
   def flagged_pending_correction?

app/models/dossier.rb

@@ -54,7 +54,7 @@ class Dossier < ApplicationRecord
   has_many :prefilled_champs_public, -> { root.public_only.prefilled }, class_name: 'Champ', inverse_of: false
 
   has_many :commentaires, inverse_of: :dossier, dependent: :destroy
-  has_many :preloaded_commentaires, -> { includes(:dossier_correction, piece_jointe_attachment: :blob) }, class_name: 'Commentaire', inverse_of: :dossier
+  has_many :preloaded_commentaires, -> { includes(:dossier_correction, piece_jointe_attachments: :blob) }, class_name: 'Commentaire', inverse_of: :dossier
 
   has_many :invites, dependent: :destroy
   has_many :follows, -> { active }, inverse_of: :dossier
@@ -294,7 +294,7 @@ class Dossier < ApplicationRecord
   scope :for_api, -> {
     with_champs
       .with_annotations
-      .includes(commentaires: { piece_jointe_attachment: :blob },
+      .includes(commentaires: { piece_jointe_attachments: :blob },
         justificatif_motivation_attachment: :blob,
         attestation: [],
         avis: { piece_justificative_file_attachment: :blob },

app/serializers/commentaire_serializer.rb

@@ -2,13 +2,9 @@ class CommentaireSerializer < ActiveModel::Serializer
   attributes :email,
     :body,
     :created_at,
-    :attachment
+    :piece_jointe_attachments
 
   def created_at
     object.created_at&.in_time_zone('UTC')
   end
-
-  def attachment
-    object.file_url
-  end
 end

app/services/pieces_justificatives_service.rb

@@ -161,7 +161,7 @@ class PiecesJustificativesService
 
   def pjs_for_commentaires(dossiers)
     commentaire_id_dossier_id = Commentaire
-      .joins(:piece_jointe_attachment)
+      .joins(:piece_jointe_attachments)
       .where(dossier: dossiers)
       .pluck(:id, :dossier_id)
       .to_h

app/views/shared/dossiers/messages/_form.html.haml

@@ -10,10 +10,11 @@
   = render Dsfr::InputComponent.new(form: f, attribute: :body, input_type: :text_area, opts: { rows: 5, placeholder: placeholder, title: placeholder, class: 'fr-input message-textarea'})
 
   - if local_assigns.has_key?(:dossier)
-    .fr-mt-3w{ data: { controller: "file-input-reset" } }
-      = render Attachment::EditComponent.new(attached_file: commentaire.piece_jointe)
-      %button.hidden.fr-btn.fr-btn--tertiary-no-outline.fr-btn--icon-left.fr-icon-delete-line{ data: { 'file-input-reset-target': 'reset', action: 'file-input-reset#reset' } }
-        = t('views.shared.messages.remove_file')
+    .fr-mt-3w.fr-input-group
+      = f.label :piece_jointe, class: "fr-label"
+      %div{ data: { controller: "file-input-reset", delete_label: t('views.shared.messages.remove_file') } }
+        = render Attachment::MultipleComponent.new(attached_file: commentaire.piece_jointe)
+        %ul{ data: { 'file-input-reset-target': 'fileList' } }
 
   .fr-mt-3w
     = f.submit t('views.shared.dossiers.messages.form.send_message'), class: 'fr-btn', data: { disable: true }

config/locales/models/commentaire/en.yml

@@ -0,0 +1,6 @@
+en:
+  activerecord:
+    attributes:
+      commentaire:
+        body: 'Your message'
+        piece_jointe: "Attachment"

config/locales/models/commentaire/fr.yml

@@ -3,4 +3,4 @@ fr:
     attributes:
       commentaire:
         body: 'Votre message'
-        file: fichier
+        piece_jointe: "Pièce jointe"

config/locales/views/shared/fr.yml

@@ -23,3 +23,4 @@ fr:
         signin: 'Se connecter'
       messages:
         remove_file: 'Supprimer le fichier'
+        remove_all: "Supprimer tous les fichiers"

lib/tasks/recovery.rake

@@ -48,7 +48,7 @@ namespace :recovery do
     rake_puts "Will export #{dossier_ids}"
 
     dossiers_with_data = Dossier.where(id: dossier_ids)
-      .preload(commentaires: { piece_jointe_attachment: :blob },
+      .preload(commentaires: { pieces_jointes_attachments: :blob },
                                avis: { introduction_file_attachment: :blob, piece_justificative_file_attachment: :blob },
                                dossier_operation_logs: { serialized_attachment: :blob },
                                attestation: { pdf_attachment: :blob },
@@ -67,7 +67,7 @@ namespace :recovery do
       blob_keys_for_dossier += dossier.commentaires.flat_map do |commentaire|
         commentaire_blob_key = []
         if commentaire.piece_jointe.attached?
-          commentaire_blob_key += [commentaire.piece_jointe_attachment.blob.key]
+          commentaire_blob_key += [commentaire.piece_jointe_attachments.blob.key]
         end
         commentaire_blob_key
       end

spec/components/attachment/edit_component_spec.rb

@@ -66,8 +66,8 @@ RSpec.describe Attachment::EditComponent, type: :component do
       )
     end
 
-    it 'does not render an empty file' do # (is is rendered by MultipleComponent)
-      expect(subject).not_to have_selector('input[type=file]')
+    it 'does render an empty file' do # (is is rendered by MultipleComponent)
+      expect(subject).to have_selector('input[type=file]')
     end
 
     it 'renders max size for first index' do

spec/components/attachment/multiple_component_spec.rb

@@ -93,6 +93,14 @@ RSpec.describe Attachment::MultipleComponent, type: :component do
     end
   end
 
+  context 'when user can replace' do
+    let(:kwargs) { { user_can_replace: true } }
+
+    before do
+      attach_to_champ(attached_file, champ)
+    end
+  end
+
   def attach_to_champ(attached_file, champ)
     attached_file.attach(
       io: StringIO.new("x" * 2),

spec/controllers/api/v2/graphql_controller_spec.rb

@@ -432,6 +432,12 @@ describe API::V2::GraphqlController do
                   byteSize
                   contentType
                 }
+                attachments {
+                  filename
+                  checksum
+                  byteSize
+                  contentType
+                }
               }
               avis {
                 expert {
@@ -519,11 +525,19 @@ describe API::V2::GraphqlController do
             {
               body: commentaire.body,
               attachment: {
-                filename: commentaire.piece_jointe.filename.to_s,
-                contentType: commentaire.piece_jointe.content_type,
-                checksum: commentaire.piece_jointe.checksum,
-                byteSize: commentaire.piece_jointe.byte_size
+                filename: commentaire.piece_jointe.first.filename.to_s,
+                contentType: commentaire.piece_jointe.first.content_type,
+                checksum: commentaire.piece_jointe.first.checksum,
+                byteSize: commentaire.piece_jointe.first.byte_size
               },
+              attachments: commentaire.piece_jointe.map do |pj|
+                {
+                  filename: pj.filename.to_s,
+                  contentType: pj.content_type,
+                  checksum: pj.checksum,
+                  byteSize: pj.byte_size
+                }
+              end,
               email: commentaire.email
             }
           end

spec/controllers/experts/avis_controller_spec.rb

@@ -321,7 +321,7 @@ describe Experts::AvisController, type: :controller do
       let(:now) { Time.zone.parse("14/07/1789") }
       let(:avis) { avis_without_answer }
 
-      subject { post :create_commentaire, params: { id: avis.id, procedure_id:, commentaire: { body: 'commentaire body', piece_jointe: file } } }
+      subject { post :create_commentaire, params: { id: avis.id, procedure_id:, commentaire: { body: 'commentaire body', piece_jointe: [file] } } }
 
       before do
         allow(ClamavService).to receive(:safe_file?).and_return(scan_result)
@@ -343,7 +343,7 @@ describe Experts::AvisController, type: :controller do
 
         it do
           expect { subject }.to change(Commentaire, :count).by(1)
-          expect(Commentaire.last.piece_jointe.filename).to eq("piece_justificative_0.pdf")
+          expect(Commentaire.last.piece_jointe.first.filename).to eq("piece_justificative_0.pdf")
         end
       end
 

spec/factories/commentaire.rb

@@ -2,11 +2,16 @@ FactoryBot.define do
   factory :commentaire do
     association :dossier, :en_construction
     email { generate(:user_email) }
-
     body { 'plop' }
 
     trait :with_file do
-      piece_jointe { Rack::Test::UploadedFile.new('spec/fixtures/files/logo_test_procedure.png', 'image/png') }
+      after(:build) do |commentaire|
+        commentaire.piece_jointe.attach(
+          io: File.open('spec/fixtures/files/logo_test_procedure.png'),
+          filename: 'logo_test_procedure.png',
+          content_type: 'image/png'
+        )
+      end
     end
   end
 end

spec/services/commentaire_service_spec.rb

@@ -26,11 +26,20 @@ describe CommentaireService do
       end
     end
 
-    context 'when it has a file' do
-      let(:file) { fixture_file_upload('spec/fixtures/files/piece_justificative_0.pdf', 'application/pdf') }
+    context 'when it has multiple files' do
+      let(:files) do
+        [
+          fixture_file_upload('spec/fixtures/files/piece_justificative_0.pdf', 'application/pdf')
+        ]
+      end
 
-      it 'attaches the file' do
+      before do
+        commentaire.piece_jointe.attach(files)
+      end
+
+      it 'attaches the files' do
         expect(commentaire.piece_jointe.attached?).to be_truthy
+        expect(commentaire.piece_jointe.count).to eq(1)
       end
     end
   end

spec/services/pieces_justificatives_service_spec.rb

@@ -69,7 +69,7 @@ describe PiecesJustificativesService do
           attach_file(witness_commentaire.piece_jointe)
         end
 
-        it { expect(subject).to match_array(dossier.commentaires.first.piece_jointe.attachment) }
+        it { expect(subject).to match_array(dossier.commentaires.first.piece_jointe.attachments) }
       end
 
       context 'with a pj not safe on a commentaire' do

spec/system/users/en_construction_spec.rb

@@ -31,7 +31,7 @@ describe "Dossier en_construction" do
 
       click_on "Supprimer le fichier toto.txt"
 
-      input_selector = "#attachment-multiple-empty-#{champ.public_id} "
+      input_selector = "#attachment-multiple-empty-#{champ.public_id}"
       expect(page).to have_selector(input_selector)
       find(input_selector).attach_file(Rails.root.join('spec/fixtures/files/file.pdf'))