end delete type de piece justificative

app/controllers/admin/pieces_justificatives_controller.rb

@@ -9,6 +9,14 @@ class Admin::PiecesJustificativesController < AdminController
     render 'show', format: :js
   end
 
+  def destroy
+    @procedure.types_de_piece_justificative.find(params[:id]).destroy
+
+    render 'show', format: :js
+  rescue ActiveRecord::RecordNotFound
+    render json: { message: 'Type de piece justificative not found' }, status: 404
+  end
+
   def update_params
     params
       .require(:procedure)

app/views/admin/pieces_justificatives/_fields.html.haml

@@ -13,4 +13,6 @@
       .form-group#add_type_de_piece_justificative_button
         %br  
         = f.submit('Ajouter la pièce')
+    - else
+      = link_to("", admin_procedure_piece_justificative_path(@procedure, ff.object.id), method: :delete, remote: true, id: "delete_type_de_piece_justificative_#{ff.object.id}", class: %w(form-control btn btn-danger fa fa-trash-o) )
 

config/routes.rb

@@ -48,6 +48,7 @@ Rails.application.routes.draw do
       end
       resources :types_de_champ, only: [:destroy]
       resource :pieces_justificatives, only: [:show, :update]
+      resources :pieces_justificatives, only: :destroy
     end
   end
 

spec/controllers/admin/pieces_justificatives_controller_spec.rb

@@ -21,7 +21,7 @@ describe Admin::PiecesJustificativesController, type: :controller  do
     end
   end
 
-  describe 'POST #update' do
+  describe 'PUT #update' do
     let(:procedure) { create(:procedure, administrateur: admin) }
     let(:procedure_id) { procedure.id }
     let(:libelle) { 'RIB' }
@@ -62,4 +62,31 @@ describe Admin::PiecesJustificativesController, type: :controller  do
       it { expect{ subject }.not_to change(TypeDePieceJustificative, :count) }
     end
   end
+
+  describe 'DELETE #destroy' do
+    let(:procedure) { create(:procedure, administrateur: admin) }
+    let!(:pj) { create(:type_de_piece_justificative, procedure: procedure) }
+    let(:procedure_id) { procedure.id }
+    let(:pj_id) { pj.id }
+    let(:request) { delete :destroy, procedure_id: procedure_id, id: pj_id }
+    subject { request }
+    context 'when procedure is not found' do
+      let(:procedure_id) { 9_999_999 }
+      it { expect(subject.status).to eq(404) }
+    end
+    context 'when pj id does not exist' do
+      let(:pj_id) { 9_999_999 }
+      it { expect(subject.status).to eq(404) }
+    end
+    context 'when pj id exist but is not linked to procedure' do
+      let(:procedure_1) { create(:procedure, administrateur: admin) }
+      let!(:pj_1) { create(:type_de_piece_justificative, procedure: procedure_1) }
+      let(:pj_id) { pj_1 }
+      it { expect(subject.status).to eq(404) }
+    end
+    context 'when pj is found' do
+      it { expect(subject.status).to eq(200) }
+      it { expect{ subject }.to change(TypeDePieceJustificative, :count).by(-1) }
+    end
+  end
 end

spec/factories/type_de_piece_justificative.rb

@@ -1,5 +1,9 @@
 FactoryGirl.define do
   factory :type_de_piece_justificative do
+
+    libelle 'RIB'
+    description 'Releve identité bancaire'
+
     trait :rib do
       libelle 'RIB'
       description 'Releve identité bancaire'

spec/features/admin/add_type_de_piece_justificative_spec.rb

@@ -43,6 +43,20 @@ feature 'add a new type de piece justificative', js: true do
           expect(page).to have_css('#procedure_types_de_piece_justificative_attributes_1_libelle')
         end
       end
+      context 'when user delete pj' do
+        before do
+          pj = procedure.types_de_piece_justificative.first
+          page.find_by_id("delete_type_de_piece_justificative_#{pj.id}").click
+          wait_for_ajax
+        end
+        scenario 'removes pj from page' do
+          within '#liste_piece_justificative' do
+            expect(page).not_to have_css('#procedure_types_de_piece_justificative_attributes_0_libelle')
+            expect(page.body).not_to match(libelle)
+            expect(page.body).not_to match(description)
+          end
+        end
+      end
     end
   end
 end