From 28e5c3bb5116d012a1f58b4cbfe1a5fb302ac227 Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Mon, 20 Apr 2026 12:05:46 -0400 Subject: [PATCH] Add coverage for "no change" scenario in admin change emails (#38754) --- spec/requests/admin/change_emails_spec.rb | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/spec/requests/admin/change_emails_spec.rb b/spec/requests/admin/change_emails_spec.rb index 3df29eb77b..06646a44d3 100644 --- a/spec/requests/admin/change_emails_spec.rb +++ b/spec/requests/admin/change_emails_spec.rb @@ -14,5 +14,17 @@ RSpec.describe 'Admin Account Change Email' do expect(response) .to have_http_status(400) end + + context 'when email is not changed' do + subject { put admin_account_change_email_path(user.account_id, user: { unconfirmed_email: 'original@host.example' }) } + + let(:user) { Fabricate :user, email: 'original@host.example' } + + it 'does not updated the user record' do + expect { expect { subject }.to_not send_email } + .to not_change { user.reload.unconfirmed_email } + .and(not_change { user.reload.updated_at }) + end + end end end