Merge branch 'qa-check-server-responding-before-suit' into 'master'

Wait for login page to load at the beginning of QA suite

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

qa/qa/page/main/login.rb

@@ -40,16 +40,8 @@ module QA
           element :login_page
         end
 
-        def initialize
-          # The login page is usually the entry point for all the scenarios so
-          # we need to wait for the instance to start. That said, in some cases
-          # we are already logged-in so we check both cases here.
-          # Check if we're already logged in first. If we don't then we have to
-          # wait 10 seconds for the check for the login page to fail every time
-          # we use this class when we're already logged in (E.g., whenever we
-          # create a personal access token to use for API access).
-          wait(max: 500) do
-            Page::Main::Menu.act { has_personal_area?(wait: 0) } ||
+        def page_loaded?
+          wait(max: 60) do
             has_element?(:login_page)
           end
         end

qa/spec/spec_helper.rb

@@ -3,6 +3,20 @@ require_relative '../qa'
 Dir[::File.join(__dir__, 'support', '**', '*.rb')].each { |f| require f }
 
 RSpec.configure do |config|
+  ServerNotRespondingError = Class.new(RuntimeError)
+
+  # The login page could take some time to load the first time it is visited.
+  # We visit the login page and wait for it to properly load only once at the beginning of the suite.
+  config.before(:suite) do
+    if QA::Runtime::Scenario.respond_to?(:gitlab_address)
+      QA::Runtime::Browser.visit(:gitlab, QA::Page::Main::Login)
+
+      unless QA::Page::Main::Login.perform(&:page_loaded?)
+        raise ServerNotRespondingError, "Login page did not load at #{QA::Page::Main::Login.perform(&:current_url)}"
+      end
+    end
+  end
+
   config.before do |example|
     QA::Runtime::Logger.debug("Starting test: #{example.full_description}") if QA::Runtime::Env.debug?