Missing or unresolvable values for renewal count are treated as zero

src/folio_migration_tools/transaction_migration/legacy_loan.py

@@ -91,7 +91,8 @@ def __init__(
         self.out_date: datetime = temp_date_out
         self.correct_for_1_day_loans()
         self.make_utc()
-        self.renewal_count = int(legacy_loan_dict["renewal_count"])
+        self.renewal_count = 0
+        self.set_renewal_count(legacy_loan_dict)
         self.next_item_status = legacy_loan_dict.get("next_item_status", "").strip()
         if self.next_item_status not in legal_statuses:
             self.errors.append(("Not an allowed status", self.next_item_status))
@@ -101,6 +102,17 @@ def __init__(
             else fallback_service_point_id
         )
 
+    def set_renewal_count(self, loan: dict):
+        if "renewal_count" in loan:
+            renewal_count = loan["renewal_count"]
+            try:
+                self.renewal_count = int(renewal_count)
+            except ValueError:
+                self.report(
+                    f"Unresolvable {renewal_count=} was replaced with 0.")
+        else:
+            self.report(f"Missing renewal count was replaced with 0.")
+
     def correct_for_1_day_loans(self):
         try:
             if self.due_date.date() <= self.out_date.date():

tests/test_legacy_loan.py

@@ -161,3 +161,34 @@ def test_init_tz_5():  # Test dates with(out) DST
         "Provided out_date is not UTC, setting tzinfo to tenant timezone (America/Chicago)"
         in migration_report.report["Details"]
     )
+
+
+def test_init_renewal_count_is_empty():
+    loan_dict = {
+        "item_barcode": "the barcode with trailing space ",
+        "patron_barcode": " the barcode with leading space",
+        "due_date": "20220113 16:00",
+        "out_date": "20220113 14:00",
+        "next_item_status": "Checked out",
+    }
+    tenant_timezone = ZoneInfo("UTC")
+    migration_report = MigrationReport()
+    legacy_loan = LegacyLoan(
+        loan_dict, "", migration_report, tenant_timezone)
+    assert legacy_loan.renewal_count == 0
+
+
+def test_init_renewal_count_is_unresolvable():
+    loan_dict = {
+        "item_barcode": "the barcode with trailing space ",
+        "patron_barcode": " the barcode with leading space",
+        "due_date": "20220113 16:00",
+        "out_date": "20220113 14:00",
+        "renewal_count": "abc",
+        "next_item_status": "Checked out",
+    }
+    tenant_timezone = ZoneInfo("UTC")
+    migration_report = MigrationReport()
+    legacy_loan = LegacyLoan(
+        loan_dict, "", migration_report, tenant_timezone)
+    assert legacy_loan.renewal_count == 0