Skip to content
Snippets Groups Projects
Commit 67bc58dd authored by Akihiko Odaki's avatar Akihiko Odaki Committed by Matt Jankowski
Browse files

Use around hook to restore context in Admin::SettingsController spec (#3428)

parent 2d39560d
No related branches found
No related tags found
No related merge requests found
......@@ -47,20 +47,36 @@ RSpec.describe Admin::SettingsController, type: :controller do
end
end
it 'updates a settings value' do
Setting.site_title = 'Original'
patch :update, params: { site_title: 'New title' }
context do
around do |example|
site_title = Setting.site_title
example.run
Setting.site_title = site_title
end
expect(response).to redirect_to(edit_admin_settings_path)
expect(Setting.site_title).to eq 'New title'
it 'updates a settings value' do
Setting.site_title = 'Original'
patch :update, params: { site_title: 'New title' }
expect(response).to redirect_to(edit_admin_settings_path)
expect(Setting.site_title).to eq 'New title'
end
end
it 'typecasts open_registrations to boolean' do
Setting.open_registrations = false
patch :update, params: { open_registrations: 'true' }
context do
around do |example|
open_registrations = Setting.open_registrations
example.run
Setting.open_registrations = open_registrations
end
expect(response).to redirect_to(edit_admin_settings_path)
expect(Setting.open_registrations).to eq true
it 'typecasts open_registrations to boolean' do
Setting.open_registrations = false
patch :update, params: { open_registrations: 'true' }
expect(response).to redirect_to(edit_admin_settings_path)
expect(Setting.open_registrations).to eq true
end
end
end
end
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment