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

fix(dossier): dossier.repasser_en_construction! should take disable_notification

Browse source
This commit is contained in:
Paul Chavard 2022-12-04 12:34:34 +01:00
parent 6fc8d66956
commit bb47e8d2ab
4 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/controllers/instructeurs/dossiers_controller.rb
View file

@ -134,7 +134,7 @@ module Instructeurs
def repasser_en_construction def repasser_en_construction
begin begin
dossier.repasser_en_construction!(current_instructeur) dossier.repasser_en_construction!(instructeur: current_instructeur)
flash.notice = 'Dossier repassé en construction.' flash.notice = 'Dossier repassé en construction.'
rescue AASM::InvalidTransition => e rescue AASM::InvalidTransition => e
flash.alert = aasm_error_message(e, target_state: :en_construction) flash.alert = aasm_error_message(e, target_state: :en_construction)

2
app/graphql/mutations/dossier_repasser_en_construction.rb
View file

@ -12,7 +12,7 @@ module Mutations
field :errors, [Types::ValidationErrorType], null: true field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, instructeur:, disable_notification:) def resolve(dossier:, instructeur:, disable_notification:)
dossier.repasser_en_construction!(instructeur) dossier.repasser_en_construction!(instructeur: instructeur, disable_notification: disable_notification)
{ dossier: dossier } { dossier: dossier }
end end

4
app/models/dossier.rb
View file

@ -860,7 +860,9 @@ class Dossier < ApplicationRecord
log_automatic_dossier_operation(:passer_en_instruction) log_automatic_dossier_operation(:passer_en_instruction)
end end
def after_repasser_en_construction(instructeur) def after_repasser_en_construction(h)
instructeur = h[:instructeur]
create_missing_traitemets create_missing_traitemets
self.en_construction_close_to_expiration_notice_sent_at = nil self.en_construction_close_to_expiration_notice_sent_at = nil

4
spec/models/dossier_spec.rb
View file

@ -493,7 +493,7 @@ describe Dossier do
it 'should keep first en_construction_at date' do it 'should keep first en_construction_at date' do
Timecop.return Timecop.return
dossier.passer_en_instruction!(instructeur: instructeur) dossier.passer_en_instruction!(instructeur: instructeur)
dossier.repasser_en_construction!(instructeur) dossier.repasser_en_construction!(instructeur: instructeur)
expect(dossier.traitements.size).to eq(3) expect(dossier.traitements.size).to eq(3)
expect(dossier.traitements.first.processed_at).to eq(beginning_of_day) expect(dossier.traitements.first.processed_at).to eq(beginning_of_day)
@ -519,7 +519,7 @@ describe Dossier do
it 'should keep first en_instruction_at date if dossier is set to en_construction again' do it 'should keep first en_instruction_at date if dossier is set to en_construction again' do
Timecop.return Timecop.return
dossier.repasser_en_construction!(instructeur) dossier.repasser_en_construction!(instructeur: instructeur)
dossier.passer_en_instruction!(instructeur: instructeur) dossier.passer_en_instruction!(instructeur: instructeur)
expect(dossier.traitements.size).to eq(4) expect(dossier.traitements.size).to eq(4)