Record "sent_at" even if sending fails

* We still want to show this in the UI

Jira-Issue: MAV-6232

Author: jf-x-dev

app/lib/careplus/automated_report_sender.rb

@@ -55,6 +55,8 @@ def send_batch!(vaccination_records:, academic_year:, date:)
 
     attach_records!(careplus_report:, vaccination_records:)
 
+    record_send_attempt!(careplus_report:)
+
     response =
       Careplus::Client.send_csv(
         username: team.careplus_username,
@@ -108,8 +110,12 @@ def attach_records!(careplus_report:, vaccination_records:)
     )
   end
 
+  def record_send_attempt!(careplus_report:)
+    careplus_report.update!(sent_at: Time.current)
+  end
+
   def mark_as_sent!(careplus_report:)
-    careplus_report.update!(status: :sent, sent_at: Time.current)
+    careplus_report.update!(status: :sent)
   end
 
   def csv_filename(date:, timestamp:)

spec/lib/careplus/automated_report_sender_spec.rb

@@ -131,7 +131,8 @@
             )
 
       report = CareplusReport.last
-      expect(report).to have_attributes(status: "failed", sent_at: nil)
+      expect(report).to have_attributes(status: "failed")
+      expect(report.sent_at).to be_present
       expect(report.vaccination_records).to contain_exactly(record)
     end
   end
@@ -153,7 +154,8 @@
       expect { call }.to raise_error(Timeout::Error)
 
       report = CareplusReport.last
-      expect(report).to have_attributes(status: "failed", sent_at: nil)
+      expect(report).to have_attributes(status: "failed")
+      expect(report.sent_at).to be_present
       expect(report.vaccination_records).to contain_exactly(record)
     end
   end