Fix PG deadlock in Api::PushesController (CHERRY-RAILS-6Q)

app/controllers/api/pushes_controller.rb

@@ -34,6 +34,8 @@ def create
             end
           )
         end
+
+      current_project.touch
     end
 
     render json: { status: :ok }, status: :created

app/models/report.rb

@@ -3,10 +3,7 @@
 class Report < ApplicationRecord
   belongs_to :metric
 
-  after_save do
-    metric.touch
-    metric.project.touch
-  end
+  after_save { metric.touch }
 
   has_many :occurrences, dependent: :destroy
 

test/controllers/api/pushes_controller_test.rb

@@ -168,6 +168,12 @@ class Api::PushesControllerTest < ActionDispatch::IntegrationTest
       assert_equal 24, metric_with_value.reports.last.value
     end
 
+    it 'bumps project.updated_at after a push' do
+      create(:project, name: 'cherrypush/cherry-app', user: user, updated_at: 1.week.ago)
+      post(api_push_path, params: { api_key: user.api_key, **payload }, as: :json)
+      assert_equal Date.current, Project.sole.reload.updated_at.to_date
+    end
+
     it 'adds up value_by_owner to existing value_by_owner by uuid' do
       post(api_push_path, params: { api_key: user.api_key, **payload }, as: :json)
       metric_with_owners = Metric.find_by(name: 'missing coverage')

test/models/metric_test.rb

@@ -11,7 +11,6 @@ class ProjectTest < ActiveSupport::TestCase
     it 'updates the updated_at field when a new report is created' do
       create(:report, metric: metric)
       assert_equal Time.current.to_date, metric.reload.updated_at.to_date
-      assert_equal Time.current.to_date, project.reload.updated_at.to_date
     end
   end
 
@@ -38,11 +37,6 @@ class ProjectTest < ActiveSupport::TestCase
       metric.destroy!
       assert_equal 1.week.ago.to_date, project.reload.updated_at.to_date
     end
-
-    it 'updates the project updated_at field when creating a report for an existing metric' do
-      create(:report, metric: metric)
-      assert_equal Date.current, project.reload.updated_at.to_date
-    end
   end
 
   describe 'when chart metrics are present' do