Circstore 506 (#464)

* CIRCSTORE-506 Added isDcb flag in loan.json * CIRCSTORE-506 added testcases and changes in LoanRequestBuilder * CIRCSTORE-506 changed the version in moduleDescriptor and loan-storage.raml to 7.3
folio-org · Jun 6, 2024 · 5137c20 · 5137c20
1 parent 995a4cf
commit 5137c20
Show file tree

Hide file tree

Showing 5 changed files with 108 additions and 4 deletions.
diff --git a/descriptors/ModuleDescriptor-template.json b/descriptors/ModuleDescriptor-template.json
@@ -45,7 +45,7 @@
     },
     {
       "id": "loan-storage",
-      "version": "7.1",
+      "version": "7.3",
       "handlers": [
         {
           "methods": ["GET"],

diff --git a/ramls/loan-storage.raml b/ramls/loan-storage.raml
@@ -1,6 +1,6 @@
 #%RAML 1.0
 title: Loan Storage
-version: v7.2
+version: v7.3
 protocols: [ HTTP, HTTPS ]
 baseUri: http://localhost:9130
 

diff --git a/ramls/loan.json b/ramls/loan.json
@@ -93,6 +93,10 @@
       "description": "Indicates whether or not this loan had its due date modified by a recall on the loaned item",
       "type": "boolean"
     },
+    "isDcb": {
+      "description": "Indicates whether or not this loan is associated for DCB use case",
+      "type": "boolean"
+    },
     "declaredLostDate" : {
       "description": "Date and time the item was declared lost during this loan",
       "type": "string",

diff --git a/src/test/java/org/folio/rest/api/LoansApiTest.java b/src/test/java/org/folio/rest/api/LoansApiTest.java
@@ -178,6 +178,98 @@ public void canCreateALoan()
   }
 
   @Test
+  public void canCreateALoanForDcb()
+    throws MalformedURLException,
+    InterruptedException,
+    ExecutionException,
+    TimeoutException {
+
+    UUID id = UUID.randomUUID();
+    UUID itemId = UUID.randomUUID();
+    UUID itemLocationAtCheckOut = UUID.randomUUID();
+    UUID userId = UUID.randomUUID();
+    UUID proxyUserId = UUID.randomUUID();
+    UUID loanPolicyId = UUID.randomUUID();
+    DateTime expectedLostDate = DateTime.now();
+    UUID overdueFinePolicyId = UUID.randomUUID();
+    UUID lostItemPolicyId = UUID.randomUUID();
+    final DateTime claimedReturnedDate = DateTime.now(DateTimeZone.UTC);
+    final DateTime agedToLostDate = DateTime.now(DateTimeZone.UTC).minusDays(10);
+    DateTime dateLostItemShouldBeBilled = new DateTime(2017, 9, 27, 10, 23, 43, DateTimeZone.UTC);
+
+    JsonObject loanRequest = new LoanRequestBuilder()
+      .withId(id)
+      .withItemId(itemId)
+      .withUserId(userId)
+      .withProxyUserId(proxyUserId)
+      .withLoanDate(new DateTime(2017, 6, 27, 10, 23, 43, DateTimeZone.UTC))
+      .open()
+      .withAction("checkedout")
+      .withItemStatus("Checked out")
+      .withDueDate(new DateTime(2017, 7, 27, 10, 23, 43, DateTimeZone.UTC))
+      .withItemEffectiveLocationIdAtCheckOut(itemLocationAtCheckOut)
+      .withLoanPolicyId(loanPolicyId)
+      .withDeclaredLostDate(expectedLostDate)
+      .withOverdueFinePolicyId(overdueFinePolicyId)
+      .withLostItemPolicyId(lostItemPolicyId)
+      .withClaimedReturnedDate(claimedReturnedDate)
+      .withAgedToLostDelayedBilling(false, dateLostItemShouldBeBilled, agedToLostDate)
+      .withIsDcb(true)
+      .create();
+
+    JsonObject loan = loansClient.create(loanRequest).getJson();
+
+    assertThat("dcb property should be true ",
+      loan.getString("isDcb"), is("true"));
+  }
+
+  @Test
+  public void canCreateALoanForNonDcb()
+    throws MalformedURLException,
+    InterruptedException,
+    ExecutionException,
+    TimeoutException {
+
+    UUID id = UUID.randomUUID();
+    UUID itemId = UUID.randomUUID();
+    UUID itemLocationAtCheckOut = UUID.randomUUID();
+    UUID userId = UUID.randomUUID();
+    UUID proxyUserId = UUID.randomUUID();
+    UUID loanPolicyId = UUID.randomUUID();
+    DateTime expectedLostDate = DateTime.now();
+    UUID overdueFinePolicyId = UUID.randomUUID();
+    UUID lostItemPolicyId = UUID.randomUUID();
+    final DateTime claimedReturnedDate = DateTime.now(DateTimeZone.UTC);
+    final DateTime agedToLostDate = DateTime.now(DateTimeZone.UTC).minusDays(10);
+    DateTime dateLostItemShouldBeBilled = new DateTime(2017, 9, 27, 10, 23, 43, DateTimeZone.UTC);
+
+    JsonObject loanRequest = new LoanRequestBuilder()
+      .withId(id)
+      .withItemId(itemId)
+      .withUserId(userId)
+      .withProxyUserId(proxyUserId)
+      .withLoanDate(new DateTime(2017, 6, 27, 10, 23, 43, DateTimeZone.UTC))
+      .open()
+      .withAction("checkedout")
+      .withItemStatus("Checked out")
+      .withDueDate(new DateTime(2017, 7, 27, 10, 23, 43, DateTimeZone.UTC))
+      .withItemEffectiveLocationIdAtCheckOut(itemLocationAtCheckOut)
+      .withLoanPolicyId(loanPolicyId)
+      .withDeclaredLostDate(expectedLostDate)
+      .withOverdueFinePolicyId(overdueFinePolicyId)
+      .withLostItemPolicyId(lostItemPolicyId)
+      .withClaimedReturnedDate(claimedReturnedDate)
+      .withAgedToLostDelayedBilling(false, dateLostItemShouldBeBilled, agedToLostDate)
+      .withIsDcb(false)
+      .create();
+
+    JsonObject loan = loansClient.create(loanRequest).getJson();
+
+    assertThat("dcb property should be false",
+      loan.getString("isDcb"), is("false"));
+  }
+
+    @Test
   public void canCreateALoanWithDueDateChangedByRecallSet()
     throws MalformedURLException,
     InterruptedException,
@@ -1483,7 +1575,7 @@ public void canSearchByLoanAgedToLostBillingDate() throws Exception {
       .getRecords().stream()
       .map(json -> json.getString("id"))
       .collect(Collectors.toList());
-    
+
     assertThat(filteredLoans, hasSize(1));
     assertThat(filteredLoans, hasItem(loanToBillTomorrow.getId()));
   }

diff --git a/src/test/java/org/folio/rest/support/builders/LoanRequestBuilder.java b/src/test/java/org/folio/rest/support/builders/LoanRequestBuilder.java
@@ -36,6 +36,7 @@ public class LoanRequestBuilder implements Builder {
   private final UUID lostItemPolicyId;
   private final DateTime claimedReturnedDate;
   private final JsonObject agedToLostDelayedBilling;
+  private final Boolean isDcb;
 
   public LoanRequestBuilder() {
     this(UUID.randomUUID(),
@@ -58,6 +59,7 @@ public LoanRequestBuilder() {
       null,
       null,
       null,
+      null,
       null);
   }
 
@@ -154,7 +156,9 @@ public static LoanRequestBuilder from(JsonObject example) {
       overdueFinePolicyId,
       lostItemPolicyId,
       claimedReturnedDate,
-      agedToLostDelayedBilling);
+      agedToLostDelayedBilling,
+      example.getBoolean("isDcb")
+      );
   }
 
   @Override
@@ -242,6 +246,10 @@ public JsonObject create() {
       request.put("agedToLostDelayedBilling", agedToLostDelayedBilling);
     }
 
+    if (isDcb != null) {
+      request.put("isDcb", isDcb);
+    }
+
     return request;
   }