Convert admin/reset controller spec to system spec (#31643)

This commit is contained in:
Matt Jankowski 2024-08-29 04:38:17 -04:00 committed by GitHub
parent 5d725b2c12
commit b9269c8d38
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 46 additions and 36 deletions

View file

@ -52,7 +52,7 @@
- if can?(:reset_password, account.user) - if can?(:reset_password, account.user)
%tr %tr
%td %td
= table_link_to 'key', t('admin.accounts.reset_password'), admin_account_reset_path(account.id), method: :create, data: { confirm: t('admin.accounts.are_you_sure') } = table_link_to 'key', t('admin.accounts.reset_password'), admin_account_reset_path(account.id), method: :post, data: { confirm: t('admin.accounts.are_you_sure') }
%tr %tr
%th= t('simple_form.labels.defaults.locale') %th= t('simple_form.labels.defaults.locale')
%td= standard_locale_name(account.user_locale) %td= standard_locale_name(account.user_locale)

View file

@ -1,35 +0,0 @@
# frozen_string_literal: true
require 'rails_helper'
describe Admin::ResetsController do
render_views
subject { post :create, params: { account_id: account.id } }
let(:account) { Fabricate(:account) }
before do
sign_in Fabricate(:user, role: UserRole.find_by(name: 'Admin')), scope: :user
end
describe 'POST #create', :inline_jobs do
it 'redirects to admin accounts page' do
emails = capture_emails { subject }
expect(emails.size)
.to eq(2)
expect(emails).to have_attributes(
first: have_attributes(
to: include(account.user.email),
subject: I18n.t('devise.mailer.password_change.subject')
),
last: have_attributes(
to: include(account.user.email),
subject: I18n.t('devise.mailer.reset_password_instructions.subject')
)
)
expect(response).to redirect_to(admin_account_path(account.id))
end
end
end

View file

@ -0,0 +1,45 @@
# frozen_string_literal: true
require 'rails_helper'
describe 'Admin::Reset' do
it 'Resets password for account user' do
account = Fabricate :account
sign_in admin_user
visit admin_account_path(account.id)
emails = capture_emails do
expect { submit_reset }
.to change(Admin::ActionLog.where(target: account.user), :count).by(1)
end
expect(emails.first)
.to be_present
.and(deliver_to(account.user.email))
.and(have_subject(password_change_subject))
expect(emails.last)
.to be_present
.and(deliver_to(account.user.email))
.and(have_subject(reset_instructions_subject))
expect(page)
.to have_content(account.username)
end
def admin_user
Fabricate(:user, role: UserRole.find_by(name: 'Admin'))
end
def submit_reset
click_on I18n.t('admin.accounts.reset_password')
end
def password_change_subject
I18n.t('devise.mailer.password_change.subject')
end
def reset_instructions_subject
I18n.t('devise.mailer.reset_password_instructions.subject')
end
end