Merge pull request #6304 from betagouv/revert-6302-dossier-pdf-export

Revert "Expose dossier PDF export as IO"

app/jobs/archive_creation_job.rb

@@ -1,6 +1,4 @@
 class ArchiveCreationJob < ApplicationJob
-  queue_as :exports
-
   def perform(procedure, archive, instructeur)
     ProcedureArchiveService
       .new(procedure)

app/models/dossier.rb

@@ -69,6 +69,7 @@ class Dossier < ApplicationRecord
   has_many :attestations, dependent: :destroy
 
   has_one_attached :justificatif_motivation
+  has_one_attached :pdf_export_for_instructeur
 
   has_many :champs, -> { root.public_ordered }, inverse_of: false, dependent: :destroy
   has_many :champs_private, -> { root.private_ordered }, class_name: 'Champ', inverse_of: false, dependent: :destroy

app/services/pieces_justificatives_service.rb

@@ -49,30 +49,6 @@ class PiecesJustificativesService
     end
   end
 
-  class FakeAttachment < Hashie::Dash
-    property :filename
-    property :name
-    property :file
-    property :id
-    property :created_at
-
-    def download
-      file
-    end
-
-    def read(*args)
-      file.read(*args)
-    end
-
-    def close
-      file.close
-    end
-
-    def attached?
-      true
-    end
-  end
-
   def self.generate_dossier_export(dossier)
     pdf = ApplicationController
       .render(template: 'dossiers/show', formats: [:pdf],
@@ -80,14 +56,10 @@ class PiecesJustificativesService
                 include_infos_administration: true,
                 dossier: dossier
               })
-
+    ActiveRecord::Base.no_touching do
-    FakeAttachment.new(
+      dossier.pdf_export_for_instructeur.attach(io: StringIO.open(pdf), filename: "export-#{dossier.id}.pdf", content_type: 'application/pdf')
-      file: StringIO.new(pdf),
+    end
-      filename: "export-#{dossier.id}.pdf",
+    dossier.pdf_export_for_instructeur
-      name: 'pdf_export_for_instructeur',
-      id: dossier.id,
-      created_at: dossier.updated_at
-    )
   end
 
   private

app/services/procedure_archive_service.rb

@@ -26,7 +26,7 @@ class ProcedureArchiveService
 
     Zip::OutputStream.open(tmp_file) do |zipfile|
       files.each do |attachment, pj_filename|
-        zipfile.put_next_entry("procedure-#{@procedure.id}/#{pj_filename}")
+        zipfile.put_next_entry(pj_filename)
         zipfile.puts(attachment.download)
       end
     end

spec/factories/dossier.rb

@@ -201,6 +201,15 @@ FactoryBot.define do
       end
     end
 
+    trait :with_pdf_export do
+      after(:create) do |dossier, _evaluator|
+        dossier.pdf_export_for_instructeur.attach(
+          io: StringIO.new('Hello World'),
+          filename: 'export.pdf'
+        )
+      end
+    end
+
     trait :with_justificatif do
       after(:create) do |dossier, _evaluator|
         dossier.justificatif_motivation.attach(

spec/lib/active_storage/downloadable_file_spec.rb

@@ -1,12 +1,12 @@
 describe ActiveStorage::DownloadableFile do
-  let(:dossier) { create(:dossier, :en_construction) }
+  let(:dossier) { create(:dossier, :en_construction, :with_pdf_export) }
 
   subject(:list) { ActiveStorage::DownloadableFile.create_list_from_dossier(dossier) }
 
   describe 'create_list_from_dossier' do
     context 'when no piece_justificative is present' do
       it { expect(list.length).to eq 1 }
-      it { expect(list.first[0].name).to eq "pdf_export_for_instructeur" }
+      it { expect(list.first[0].record_type).to eq "Dossier" }
     end
 
     context 'when there is a piece_justificative' do

spec/services/pieces_justificatives_service_spec.rb

@@ -49,6 +49,7 @@ describe PiecesJustificativesService do
     subject { PiecesJustificativesService.generate_dossier_export(dossier) }
     it "generates pdf export for instructeur" do
       subject
+      expect(dossier.pdf_export_for_instructeur).to be_attached
     end
 
     it "doesn't update dossier" do