Show Days instead of hours in order

app/support/reservations/date_support.rb

@@ -91,6 +91,16 @@ def actual_duration_mins(actual_end_fallback: nil)
     end
   end
 
+  def actual_duration_days(actual_end_fallback: nil)
+    if @actual_duration_days
+      @actual_duration_days.to_i
+    elsif actual_start_at
+      TimeRange.new(actual_start_at, actual_end_at || actual_end_fallback).duration_days
+    else
+      0
+    end
+  end
+
   def actual_start_date
     date_field(:actual_start)
   end

app/views/facility_orders/_order_table.html.haml

@@ -29,9 +29,19 @@
         %td.order_date
           = od.ordered_at&.strftime("%m/%d/%Y")
 
-      %td.currency.timeinput= od.reservation.try(:duration_mins)
+      - daily_booking_product = od.product.daily_booking?
+      - if daily_booking_product
+        %td.currency
+          = od.reservation.try(:duration_days)
+      - else
+        %td.currency.timeinput= od.reservation.try(:duration_mins)
       - if od.time_data.present?
-        %td.currency{class: od.time_data.actual_duration_mins ? "timeinput" : ""}= od.time_data.actual_duration_mins || "???"
+        - if daily_booking_product
+          %td.currency
+            = od.time_data.actual_duration_days || "???"
+        - else
+          %td.currency{class: od.time_data.actual_duration_mins ? "timeinput" : ""}
+            = od.time_data.actual_duration_mins || "???"
       - elsif od.quantity_as_time?
         %td.currency.timeinput= od.quantity
       - else

spec/system/admin/order_show_spec.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe "Order show" do
+  let(:facility) { create(:setup_facility) }
+  let(:account) { create(:setup_account) }
+  let!(:instrument) { create(:setup_instrument, :always_available, :daily_booking, facility:) }
+  let(:reservation_order) { create(:setup_order, product: instrument, account:) }
+  let!(:reservation) do
+    create(
+      :purchased_reservation,
+      product: instrument,
+      order_detail: reservation_order.order_details.first,
+      reserve_start_at: 1.day.from_now,
+      reserve_end_at: 4.days.from_now
+    )
+  end
+  let(:facility_administrator) { create(:user, :facility_administrator, facility:) }
+
+  before do
+    login_as facility_administrator
+    visit facility_order_path(facility, reservation_order)
+  end
+
+  describe "reservation duration" do
+    it "is correctly displayed in days" do
+      # Reserved is the third column
+      expect(all("tbody tr").first.all("td")[2]).to have_text("3", exact: true)
+    end
+  end
+
+  describe "reservation actual" do
+    it "is correctly displayed in days" do
+      # Actual is the fourth column
+      expect(all("tbody tr").first.all("td")[3]).to have_text("0", exact: true)
+    end
+  end
+end