This is an automated email from the ASF dual-hosted git repository.

adamsaghy pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/fineract.git


The following commit(s) were added to refs/heads/develop by this push:
     new 570f44ac93 FINERACT-2181: Swagger enhancement for 
PostSavingsProductsRequest
570f44ac93 is described below

commit 570f44ac93f91ec61b515b7396a8ff4744f38030
Author: Jose Alberto Hernandez <[email protected]>
AuthorDate: Mon Jun 30 20:55:29 2025 -0500

    FINERACT-2181: Swagger enhancement for PostSavingsProductsRequest
---
 .../fineract/test/factory/SavingsProductRequestFactory.java  |  6 +++---
 .../savings/api/SavingsProductsApiResourceSwagger.java       | 12 +++++++++++-
 .../savings/base/BaseSavingsIntegrationTest.java             |  7 ++++++-
 3 files changed, 20 insertions(+), 5 deletions(-)

diff --git 
a/fineract-e2e-tests-core/src/test/java/org/apache/fineract/test/factory/SavingsProductRequestFactory.java
 
b/fineract-e2e-tests-core/src/test/java/org/apache/fineract/test/factory/SavingsProductRequestFactory.java
index 54baa519bf..cfa3914386 100644
--- 
a/fineract-e2e-tests-core/src/test/java/org/apache/fineract/test/factory/SavingsProductRequestFactory.java
+++ 
b/fineract-e2e-tests-core/src/test/java/org/apache/fineract/test/factory/SavingsProductRequestFactory.java
@@ -18,8 +18,8 @@
  */
 package org.apache.fineract.test.factory;
 
-import java.util.HashSet;
-import java.util.Set;
+import java.util.ArrayList;
+import java.util.List;
 import org.apache.fineract.client.models.PostSavingsCharges;
 import org.apache.fineract.client.models.PostSavingsProductsRequest;
 
@@ -43,7 +43,7 @@ public final class SavingsProductRequestFactory {
     private SavingsProductRequestFactory() {}
 
     public static PostSavingsProductsRequest defaultSavingsProductRequest() {
-        Set<PostSavingsCharges> charges = new HashSet<>();
+        List<PostSavingsCharges> charges = new ArrayList<>();
 
         return new 
PostSavingsProductsRequest().name(DEFAULT_SAVINGS_PRODUCT_NAME)//
                 .shortName(DEFAULT_SAVINGS_PRODUCT_SHORT_NAME)//
diff --git 
a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/api/SavingsProductsApiResourceSwagger.java
 
b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/api/SavingsProductsApiResourceSwagger.java
index 724a781d06..526ccb254e 100644
--- 
a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/api/SavingsProductsApiResourceSwagger.java
+++ 
b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/api/SavingsProductsApiResourceSwagger.java
@@ -69,9 +69,19 @@ final class SavingsProductsApiResourceSwagger {
         public Integer interestCalculationDaysInYearType;
         @Schema(example = "1")
         public Integer accountingRule;
-        public Set<PostSavingsCharges> charges;
+        public List<PostSavingsCharges> charges;
         @Schema(example = "accountMappingForPayment")
         public String accountMappingForPayment;
+        @Schema(example = "false")
+        public Boolean withdrawalFeeForTransfers;
+        @Schema(example = "false")
+        public Boolean enforceMinRequiredBalance;
+        @Schema(example = "false")
+        public Boolean allowOverdraft;
+        @Schema(example = "false")
+        public Boolean withHoldTax;
+        @Schema(example = "false")
+        public Boolean isDormancyTrackingActive;
     }
 
     @Schema(description = "PostSavingsProductsResponse")
diff --git 
a/integration-tests/src/test/java/org/apache/fineract/integrationtests/savings/base/BaseSavingsIntegrationTest.java
 
b/integration-tests/src/test/java/org/apache/fineract/integrationtests/savings/base/BaseSavingsIntegrationTest.java
index 95873fb26a..9de4dbbea9 100644
--- 
a/integration-tests/src/test/java/org/apache/fineract/integrationtests/savings/base/BaseSavingsIntegrationTest.java
+++ 
b/integration-tests/src/test/java/org/apache/fineract/integrationtests/savings/base/BaseSavingsIntegrationTest.java
@@ -119,7 +119,12 @@ public class BaseSavingsIntegrationTest extends 
IntegrationTest {
                 .accountingRule(1) // none
                 
.interestCalculationDaysInYearType(DaysInYearType.DAYS_365).interestCompoundingPeriodType(InterestPeriodType.DAILY)
                 
.interestCalculationType(InterestCalculationType.AVERAGE_DAILY_BALANCE) //
-                .interestPostingPeriodType(InterestPeriodType.DAILY);//
+                .interestPostingPeriodType(InterestPeriodType.DAILY) //
+                .withdrawalFeeForTransfers(false) //
+                .enforceMinRequiredBalance(false) //
+                .allowOverdraft(false) //
+                .withHoldTax(false) //
+                .isDormancyTrackingActive(false); //
     }
 
     protected PostSavingsProductsResponse 
createProduct(PostSavingsProductsRequest productsRequest) {

Reply via email to