diff options
author | Stan Hu <stanhu@gmail.com> | 2018-04-06 22:38:37 +0000 |
---|---|---|
committer | Filipa Lacerda <filipa@gitlab.com> | 2018-04-09 14:44:04 +0100 |
commit | 126551b7564c9e9e707e68f2349672e1c6e60d32 (patch) | |
tree | 61fec5d009f13895afcf90e4005dbc76f4823884 | |
parent | 8572c3828172cf477f8f016d2d1efb2155a73cc6 (diff) | |
download | gitlab-ce-126551b7564c9e9e707e68f2349672e1c6e60d32.tar.gz |
Merge branch 'sh-fix-recaptcha-test' into 'master'10-6-stable-patch-4
Fix the reCAPTCHA spec failures by not rendering the widget in test mode
Closes #45138
See merge request gitlab-org/gitlab-ce!18235
-rw-r--r-- | app/views/shared/_recaptcha_form.html.haml | 2 | ||||
-rw-r--r-- | spec/features/issues/spam_issues_spec.rb | 3 |
2 files changed, 1 insertions, 4 deletions
diff --git a/app/views/shared/_recaptcha_form.html.haml b/app/views/shared/_recaptcha_form.html.haml index 93a4301f366..a0ba1afc284 100644 --- a/app/views/shared/_recaptcha_form.html.haml +++ b/app/views/shared/_recaptcha_form.html.haml @@ -10,7 +10,7 @@ = hidden_field(resource_name, field, value: value) = hidden_field_tag(:spam_log_id, spammable.spam_log.id) = hidden_field_tag(:recaptcha_verification, true) - = recaptcha_tags script: script, callback: 'recaptchaDialogCallback' + = recaptcha_tags script: script, callback: 'recaptchaDialogCallback' unless Rails.env.test? -# Yields a block with given extra params. = yield diff --git a/spec/features/issues/spam_issues_spec.rb b/spec/features/issues/spam_issues_spec.rb index a75ca1d42b3..73022afbda2 100644 --- a/spec/features/issues/spam_issues_spec.rb +++ b/spec/features/issues/spam_issues_spec.rb @@ -34,9 +34,6 @@ describe 'New issue', :js do click_button 'Submit issue' - # reCAPTCHA alerts when it can't contact the server, so just accept it and move on - page.driver.browser.switch_to.alert.accept - # it is impossible to test recaptcha automatically and there is no possibility to fill in recaptcha # recaptcha verification is skipped in test environment and it always returns true expect(page).not_to have_content('issue title') |