Refactor handling Usage Ping response

app/models/raw_usage_data.rb

@@ -4,11 +4,7 @@ class RawUsageData < ApplicationRecord
   validates :payload, presence: true
   validates :recorded_at, presence: true, uniqueness: true
 
-  def update_sent_at!
-    self.update_column(:sent_at, Time.current)
-  end
-
-  def update_version_usage_data_id!(usage_data_id)
-    self.update_column(:version_usage_data_id_value, usage_data_id)
+  def update_version_metadata!(usage_data_id:)
+    self.update_columns(sent_at: Time.current, version_usage_data_id_value: usage_data_id)
   end
 end

app/services/submit_usage_ping_service.rb

@@ -35,14 +35,9 @@ class SubmitUsagePingService
 
     raise SubmissionError.new("Unsuccessful response code: #{response.code}") unless response.success?
 
-    RawUsageData.transaction do
-      if raw_usage_data
-        raw_usage_data.update_sent_at!
-        raw_usage_data.update_version_usage_data_id!(response.dig('conv_index', 'usage_data_id'))
-      end
+    raw_usage_data.update_version_metadata!(usage_data_id: response.dig('conv_index', 'usage_data_id'))
 
-      store_metrics(response)
-    end
+    store_metrics(response)
   end
 
   private