Merge branch 'qa-handle-sign-up-with-terms' into 'master'

[QA] Handle sign-up when terms must be accepted

See merge request gitlab-org/gitlab-ce!22374

app/views/devise/shared/_signup_box.html.haml

@@ -24,7 +24,7 @@
         %p.gl-field-hint.text-secondary Minimum length is #{@minimum_password_length} characters
       - if Gitlab::CurrentSettings.current_application_settings.enforce_terms?
         .form-group
-          = check_box_tag :terms_opt_in, '1', false, required: true
+          = check_box_tag :terms_opt_in, '1', false, required: true, class: 'qa-new-user-accept-terms'
           = label_tag :terms_opt_in do
             - terms_link = link_to s_("I accept the|Terms of Service and Privacy Policy"), terms_path, target: "_blank"
             - accept_terms_label = _("I accept the %{terms_link}") % { terms_link: terms_link }

qa/qa/page/main/sign_up.rb

@@ -11,6 +11,7 @@ module QA
           element :new_user_email_confirmation
           element :new_user_password
           element :new_user_register_button
+          element :new_user_accept_terms
         end
         def sign_up!(user)
@@ -20,6 +21,8 @@ module QA
           fill_element :new_user_email_confirmation, user.email
           fill_element :new_user_password, user.password
+          check_element :new_user_accept_terms if has_element?(:new_user_accept_terms)
+
           signed_in = with_retry do
             click_element :new_user_register_button