Merge branch 'qa/de-quarantine-issue-analytics-test' into 'master'

De-quarantine qa test

Closes gitlab-org/quality/nightly#131

See merge request gitlab-org/gitlab-ee!16325

ee/app/assets/javascripts/issues_analytics/components/issues_analytics.vue

@@ -134,7 +134,7 @@ export default {
 };
 </script>
 <template>
-  <div class="issues-analytics-wrapper">
+  <div class="issues-analytics-wrapper" data-qa-selector="issues_analytics_wrapper">
     <div v-if="loading" class="issues-analytics-loading text-center">
       <gl-loading-icon :inline="true" :size="4" />
     </div>

qa/qa/ee/page/group/issues_analytics.rb

@@ -7,10 +7,24 @@ module QA
         class IssuesAnalytics < QA::Page::Base
           view 'ee/app/assets/javascripts/issues_analytics/components/issues_analytics.vue' do
             element :issues_analytics_graph
+            element :issues_analytics_wrapper
           end
 
           def graph
-            find_element(:issues_analytics_graph)
+            wait_issues_analytics_graph_finish_loading do
+              find_element(:issues_analytics_graph)
+            end
+          end
+
+          private
+
+          def wait_issues_analytics_graph_finish_loading
+            within_element(:issues_analytics_wrapper) do
+              wait(reload: false, max: 5, interval: 1) do
+                finished_loading?
+                yield
+              end
+            end
           end
         end
       end

qa/qa/specs/features/ee/browser_ui/2_plan/issues_analytics/issues_analytics_spec.rb

 # frozen_string_literal: true
 
 module QA
-  # Failure issue: https://gitlab.com/gitlab-org/quality/nightly/issues/131
-  context 'Plan', :quarantine do
+  context 'Plan' do
     describe 'Issues analytics' do
       let(:issue) do
         Resource::Issue.fabricate_via_api! do |issue|