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

feat(graphql): add an option to dossier state change mutations to disable notifications

Browse source
This commit is contained in:
Paul Chavard 2021-09-01 15:11:11 +02:00
parent 9e30d5fc22
commit 74e277d0a2
10 changed files with 86 additions and 34 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -151,15 +151,15 @@ module Instructeurs
case params[:process_action]
when "refuser"
target_state = :refuse
dossier.refuser!(current_instructeur, motivation, justificatif)
dossier.refuser!(current_instructeur, motivation, justificatif: justificatif)
flash.notice = "Dossier considéré comme refusé."
when "classer_sans_suite"
target_state = :sans_suite
dossier.classer_sans_suite!(current_instructeur, motivation, justificatif)
dossier.classer_sans_suite!(current_instructeur, motivation, justificatif: justificatif)
flash.notice = "Dossier considéré comme sans suite."
when "accepter"
target_state = :accepte
dossier.accepter!(current_instructeur, motivation, justificatif)
dossier.accepter!(current_instructeur, motivation, justificatif: justificatif)
flash.notice = "Dossier traité avec succès."
end
rescue AASM::InvalidTransition => e

5
app/graphql/mutations/dossier_accepter.rb
View file

@ -8,12 +8,13 @@ module Mutations
argument :instructeur_id, ID, "Instructeur qui prend la décision sur le dossier.", required: true, loads: Types::ProfileType
argument :motivation, String, required: false
argument :justificatif, ID, required: false
argument :disable_notification, Boolean, "Désactiver l’envoi de l’email de notification après l’opération", required: false, default_value: false
field :dossier, Types::DossierType, null: true
field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, instructeur:, motivation: nil, justificatif: nil)
dossier.accepter!(instructeur, motivation, justificatif)
def resolve(dossier:, instructeur:, motivation: nil, justificatif: nil, disable_notification:)
dossier.accepter!(instructeur, motivation, justificatif: justificatif, disable_notification: disable_notification)
{ dossier: dossier }
end

5
app/graphql/mutations/dossier_classer_sans_suite.rb
View file

@ -8,12 +8,13 @@ module Mutations
argument :instructeur_id, ID, "Instructeur qui prend la décision sur le dossier.", required: true, loads: Types::ProfileType
argument :motivation, String, required: true
argument :justificatif, ID, required: false
argument :disable_notification, Boolean, "Désactiver l’envoi de l’email de notification après l’opération", required: false, default_value: false
field :dossier, Types::DossierType, null: true
field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, instructeur:, motivation:, justificatif: nil)
dossier.classer_sans_suite!(instructeur, motivation, justificatif)
def resolve(dossier:, instructeur:, motivation:, justificatif: nil, disable_notification:)
dossier.classer_sans_suite!(instructeur, motivation, justificatif: justificatif, disable_notification: disable_notification)
{ dossier: dossier }
end

7
app/graphql/mutations/dossier_passer_en_instruction.rb
View file

@ -6,17 +6,18 @@ module Mutations
argument :dossier_id, ID, "Dossier ID", required: true, loads: Types::DossierType
argument :instructeur_id, ID, "Instructeur qui prend la décision sur le dossier.", required: true, loads: Types::ProfileType
argument :disable_notification, Boolean, "Désactiver l’envoi de l’email de notification après l’opération", required: false, default_value: false
field :dossier, Types::DossierType, null: true
field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, instructeur:)
dossier.passer_en_instruction!(instructeur)
def resolve(dossier:, instructeur:, disable_notification:)
dossier.passer_en_instruction!(instructeur, disable_notification: disable_notification)
{ dossier: dossier }
end
def authorized?(dossier:, instructeur:)
def authorized?(dossier:, instructeur:, **args)
if !dossier.en_construction?
return false, { errors: ["Le dossier est déjà #{dossier_display_state(dossier, lower: true)}"] }
end

5
app/graphql/mutations/dossier_refuser.rb
View file

