Merge pull request #7689 from mfo/sentry/3462032683

bug(instructeurs/archives#create): month should not be nil

app/controllers/instructeurs/archives_controller.rb

@@ -30,7 +30,7 @@ module Instructeurs
     end
 
     def create_archive_url(procedure, date)
-      instructeur_archives_path(procedure, type: 'monthly', month: date.strftime('%Y-%m'))
+      instructeur_archives_path(procedure, type: 'monthly', year_month: date.strftime('%Y-%m'))
     end
 
     def groupe_instructeurs

spec/system/administrateurs/procedure_archive_and_export_spec.rb

@@ -32,6 +32,7 @@ describe 'Creating a new procedure', js: true do
       page.first(".archive-table .button").click
     }.to have_enqueued_job(ArchiveCreationJob).with(procedure, an_instance_of(Archive), administrateur)
     expect(page).to have_content("Votre demande a été prise en compte. Selon le nombre de dossiers, cela peut prendre de quelques minutes a plusieurs heures. Vous recevrez un courriel lorsque le fichier sera disponible.")
+    expect(Archive.first.month).not_to be_nil
 
     # check exports
     click_on "Télécharger tous les dossiers"

spec/system/instructeurs/instruction_spec.rb

@@ -180,6 +180,20 @@ describe 'Instructing a dossier:', js: true do
     expect(page).to have_text("Dossier envoyé")
   end
 
+  context 'A instructeur can ask for an Archive' do
+    let(:procedure) { create(:procedure, :published, :with_piece_justificative, instructeurs: [instructeur]) }
+    let(:dossier) { create(:dossier, :accepte, procedure: procedure) }
+    before do
+      log_in(instructeur.email, password)
+      visit instructeur_archives_path(procedure)
+    end
+    scenario 'download' do
+      expect {
+        page.first(".archive-table .button").click
+      }.to have_enqueued_job(ArchiveCreationJob).with(procedure, an_instance_of(Archive), instructeur)
+      expect(Archive.first.month).not_to be_nil
+    end
+  end
   context 'with dossiers having attached files', js: true do
     let(:procedure) { create(:procedure, :published, :with_piece_justificative, instructeurs: [instructeur]) }
     let(:dossier) { create(:dossier, :en_construction, procedure: procedure) }