Skip to content
Snippets Groups Projects
Unverified Commit fa5c5221 authored by Yorick Peterse's avatar Yorick Peterse
Browse files

Fix global state in registrations_controller_spec

This spec would mess with global state used for captchas, but not reset
the state in all cases. This would then lead to the spec
spec/controllers/snippets_controller_spec.rb failing, but only on a fork
of gitlab-ee. To fix this we ensure the state is properly reset after
running the specs that change this state.
parent a76643ad
No related branches found
No related tags found
No related merge requests found
Loading
Loading
@@ -83,6 +83,13 @@ describe RegistrationsController do
stub_application_setting(recaptcha_enabled: true)
end
 
after do
# Avoid test ordering issue and ensure `verify_recaptcha` returns true
unless Recaptcha.configuration.skip_verify_env.include?('test')
Recaptcha.configuration.skip_verify_env << 'test'
end
end
it 'displays an error when the reCAPTCHA is not solved' do
fail_recaptcha
 
Loading
Loading
@@ -93,11 +100,6 @@ describe RegistrationsController do
end
 
it 'redirects to the dashboard when the recaptcha is solved' do
# Avoid test ordering issue and ensure `verify_recaptcha` returns true
unless Recaptcha.configuration.skip_verify_env.include?('test')
Recaptcha.configuration.skip_verify_env << 'test'
end
post(:create, params: user_params)
 
expect(flash[:notice]).to include 'Welcome! You have signed up successfully.'
Loading
Loading
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