[issue-4351] edit test case to reflect old mail sent

zooniverse · Aug 16, 2024 · d015acf · d015acf
1 parent aae5b02
commit d015acf
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/app/controllers/api/v1/users_controller.rb b/app/controllers/api/v1/users_controller.rb
@@ -57,7 +57,6 @@ def update
       user.update(valid_email: true)
       UserInfoChangedMailerWorker.perform_async(user.id, 'email', prev_email)
     end
-
   end
 
   def destroy
@@ -120,4 +119,4 @@ def filter_by_params
   def downcase_filter_params(param_value)
     param_value.split(',').map(&:downcase)
   end
-end
+end
diff --git a/app/mailers/user_info_changed_mailer.rb b/app/mailers/user_info_changed_mailer.rb
@@ -15,6 +15,6 @@ def user_info_changed(user, info, previous_email=nil)
       template = "password_changed"
     end
 
-    mail(to: @recipient_emails, subject: subject, template_name: template )
+    mail(to: @recipient_emails, subject: subject, template_name: template)
   end
 end
diff --git a/spec/controllers/api/v1/users_controller_spec.rb b/spec/controllers/api/v1/users_controller_spec.rb
@@ -479,7 +479,7 @@ def update_request
         update_request
       end
 
-      it "sends an email to the new address if user is valid" do
+      it "sends an email to the new and old address if user is valid" do
         expect(UserInfoChangedMailerWorker).to receive(:perform_async).with(user.id, "email", user_email)
       end