fix upload piece justificative

app/controllers/description_controller.rb

@@ -32,7 +32,7 @@ class DescriptionController < ApplicationController
 
     @dossier.pieces_jointes.each do |piece_jointe|
       unless params["piece_jointe_#{piece_jointe.type}"].nil?
-        piece_jointe.content = params["piece_jointe_#{piece_jointe.id}"]
+        piece_jointe.content = params["piece_jointe_#{piece_jointe.type}"]
         piece_jointe.save
       end
     end

app/views/description/show.html.haml

@@ -1,4 +1,4 @@
-.container
+.container#description_page
   %h2 Description de votre projet
   %br
 
@@ -56,9 +56,7 @@
               Modifier
               %input{type: 'file', name:'cerfa_pdf', id:'cerfa_pdf', accept: ".pdf"}
           -else
-            %span.btn.btn-sm.btn-file.btn-info
-              Choisir
-              %input{type: 'file', name:'cerfa_pdf', id:'cerfa_pdf', accept: ".pdf"}
+            %input{type: 'file', name:'cerfa_pdf', id:'cerfa_pdf', accept: ".pdf"}
 
       - @dossier.pieces_jointes.each do |piece_jointe|
         %tr
@@ -69,9 +67,7 @@
               %span.text-success{ id: "piece_jointe_#{piece_jointe.type}" } Nous l'avons récupéré pour vous.
             -else
               -if piece_jointe.empty?
-                %span.btn.btn-sm.btn-file.btn-info
-                  Choisir
-                  = file_field_tag "piece_jointe_#{piece_jointe.type}", accept: '.pdf'
+                = file_field_tag "piece_jointe_#{piece_jointe.type}", accept: '.pdf'
               -else
                 %span.btn.btn-sm.btn-file.btn-success
                   Modifier

spec/features/description_page/upload_piece_justificative_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+feature 'user is on description page' do
+  let(:dossier) { create(:dossier, :with_entreprise) }
+  before do
+    visit dossier_description_path dossier
+  end
+  it { expect(page).to have_css('#description_page') }
+
+  context 'he fill description fields' do
+    before do
+      find_by_id('nom_projet').set 'mon nom'
+      find_by_id('description').set 'ma description'
+      find_by_id('montant_projet').set 10_000
+      find_by_id('montant_aide_demande').set 100
+      find_by_id('date_previsionnelle').set '10/10/2010'
+      find_by_id('mail_contact').set 'plop@plop.com'
+    end
+    context 'before submit' do
+      it 'dossier cerfa is empty' do
+        expect(dossier.cerfa).to be_empty
+      end
+      it 'pieces_jointes are empty' do
+        dossier.pieces_jointes.each do |piece_jointe|
+          expect(piece_jointe).to be_empty
+        end
+      end
+    end
+    context 'he adds cerfa' do
+      before do
+        attach_file('cerfa_pdf', File.path('spec/support/files/dossierPDF.pdf'))
+        click_on("Terminer la procédure")
+        dossier.reload
+      end
+      it 'fills dossier cerfa' do
+        expect(dossier.cerfa).not_to be_empty
+      end
+    end
+    context 'when he adds a piece_jointe and submit form' do
+      before do
+        file_input_id = 'piece_jointe_' + dossier.pieces_jointes.first.type.to_s
+        attach_file(file_input_id, File.path('spec/support/files/dossierPDF.pdf'))
+        click_on('Terminer la procédure')
+        dossier.reload
+      end
+      scenario 'fills the given piece_jointe' do
+        expect(dossier.pieces_jointes.first).not_to be_empty
+      end
+    end
+  end
+end