Merge branch 'qa-nightly-115-fix-retry-until' into 'master'

CE port of "Fix page refresh in retry_until"

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

(cherry picked from commit d32cdb84)

f4a4f6e2 Fix page refresh in retry_until

qa/qa/page/base.rb

@@ -26,7 +26,7 @@ module QA
       end
       def retry_until(max_attempts: 3, reload: false, sleep_interval: 0)
-        QA::Support::Retrier.retry_until(max_attempts: max_attempts, reload: reload, sleep_interval: sleep_interval) do
+        QA::Support::Retrier.retry_until(max_attempts: max_attempts, reload_page: (reload && self), sleep_interval: sleep_interval) do
           yield
         end
       end

qa/qa/specs/features/browser_ui/1_manage/login/log_in_spec.rb

@@ -11,7 +11,7 @@ module QA
           expect(menu).to have_personal_area
         end
-        Support::Retrier.retry_until(reload: false, sleep_interval: 0.5) do
+        Support::Retrier.retry_until(sleep_interval: 0.5) do
           Page::Main::Menu.perform(&:sign_out)
           Page::Main::Login.perform(&:has_sign_in_tab?)

qa/qa/support/retrier.rb

@@ -24,8 +24,8 @@ module QA
         end
       end
-      def retry_until(max_attempts: 3, reload: false, sleep_interval: 0)
-        QA::Runtime::Logger.debug("with retry_until: max_attempts #{max_attempts}; sleep_interval #{sleep_interval}; reload:#{reload}")
+      def retry_until(max_attempts: 3, reload_page: nil, sleep_interval: 0)
+        QA::Runtime::Logger.debug("with retry_until: max_attempts #{max_attempts}; sleep_interval #{sleep_interval}; reload_page:#{reload_page}")
         attempts = 0
         while attempts < max_attempts
@@ -35,7 +35,7 @@ module QA
           sleep sleep_interval
-          refresh if reload
+          reload_page.refresh if reload_page
           attempts += 1
         end