Merge pull request #1133 from sul-dlss/hb-loans

Add a honeybadger notice if we can't find a loan schedule for the policy

app/models/folio/checkout.rb

@@ -181,14 +181,19 @@ def effective_loan_policy
     # NOTE: We need to fetch the latest loan policy to evaluate renewability. The loan policy returned
     # with the loan is the policy at the time of checkout and could have changed in ways that impact
     # eligibility for renewal.
-    def effective_loan_policy_id
+    def effective_loan_policy_id # rubocop:disable Metrics/AbcSize,Metrics/MethodLength
       cache_key = ['effective_loan_policy_id', item_type_id, loan_type_id, patron_type_id, location_id].join(':')
       Rails.cache.fetch(cache_key, expires_in: 1.day) do
         response = FolioClient.new.find_effective_loan_policy(item_type_id:,
                                                               loan_type_id:,
                                                               patron_type_id:,
                                                               location_id:)
 
+        unless response['loanPolicyId']
+          Honeybadger.notify('Unable to find effective loan policy for checkout',
+                             context: { key:, cache_key:, response: })
+        end
+
         response['loanPolicyId']
       end
     end

app/models/folio/loan_policy.rb

@@ -11,6 +11,10 @@ def initialize(loan_policy:, due_date:, renewal_count:, hold_queue_length:)
       @hold_queue_length = hold_queue_length
     end
 
+    def name
+      loan_policy['name']
+    end
+
     def loan_policy_interval
       loan_policy.dig('loansPolicy', 'period', 'intervalId')
     end
@@ -63,6 +67,8 @@ def due_date_from_schedule
         Time.zone.now.between?(policy['from'], policy['to'])
       end
 
+      Honeybadger.notify('No schedule found for loan policy', context: { loan_policy_name: name }) unless schedule
+
       schedule['due'] if schedule
     end