Merge branch '428072-reflect-star-count-update' into 'master'

Reflect updates in star count to elasticsearch

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



Merged-by: Madelein van Niekerk <mvanniekerk@gitlab.com>
Approved-by: Madelein van Niekerk <mvanniekerk@gitlab.com>
Reviewed-by: Terri Chu <tchu@gitlab.com>
Co-authored-by: Siddharth Dungarwal <sdungarwal@gitlab.com>

ee/app/models/ee/user.rb

@@ -257,6 +257,12 @@ def clear_group_with_ai_available_cache(ids)
       end
     end
 
+    override :toggle_star
+    def toggle_star(project)
+      super
+      project.maintain_elasticsearch_update if self.active? && project.maintaining_elasticsearch?
+    end
+
     def pending_billable_invitations
       if ::License.current.exclude_guests_from_active_count?
         pending_invitations.where('access_level > ?', ::Gitlab::Access::GUEST)

ee/spec/models/ee/user_spec.rb

@@ -3897,6 +3897,44 @@
     end
   end
 
+  describe 'starred_projects' do
+    let_it_be(:project) { create(:project) }
+    let_it_be(:user) { create(:user) }
+
+    context 'when project is not maintaining elasticsearch' do
+      before do
+        stub_ee_application_setting(elasticsearch_indexing: false)
+      end
+
+      it 'doest not call Elastic::ProcessBookkeepingService' do
+        expect(::Elastic::ProcessBookkeepingService).not_to receive(:track!)
+        user.toggle_star(project)
+      end
+    end
+
+    context 'when project is maintaining elasticsearch' do
+      before do
+        stub_ee_application_setting(elasticsearch_indexing: true)
+      end
+
+      it 'calls Elastic::ProcessBookkeepingService' do
+        expect(::Elastic::ProcessBookkeepingService).to receive(:track!).with(project).once
+        user.toggle_star(project)
+      end
+    end
+
+    context 'when user is inactive' do
+      before do
+        user.block
+      end
+
+      it 'doest not call Elastic::ProcessBookkeepingService' do
+        expect(::Elastic::ProcessBookkeepingService).not_to receive(:track!)
+        user.toggle_star(project)
+      end
+    end
+  end
+
   describe '#external?' do
     subject { user.external? }