@ -8,12 +8,13 @@ module Mutations
argument :instructeur_id, ID, "Instructeur qui prend la décision sur le dossier.", required: true, loads: Types::ProfileType
argument :motivation, String, required: true
argument :justificatif, ID, required: false
argument :disable_notification, Boolean, "Désactiver l’envoi de l’email de notification après l’opération", required: false, default_value: false
field :dossier, Types::DossierType, null: true
field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, instructeur:, motivation:, justificatif: nil)
dossier.refuser!(instructeur, motivation, justificatif)
def resolve(dossier:, instructeur:, motivation:, justificatif: nil, disable_notification:)
dossier.refuser!(instructeur, motivation, justificatif: justificatif, disable_notification: disable_notification)
{ dossier: dossier }
end

20
app/graphql/schema.graphql
View file

@ -748,6 +748,11 @@ input DossierAccepterInput {
"""
clientMutationId: String
"""
Désactiver l’envoi de l’email de notification après l’opération
"""
disableNotification: Boolean = false
"""
Dossier ID
"""
@ -846,6 +851,11 @@ input DossierClasserSansSuiteInput {
"""
clientMutationId: String
"""
Désactiver l’envoi de l’email de notification après l’opération
"""
disableNotification: Boolean = false
"""
Dossier ID
"""
@ -1164,6 +1174,11 @@ input DossierPasserEnInstructionInput {
"""
clientMutationId: String
"""
Désactiver l’envoi de l’email de notification après l’opération
"""
disableNotification: Boolean = false
"""
Dossier ID
"""
@ -1196,6 +1211,11 @@ input DossierRefuserInput {
"""
clientMutationId: String
"""
Désactiver l’envoi de l’email de notification après l’opération
"""
disableNotification: Boolean = false
"""
Dossier ID
"""

36
app/models/dossier.rb
View file

@ -374,7 +374,6 @@ class Dossier < ApplicationRecord
before_save :build_default_champs, if: Proc.new { revision_id_was.nil? }
before_save :update_search_terms
after_save :send_dossier_en_instruction
after_save :send_web_hook
after_create_commit :send_draft_notification_email
@ -676,10 +675,13 @@ class Dossier < ApplicationRecord
update!(en_construction_at: Time.zone.now) if self.en_construction_at.nil?
end
def after_passer_en_instruction(instructeur)
def after_passer_en_instruction(instructeur, disable_notification: false)
instructeur.follow(self)
update!(en_instruction_at: Time.zone.now) if self.en_instruction_at.nil?
if !procedure.declarative_accepte? && !disable_notification
NotificationMailer.send_en_instruction_notification(self).deliver_later
end
log_dossier_operation(instructeur, :passer_en_instruction)
end
@ -695,17 +697,19 @@ class Dossier < ApplicationRecord
log_dossier_operation(instructeur, :repasser_en_construction)
end
def after_repasser_en_instruction(instructeur)
def after_repasser_en_instruction(instructeur, disable_notification: false)
self.archived = false
self.en_instruction_at = Time.zone.now
attestation&.destroy
save!
DossierMailer.notify_revert_to_instruction(self).deliver_later
if !disable_notification
DossierMailer.notify_revert_to_instruction(self).deliver_later
end
log_dossier_operation(instructeur, :repasser_en_instruction)
end
def after_accepter(instructeur, motivation, justificatif = nil)
def after_accepter(instructeur, motivation, justificatif: nil, disable_notification: false)
self.traitements.accepter(motivation: motivation, instructeur: instructeur)
if justificatif
@ -718,7 +722,9 @@ class Dossier < ApplicationRecord
save!
remove_titres_identite!
NotificationMailer.send_accepte_notification(self).deliver_later
if !disable_notification
NotificationMailer.send_accepte_notification(self).deliver_later
end
send_dossier_decision_to_experts(self)
log_dossier_operation(instructeur, :accepter, self)
end
@ -738,7 +744,7 @@ class Dossier < ApplicationRecord
log_automatic_dossier_operation(:accepter, self)
end
def after_refuser(instructeur, motivation, justificatif = nil)
def after_refuser(instructeur, motivation, justificatif: nil, disable_notification: false)
self.traitements.refuser(motivation: motivation, instructeur: instructeur)
if justificatif
@ -747,12 +753,14 @@ class Dossier < ApplicationRecord
save!
remove_titres_identite!
NotificationMailer.send_refuse_notification(self).deliver_later
if !disable_notification
NotificationMailer.send_refuse_notification(self).deliver_later
end
send_dossier_decision_to_experts(self)
log_dossier_operation(instructeur, :refuser, self)
end
def after_classer_sans_suite(instructeur, motivation, justificatif = nil)
def after_classer_sans_suite(instructeur, motivation, justificatif: nil, disable_notification: false)
self.traitements.classer_sans_suite(motivation: motivation, instructeur: instructeur)
if justificatif
@ -761,7 +769,9 @@ class Dossier < ApplicationRecord
save!
remove_titres_identite!
NotificationMailer.send_sans_suite_notification(self).deliver_later
if !disable_notification
NotificationMailer.send_sans_suite_notification(self).deliver_later
end
send_dossier_decision_to_experts(self)
log_dossier_operation(instructeur, :classer_sans_suite, self)
end
@ -968,12 +978,6 @@ class Dossier < ApplicationRecord
end
end
def send_dossier_en_instruction
if saved_change_to_state? && en_instruction? && !procedure.declarative_accepte?
NotificationMailer.send_en_instruction_notification(self).deliver_later
end
end
def send_draft_notification_email
if brouillon? && !procedure.declarative?
DossierMailer.notify_new_draft(self).deliver_later

26
spec/controllers/api/v2/graphql_controller_spec.rb
View file

@ -904,11 +904,13 @@ describe API::V2::GraphqlController do
describe 'dossierPasserEnInstruction' do
let(:dossier) { create(:dossier, :en_construction, :with_individual, procedure: procedure) }
let(:instructeur_id) { instructeur.to_typed_id }
let(:disable_notification) { false }
let(:query) do
"mutation {
dossierPasserEnInstruction(input: {
dossierId: \"#{dossier.to_typed_id}\",
instructeurId: \"#{instructeur_id}\"
instructeurId: \"#{instructeur_id}\",
disableNotification: #{disable_notification}
}) {
dossier {
id
@ -934,6 +936,9 @@ describe API::V2::GraphqlController do
},
errors: nil
})
perform_enqueued_jobs
expect(ActionMailer::Base.deliveries.size).to eq(4)
end
end
@ -960,6 +965,25 @@ describe API::V2::GraphqlController do
})
end
end
context 'disable notification' do
let(:disable_notification) { true }
it "should passer en instruction dossier without notification" do
expect(gql_errors).to eq(nil)
expect(gql_data).to eq(dossierPasserEnInstruction: {
dossier: {
id: dossier.to_typed_id,
state: "en_instruction",
motivation: nil
},
errors: nil
})
perform_enqueued_jobs
expect(ActionMailer::Base.deliveries.size).to eq(3)
end
end
end
describe 'dossierClasserSansSuite' do

