Browse Source

Fixes #4127 - Webhhok transfers "accounted_time" of an article as integer instead of float/double.

Rolf Schmidt 2 years ago
parent
commit
2c66cd79da

+ 1 - 1
app/jobs/trigger_webhook_job/record_payload/ticket/article.rb

@@ -10,7 +10,7 @@ class TriggerWebhookJob::RecordPayload::Ticket::Article < TriggerWebhookJob::Rec
   end
 
   def add_accounted_time(result)
-    result['accounted_time'] = record.ticket_time_accounting&.time_unit.to_i
+    result['accounted_time'] = record.ticket_time_accounting&.time_unit.to_f
     result
   end
 

+ 9 - 0
spec/jobs/trigger_webhook_job/record_payload/ticket/article_spec.rb

@@ -24,6 +24,15 @@ RSpec.describe TriggerWebhookJob::RecordPayload::Ticket::Article do
       end
     end
 
+    describe 'Webhhok transfers "accounted_time" of an article as integer instead of float/double #4127' do
+      let(:entry) { create(:ticket_time_accounting, ticket_id: record.ticket.id, ticket_article_id: record.id, time_unit: 3.33) }
+
+      it "stores value as 'accounted_time' key" do
+        entry
+        expect(generate['accounted_time']).to eq(3.33)
+      end
+    end
+
     context 'when Article has stored attachments' do
 
       before do