Merge branch...

Merge branch '524092-visiting-ci-cd-analytics-as-an-non-logged-in-user-yields-an-error' into 'master' 

Allow anonymous users to load pipeline analytics

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/184032



Merged-by: Fred de Gier <fdegier@gitlab.com>
Approved-by: Tiger Watson <twatson@gitlab.com>
Approved-by: Pedro Pombeiro <noreply@pedro.pombei.ro>
Approved-by: Fred de Gier <fdegier@gitlab.com>
Co-authored-by: Miguel Rincon <mrincon@gitlab.com>

app/services/ci/collect_pipeline_analytics_service_base.rb

@@ -38,7 +38,7 @@ def execute
     private
 
     def allowed?
-      current_user&.can?(:read_ci_cd_analytics, container)
+      Ability.allowed?(current_user, :read_ci_cd_analytics, container)
     end
 
     def clickhouse_model

spec/graphql/resolvers/ci/pipeline_analytics_resolver_spec.rb

@@ -119,6 +119,12 @@ def resolve_statistics(container, args)
             project.update!(public_builds: true)
           end
 
+          context 'when user is anonymous' do
+            let(:current_user) { nil }
+
+            it_behaves_like 'returns the pipeline analytics for a given container'
+          end
+
           context 'when user is not a member' do
             let(:current_user) { create(:user) }