2
spec/models/concern/tags_substitution_concern_spec.rb
View file

@ -320,7 +320,7 @@ describe TagsSubstitutionConcern, type: :model do
Timecop.freeze(Time.zone.local(2004, 5, 6))
dossier.passer_en_instruction!(instructeur)
Timecop.freeze(Time.zone.local(2007, 8, 9))
dossier.accepter!(instructeur, nil, nil)
dossier.accepter!(instructeur, nil)
Timecop.return
end

8
spec/models/dossier_spec.rb
View file

@ -239,7 +239,7 @@ describe Dossier do
Timecop.freeze(date2)
d.passer_en_instruction!(instructeur)
Timecop.freeze(date3)
d.accepter!(instructeur, "Motivation", nil)
d.accepter!(instructeur, "Motivation")
Timecop.return
d
end
@ -422,7 +422,7 @@ describe Dossier do
let(:dossier) { create(:dossier, :en_instruction, :with_individual) }
before do
dossier.accepter!(instructeur, nil, nil)
dossier.accepter!(instructeur, nil)
dossier.reload
end
@ -435,7 +435,7 @@ describe Dossier do
let(:dossier) { create(:dossier, :en_instruction, :with_individual) }
before do
dossier.refuser!(instructeur, nil, nil)
dossier.refuser!(instructeur, nil)
dossier.reload
end
@ -447,7 +447,7 @@ describe Dossier do
let(:dossier) { create(:dossier, :en_instruction, :with_individual) }
before do
dossier.classer_sans_suite!(instructeur, nil, nil)
dossier.classer_sans_suite!(instructeur, nil)
dossier.reload
end