Add latest changes from gitlab-org/gitlab@master

doc/subscriptions/gitlab_com/index.md

@@ -192,7 +192,7 @@ To add seats to a subscription:
 1. Select **Add more seats** on the relevant subscription card.
 1. Enter the number of additional users.
 1. Review the **Purchase summary** section. The system lists the total price for all users on the
-   system and a credit for what you've already paid. You are only be charged for the net change.
+   system and a credit for what you've already paid. You are only charged for the net change.
 1. Enter your payment information.
 1. Select **Purchase seats**.
 

qa/qa/ce/strategy.rb

@@ -20,10 +20,10 @@ module QA
           )
         end
 
-        QA::Runtime::Logger.info("Browser: #{QA::Runtime::Env.browser}")
+        QA::Runtime::Logger.info("Using Browser: #{QA::Runtime::Env.browser}")
 
         if QA::Runtime::Env.use_selenoid?
-          QA::Runtime::Logger.info("Browser version: #{QA::Runtime::Env.selenoid_browser_version}")
+          QA::Runtime::Logger.info("Using Selenoid Browser version: #{QA::Runtime::Env.selenoid_browser_version}")
         end
 
         # The login page could take some time to load the first time it is visited.

qa/qa/runtime/browser.rb

@@ -43,6 +43,7 @@ module QA
       end
 
       def self.configure! # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+        return if QA::Runtime::Env.dry_run
         return if @configured
 
         RSpec.configure do |config|

qa/qa/scenario/template.rb

@@ -32,16 +32,6 @@ module QA
         # Set large setup attribute
         Runtime::Scenario.define(:large_setup?, args.include?('can_use_large_setup'))
 
-        ##
-        # Configure browser
-        #
-        Runtime::Browser.configure!
-
-        ##
-        # Perform before hooks, which are different for CE and EE
-        #
-        QA::Runtime::Release.perform_before_hooks unless QA::Runtime::Env.dry_run
-
         Specs::Runner.perform do |specs|
           specs.tty = true
           specs.tags = self.class.focus

qa/qa/specs/spec_helper.rb

@@ -14,7 +14,7 @@ Knapsack::Adapters::RSpecAdapter.bind if QA::Runtime::Env.knapsack?
 
 # TODO: move all classes that perform rspec configuration under spec/helpers
 QA::Support::GitlabAddress.define_gitlab_address_attribute!
-QA::Runtime::Browser.configure! unless QA::Runtime::Env.dry_run
+QA::Runtime::Browser.configure!
 QA::Specs::Helpers::FeatureSetup.configure!
 QA::Runtime::AllureReport.configure!
 QA::Runtime::Scenario.from_env(QA::Runtime::Env.runtime_scenario_attributes)
@@ -51,6 +51,11 @@ RSpec.configure do |config|
     Thread.current[:browser_ui_fabrication] = 0
   end
 
+  config.prepend_before(:suite) do
+    # Perform before hooks at the very start of the test run
+    QA::Runtime::Release.perform_before_hooks unless QA::Runtime::Env.dry_run
+  end
+
   config.before(:suite) do
     FactoryBot.find_definitions
   end

qa/spec/scenario_shared_examples.rb

@@ -28,12 +28,6 @@ module QA
       expect(subject).to respond_to(:perform)
     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)