Merge branch '217469-follow-up-from-fix-minutes-notification-when-unauthenticated' into 'master'

Allow developers to see ci minutes notifications

Closes #217469

See merge request gitlab-org/gitlab!31937

ee/app/policies/ee/group_policy.rb

@@ -113,7 +113,6 @@ module EE
         enable :create_jira_connect_subscription
         enable :maintainer_access
         enable :admin_wiki
-        enable :read_ci_minutes_quota
       end
 
       rule { owner }.policy do
@@ -198,6 +197,7 @@ module EE
       rule { developer }.policy do
         enable :create_wiki
         enable :admin_merge_request
+        enable :read_ci_minutes_quota
       end
 
       rule { security_dashboard_enabled & developer }.enable :read_group_security_dashboard

ee/changelogs/unreleased/217469-follow-up-from-fix-minutes-notification-when-unauthenticated.yml

+---
+title: Allow developers to see ci minutes notifications
+merge_request: 31937
+author:
+type: changed

ee/spec/models/ci/minutes/context_spec.rb

@@ -53,7 +53,7 @@ describe Ci::Minutes::Context do
     describe '#can_see_status' do
       context 'when eligible to see status' do
         before do
-          group.add_owner(user)
+          group.add_developer(user)
         end
 
         it 'can see status' do

ee/spec/models/ci/minutes/notification_spec.rb

@@ -128,7 +128,7 @@ describe Ci::Minutes::Notification do
     describe '#show?' do
       context 'when eligible to see notifications' do
         before do
-          group.add_owner(user)
+          group.add_developer(user)
         end
 
         context 'with a project that has runners enabled inside namespace' do

ee/spec/policies/group_policy_spec.rb

@@ -1051,7 +1051,7 @@ describe GroupPolicy do
     where(:role, :allowed) do
       :guest      | false
       :reporter   | false
-      :developer  | false
+      :developer  | true
       :maintainer | true
       :owner      | true
       :admin      | true