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

fix(API): remove notification to group when instructeurs removed

Browse source
This commit is contained in:
Eric Leroy-Terquem 2023-03-02 10:17:31 +01:00
parent 6c277e8678
commit 72b6dacfbd
2 changed files with 1 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -14,11 +14,8 @@ module Mutations
if instructeurs.present? if instructeurs.present?
instructeurs.each { groupe_instructeur.remove(_1) } instructeurs.each { groupe_instructeur.remove(_1) }
groupe_instructeur.reload
GroupeInstructeurMailer groupe_instructeur.reload
.notify_group_when_instructeurs_removed(groupe_instructeur, instructeurs, current_administrateur.email)
.deliver_later
instructeurs.each do |instructeur| instructeurs.each do |instructeur|
GroupeInstructeurMailer GroupeInstructeurMailer

12
spec/controllers/api/v2/graphql_controller_stored_queries_spec.rb
View file

@ -372,11 +372,6 @@ describe API::V2::GraphqlController do
expect(gql_data[:groupeInstructeurAjouterInstructeurs][:groupeInstructeur][:id]).to eq(groupe_instructeur.to_typed_id) expect(gql_data[:groupeInstructeurAjouterInstructeurs][:groupeInstructeur][:id]).to eq(groupe_instructeur.to_typed_id)
expect(groupe_instructeur.instructeurs.count).to eq(2) expect(groupe_instructeur.instructeurs.count).to eq(2)
expect(gql_data[:groupeInstructeurAjouterInstructeurs][:groupeInstructeur][:instructeurs]).to eq([{ id: existing_instructeur.to_typed_id, email: existing_instructeur.email }, { id: Instructeur.last.to_typed_id, email: }]) expect(gql_data[:groupeInstructeurAjouterInstructeurs][:groupeInstructeur][:instructeurs]).to eq([{ id: existing_instructeur.to_typed_id, email: existing_instructeur.email }, { id: Instructeur.last.to_typed_id, email: }])
expect(GroupeInstructeurMailer).to have_received(:notify_added_instructeurs).with(
groupe_instructeur,
[Instructeur.last],
admin.email
)
} }
end end
@ -390,8 +385,6 @@ describe API::V2::GraphqlController do
let(:operation_name) { 'groupeInstructeurSupprimerInstructeurs' } let(:operation_name) { 'groupeInstructeurSupprimerInstructeurs' }
before do before do
allow(GroupeInstructeurMailer).to receive(:notify_group_when_instructeurs_removed)
.and_return(double(deliver_later: true))
allow(GroupeInstructeurMailer).to receive(:notify_removed_instructeur) allow(GroupeInstructeurMailer).to receive(:notify_removed_instructeur)
.and_return(double(deliver_later: true)) .and_return(double(deliver_later: true))
existing_instructeur existing_instructeur
@ -406,11 +399,6 @@ describe API::V2::GraphqlController do
expect(gql_data[:groupeInstructeurSupprimerInstructeurs][:groupeInstructeur][:id]).to eq(groupe_instructeur.to_typed_id) expect(gql_data[:groupeInstructeurSupprimerInstructeurs][:groupeInstructeur][:id]).to eq(groupe_instructeur.to_typed_id)
expect(groupe_instructeur.instructeurs.count).to eq(1) expect(groupe_instructeur.instructeurs.count).to eq(1)
expect(gql_data[:groupeInstructeurSupprimerInstructeurs][:groupeInstructeur][:instructeurs]).to eq([{ id: existing_instructeur.to_typed_id, email: existing_instructeur.email }]) expect(gql_data[:groupeInstructeurSupprimerInstructeurs][:groupeInstructeur][:instructeurs]).to eq([{ id: existing_instructeur.to_typed_id, email: existing_instructeur.email }])
expect(GroupeInstructeurMailer).to have_received(:notify_group_when_instructeurs_removed).with(
groupe_instructeur,
[instructeur_2, instructeur_3],
admin.email
)
expect(GroupeInstructeurMailer).to have_received(:notify_removed_instructeur).twice expect(GroupeInstructeurMailer).to have_received(:notify_removed_instructeur).twice
} }
end end