Merge branch 'qa-shl-consider-onboarding-on-login' into 'master'

Add ability to skip Guided Tour if shown

See merge request gitlab-org/gitlab!35217

ee/app/assets/javascripts/onboarding/onboarding_welcome/components/welcome_page.vue

@@ -129,7 +129,7 @@ export default {
         {{ __("Ok let's go") }}
       </gl-link>
       <p class="small mt-8">
-        <gl-link class="qa-skip-tour-btn" @click="skipTour">
+        <gl-link class="qa-skip-tour-btn" data-qa-selector="skip_for_now_link" @click="skipTour">
           {{ skipText }}
         </gl-link>
       </p>

qa/qa.rb

@@ -183,6 +183,7 @@ module QA
       autoload :OAuth, 'qa/page/main/oauth'
       autoload :SignUp, 'qa/page/main/sign_up'
       autoload :Terms, 'qa/page/main/terms'
+      autoload :Onboarding, 'qa/page/main/onboarding'
     end
 
     module Settings

qa/qa/ee.rb

@@ -49,6 +49,7 @@ module QA
 
       module Main
         autoload :Banner, 'qa/ee/page/main/banner'
+        autoload :Onboarding, 'qa/ee/page/main/onboarding'
       end
 
       module Admin

qa/qa/ee/page/main/onboarding.rb

+# frozen_string_literal: true
+
+module QA
+  module EE
+    module Page
+      module Main
+        module Onboarding
+          extend QA::Page::PageConcern
+
+          def self.prepended(base)
+            super
+
+            base.class_eval do
+              view 'ee/app/assets/javascripts/onboarding/onboarding_welcome/components/welcome_page.vue' do
+                element :skip_for_now_link, required: true
+              end
+            end
+          end
+
+          def skip_for_now
+            click_element :skip_for_now_link, ::QA::Page::Main::Menu
+          end
+        end
+      end
+    end
+  end
+end

qa/qa/page/main/login.rb

@@ -165,6 +165,10 @@ module QA
             terms.accept_terms if terms.visible?
           end
 
+          QA::Page::Main::Onboarding.perform do |onboarding|
+            onboarding.skip_for_now if onboarding.respond_to?(:skip_for_now) && onboarding.visible?
+          end
+
           Page::Main::Menu.validate_elements_present! unless skip_page_validation
         end
 

qa/qa/page/main/onboarding.rb

+# frozen_string_literal: true
+
+module QA
+  module Page
+    module Main
+      class Onboarding < Page::Base
+      end
+    end
+  end
+end
+
+QA::Page::Main::Onboarding.prepend_if_ee('QA::EE::Page::Main::Onboarding')

qa/qa/page/main/terms.rb

@@ -15,7 +15,7 @@ module QA
         end
 
         def accept_terms
-          click_element :accept_terms_button, Page::Main::Menu
+          click_element :accept_terms_button
         end
       end
     end