Merge branch '198393-broken-foss-master-in-qa-internal' into 'master'

Resolve "Broken FOSS `master` in `qa:internal`"

Closes #198393

See merge request gitlab-org/gitlab!23528

.gitlab/ci/global.gitlab-ci.yml

@@ -13,7 +13,7 @@
 .default-before_script:
   before_script:
     - date
-    - '[ "$FOSS_ONLY" = "1" ] && rm -rf ee/'
+    - '[ "$FOSS_ONLY" = "1" ] && rm -rf ee/ qa/spec/ee/ qa/qa/specs/features/ee/ qa/qa/ee/ qa/qa/ee.rb'
     - export GOPATH=$CI_PROJECT_DIR/.go
     - mkdir -p $GOPATH
     - source scripts/utils.sh

.gitlab/ci/qa.gitlab-ci.yml

@@ -40,7 +40,7 @@
     paths:
       - vendor/ruby
   before_script:
-    - '[ "$FOSS_ONLY" = "1" ] && rm -rf ee/'
+    - '[ "$FOSS_ONLY" = "1" ] && rm -rf ee/ qa/spec/ee/ qa/qa/specs/features/ee/ qa/qa/ee/ qa/qa/ee.rb'
     - cd qa/
     - bundle install --clean --jobs=$(nproc) --path=vendor --retry=3 --quiet
     - bundle check
@@ -50,6 +50,13 @@ qa:internal:
   script:
     - bundle exec rspec
 
+qa:internal-foss:
+  extends:
+    - .qa-job-base
+    - .only-ee-as-if-foss
+  script:
+    - bundle exec rspec
+
 qa:selectors:
   extends: .qa-job-base
   script:

qa/qa/ee/strategy.rb

@@ -6,11 +6,16 @@ module QA
       extend self
 
       def extend_autoloads!
+        require 'qa/ce/strategy'
         require 'qa/ee'
       end
 
       def perform_before_hooks
-        return unless ENV['EE_LICENSE']
+        # Without a license, perform the CE before hooks only.
+        unless ENV['EE_LICENSE']
+          QA::CE::Strategy.perform_before_hooks
+          return
+        end
 
         QA::Support::Retrier.retry_on_exception do
           QA::Runtime::Browser.visit(:gitlab, QA::Page::Main::Login) do

qa/qa/runtime/browser.rb

@@ -15,10 +15,6 @@ module QA
 
       CAPYBARA_MAX_WAIT_TIME = 10
 
-      class << self
-        attr_accessor :rspec_configured, :capybara_configured
-      end
-
       def initialize
         self.class.configure!
       end
@@ -49,49 +45,11 @@ module QA
       end
 
       def self.configure!
-        configure_rspec!
-        configure_capybara!
-      end
-
-      def self.configure_rspec!
-        # We don't want to enter this infinite loop:
-        #   Runtime::Release.perform_before_hooks -> `QA::Runtime::Browser.visit` -> configure! -> configure_rspec! -> Runtime::Release.perform_before_hooks
-        # So we make sure this method is called only once.
-        return if self.rspec_configured
-
-        browser = self
-
         RSpec.configure do |config|
           config.define_derived_metadata(file_path: %r{/qa/specs/features/}) do |metadata|
             metadata[:type] = :feature
           end
-
-          config.before(:suite) do
-            unless browser.rspec_configured
-              browser.rspec_configured = true
-
-              ##
-              # Perform before hooks, which are different for CE and EE
-              #
-              begin
-                Runtime::Release.perform_before_hooks
-              rescue
-                saved = Capybara::Screenshot.screenshot_and_save_page
-
-                QA::Runtime::Logger.error("Screenshot: #{saved[:image]}") if saved&.key?(:image)
-                QA::Runtime::Logger.error("HTML capture: #{saved[:html]}") if saved&.key?(:html)
-
-                raise
-              end
-            end
-          end
         end
-      end
-
-      def self.configure_capybara!
-        return if self.capybara_configured
-
-        self.capybara_configured = true
 
         Capybara.server_port = 9887 + ENV['TEST_ENV_NUMBER'].to_i
 

qa/qa/runtime/release.rb

 # frozen_string_literal: true
 
+require 'capybara/rspec'
+require 'capybara-screenshot/rspec'
+
 module QA
   module Runtime
     ##
@@ -24,6 +27,13 @@ module QA
 
       def self.method_missing(name, *args)
         self.new.strategy.public_send(name, *args)
+      rescue
+        saved = Capybara::Screenshot.screenshot_and_save_page
+
+        QA::Runtime::Logger.error("Screenshot: #{saved[:image]}") if saved&.key?(:image)
+        QA::Runtime::Logger.error("HTML capture: #{saved[:html]}") if saved&.key?(:html)
+
+        raise
       end
     end
   end

qa/qa/scenario/template.rb

@@ -23,7 +23,10 @@ module QA
       def perform(options, *args)
         extract_address(:gitlab_address, options, args)
 
-        QA::Runtime::Browser.configure!
+        ##
+        # Perform before hooks, which are different for CE and EE
+        #
+        Runtime::Release.perform_before_hooks
 
         Runtime::Feature.enable(options[:enable_feature]) if options.key?(:enable_feature)
 

qa/qa/scenario/test/instance.rb

@@ -20,7 +20,10 @@ module QA
         def self.do_perform(address, *rspec_options)
           Runtime::Scenario.define(:gitlab_address, address)
 
-          QA::Runtime::Browser.configure!
+          ##
+          # Perform before hooks, which are different for CE and EE
+          #
+          Runtime::Release.perform_before_hooks
 
           Specs::Runner.perform do |specs|
             specs.tty = true

qa/spec/support/shared_examples/scenario_shared_examples.rb

@@ -31,6 +31,12 @@ shared_examples 'a QA scenario class' do
     expect(attributes).to have_received(:define).with(:gitlab_address, 'http://gitlab_address').at_least(:once)
   end
 
+  it 'performs before hooks only once' do
+    subject.perform(args)
+
+    expect(release).to have_received(:perform_before_hooks).once
+  end
+
   it 'sets tags on runner' do
     subject.perform(args)