DGNum/demarches-normaliennes
15
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

DossierController: giving an advice does not remove the notifications

Browse source
This commit is contained in:
Simon Lehericey 2017-05-24 17:55:00 +02:00
parent 3483ec075b
commit 34a7905828
2 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
app/controllers/backoffice/dossiers_controller.rb
View file

@ -32,7 +32,11 @@ class Backoffice::DossiersController < Backoffice::DossiersListController
@headers_private = @champs_private.select { |champ| champ.type_champ == 'header_section' }
end
Notification.where(dossier_id: dossier_id).update_all already_read: true
# if the current_gestionnaire does not own the dossier, it is here to give an advice
# and it should not remove the notifications
if current_gestionnaire.dossiers.find_by(id: dossier_id).present?
Notification.where(dossier_id: dossier_id).update_all(already_read: true)
end
@new_avis = Avis.new(introduction: "Bonjour, merci de me donner votre avis sur ce dossier.")
end

7
spec/controllers/backoffice/dossiers_controller_spec.rb
View file

@ -127,6 +127,13 @@ describe Backoffice::DossiersController, type: :controller do
it { expect(subject.status).to eq(200) }
it { expect(subject.body).to include("Votre avis est sollicité sur le dossier") }
it { expect(subject.body).to_not include("Invitez une personne externe à consulter le dossier et à vous donner un avis sur celui ci.") }
describe 'the notifications are not marked as read' do
it do
expect(Notification).not_to receive(:where)
subject
end
end
end
end