Commit 43141f7d authored by Stan Hu's avatar Stan Hu

Merge branch 'correctly-stub-application-settings-in_signup_spec' into 'master'

Correctly stub application settings in signup_spec.rb

See merge request !11151
parent 00d9a0e3
...@@ -3,7 +3,7 @@ require 'spec_helper' ...@@ -3,7 +3,7 @@ require 'spec_helper'
feature 'Signup', feature: true do feature 'Signup', feature: true do
describe 'signup with no errors' do describe 'signup with no errors' do
context "when sending confirmation email" do context "when sending confirmation email" do
before { allow_any_instance_of(ApplicationSetting).to receive(:send_user_confirmation_email).and_return(true) } before { stub_application_setting(send_user_confirmation_email: true) }
it 'creates the user account and sends a confirmation email' do it 'creates the user account and sends a confirmation email' do
user = build(:user) user = build(:user)
...@@ -23,7 +23,7 @@ feature 'Signup', feature: true do ...@@ -23,7 +23,7 @@ feature 'Signup', feature: true do
end end
context "when not sending confirmation email" do context "when not sending confirmation email" do
before { allow_any_instance_of(ApplicationSetting).to receive(:send_user_confirmation_email).and_return(false) } before { stub_application_setting(send_user_confirmation_email: false) }
it 'creates the user account and goes to dashboard' do it 'creates the user account and goes to dashboard' do
user = build(:user) user = build(:user)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment