Skip to content

FINERACT-2165: Migrate event, provisioning, and standing instruction helpers to fineract-client-feign #5946

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
elnafateh wants to merge 2 commits into
base: develop
Choose a base branch
from
+493 −1,312
Open

FINERACT-2165: Migrate event, provisioning, and standing instruction helpers to fineract-client-feign #5946

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
41e573a
FINERACT-2165: Fix ProvisioningCategoryDeleteIntegrationTest to use f…
elnafateh Jun 5, 2026
a8b6ea9
FINERACT-2165: revert to enum and fix compilation errors w style issue
elnafateh Jul 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion ...ract-client-feign/src/main/java/org/apache/fineract/client/feign/ObjectMapperFactory.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ public static ObjectMapper createObjectMapper() {

// Disable FAIL_ON_EMPTY_BEANS for empty responses
mapper.disable(SerializationFeature.FAIL_ON_EMPTY_BEANS);

return mapper;
}

Expand Down
6 changes: 3 additions & 3 deletions ...apache/fineract/organisation/provisioning/api/ProvisioningCriteriaApiResourceSwagger.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ private PostProvisioningCriteriaRequest() {
@Schema(example = "High Risk Products Criteria")
public String criteriaName;
public Collection<LoanProductData> loanProducts;
public Collection<ProvisioningCriteriaDefinitionData> provisioningcriteria;
public Collection<ProvisioningCriteriaDefinitionData> definitions;
Comment thread
adamsaghy marked this conversation as resolved.
}

@Schema(description = "PostProvisioningCriteriaResponse")
Expand Down Expand Up @@ -85,7 +85,7 @@ private GetProvisioningCriteriaCriteriaIdResponse() {
@Schema(example = "mifos")
public String createdBy;
public Collection<LoanProductData> loanProducts;
public Collection<ProvisioningCriteriaDefinitionData> provisioningcriteria;
public Collection<ProvisioningCriteriaDefinitionData> definitions;
}

@Schema(description = "PutProvisioningCriteriaRequest")
Expand All @@ -98,7 +98,7 @@ private PutProvisioningCriteriaRequest() {
@Schema(example = "High Risk Products Criteria")
public String criteriaName;
public Collection<LoanProductData> loanProducts;
public Collection<ProvisioningCriteriaDefinitionData> provisioningcriteria;
public Collection<ProvisioningCriteriaDefinitionData> definitions;
}

@Schema(description = "PutProvisioningCriteriaResponse")
Expand Down
70 changes: 35 additions & 35 deletions ...java/org/apache/fineract/portfolio/account/api/StandingInstructionApiResourceSwagger.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ private GetFromAccountTypeResponseStandingInstructionSwagger() {}
@Schema(example = "accountType.savings")
public String code;
@Schema(example = "Savings Account")
public String description;
public String value;
Comment thread
adamsaghy marked this conversation as resolved.
}

static final class GetFromOfficeOptionsResponseStandingInstructionSwagger {
Expand Down Expand Up @@ -101,7 +101,7 @@ private GetFromAccountTypeOptionsResponseStandingInstructionSwagger() {}
@Schema(example = "accountType.savings")
public String code;
@Schema(example = "Savings Account")
public String description;
public String value;
}

static final class GetToOfficeOptionsResponseStandingInstructionSwagger {
Expand All @@ -125,7 +125,7 @@ private GetToAccountTypeOptionsResponseStandingInstructionSwagger() {}
@Schema(example = "accountType.loan")
public String code;
@Schema(example = "Loan Account")
public String description;
public String value;
}

static final class GetTransferTypeOptionsResponseStandingInstructionSwagger {
Expand All @@ -137,7 +137,7 @@ private GetTransferTypeOptionsResponseStandingInstructionSwagger() {}
@Schema(example = "accountTransferType.account.transfer")
public String code;
@Schema(example = "Account Transfer")
public String description;
public String value;
}

static final class GetStatusOptionsResponseStandingInstructionSwagger {
Expand All @@ -149,7 +149,7 @@ private GetStatusOptionsResponseStandingInstructionSwagger() {}
@Schema(example = "standingInstructionStatus.active")
public String code;
@Schema(example = "Active")
public String description;
public String value;
}

static final class GetInstructionTypeOptionsResponseStandingInstructionSwagger {
Expand All @@ -161,7 +161,7 @@ private GetInstructionTypeOptionsResponseStandingInstructionSwagger() {}
@Schema(example = "standingInstructionType.fixed")
public String code;
@Schema(example = "Fixed")
public String description;
public String value;
}

static final class GetPriorityOptionsResponseStandingInstructionSwagger {
Expand All @@ -173,7 +173,7 @@ private GetPriorityOptionsResponseStandingInstructionSwagger() {}
@Schema(example = "standingInstructionPriority.urgent")
public String code;
@Schema(example = "Urgent Priority")
public String description;
public String value;
}

static final class GetRecurrenceTypeOptionsResponseStandingInstructionSwagger {
Expand All @@ -185,7 +185,7 @@ private GetRecurrenceTypeOptionsResponseStandingInstructionSwagger() {}
@Schema(example = "accountTransferRecurrenceType.periodic")
public String code;
@Schema(example = "Periodic Recurrence")
public String description;
public String value;
}

static final class GetRecurrenceFrequencyOptionsResponseStandingInstructionSwagger {
Expand All @@ -197,7 +197,7 @@ private GetRecurrenceFrequencyOptionsResponseStandingInstructionSwagger() {}
@Schema(example = "frequencyperiodFrequencyType.days")
public String code;
@Schema(example = "Days")
public String description;
public String value;
}

public GetFromOfficeResponseStandingInstructionSwagger fromOffice;
Expand Down Expand Up @@ -266,7 +266,7 @@ private GetFromAccountTypeStandingInstructionSwagger() {}
@Schema(example = "accountType.savings")
public String code;
@Schema(example = "Savings Account")
public String description;
public String value;
}

static final class GetFromAccountStandingInstructionSwagger {
Expand Down Expand Up @@ -316,7 +316,7 @@ private GetToAccountTypeStandingInstructionSwagger() {}
@Schema(example = "accountType.savings")
public String code;
@Schema(example = "Savings Account")
public String description;
public String value;
}

static final class GetToAccountStandingInstructionSwagger {
Expand All @@ -333,52 +333,52 @@ private GetToAccountStandingInstructionSwagger() {}
public String productName;
}

static final class GetTransferTypeStandingInstructionSwagger {
static final class GetPriorityStandingInstructionSwagger {

private GetTransferTypeStandingInstructionSwagger() {}
private GetPriorityStandingInstructionSwagger() {}

@Schema(example = "1")
@Schema(example = "3")
public Integer id;
@Schema(example = "accountTransferType.account.transfer")
@Schema(example = "standingInstructionPriority.medium")
public String code;
@Schema(example = "Account Transfer")
public String description;
@Schema(example = "Medium Priority")
public String value;
}

static final class GetPriorityStandingInstructionSwagger {
static final class GetStatusStandingInstructionSwagger {

private GetPriorityStandingInstructionSwagger() {}
private GetStatusStandingInstructionSwagger() {}

@Schema(example = "3")
@Schema(example = "1")
public Integer id;
@Schema(example = "standingInstructionPriority.medium")
@Schema(example = "standingInstructionStatus.deleted")
public String code;
@Schema(example = "Medium Priority")
public String description;
@Schema(example = "Deleted")
public String value;
}

static final class GetInstructionTypeStandingInstructionSwagger {
static final class GetTransferTypeStandingInstructionSwagger {

private GetInstructionTypeStandingInstructionSwagger() {}
private GetTransferTypeStandingInstructionSwagger() {}

@Schema(example = "1")
public Integer id;
@Schema(example = "standingInstructionType.fixed")
@Schema(example = "accountTransferType.account.transfer")
public String code;
@Schema(example = "Fixed")
public String description;
@Schema(example = "Account Transfer")
public String value;
}

static final class GetStatusStandingInstructionSwagger {
static final class GetInstructionTypeStandingInstructionSwagger {

private GetStatusStandingInstructionSwagger() {}
private GetInstructionTypeStandingInstructionSwagger() {}

@Schema(example = "1")
public Integer id;
@Schema(example = "standingInstructionStatus.deleted")
@Schema(example = "standingInstructionType.fixed")
public String code;
@Schema(example = "Deleted")
public String description;
@Schema(example = "Fixed")
public String value;
}

static final class GetRecurrenceTypeStandingInstructionSwagger {
Expand All @@ -390,7 +390,7 @@ private GetRecurrenceTypeStandingInstructionSwagger() {}
@Schema(example = "accountTransferRecurrenceType.periodic")
public String code;
@Schema(example = "Periodic Recurrence")
public String description;
public String value;
}

static final class GetRecurrenceFrequencyStandingInstructionSwagger {
Expand All @@ -402,7 +402,7 @@ private GetRecurrenceFrequencyStandingInstructionSwagger() {}
@Schema(example = "frequencyperiodFrequencyType.months")
public String code;
@Schema(example = "Months")
public String description;
public String value;
}

@Schema(example = "1")
Expand Down
18 changes: 5 additions & 13 deletions ...rc/test/java/org/apache/fineract/integrationtests/CustomSnapshotEventIntegrationTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,13 +18,12 @@
*/
package org.apache.fineract.integrationtests;

import com.google.gson.Gson;
import java.math.BigDecimal;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import lombok.extern.slf4j.Slf4j;
import org.apache.fineract.client.models.BusinessDateUpdateRequest;
import org.apache.fineract.client.models.ExternalEventConfigurationUpdateRequest;
import org.apache.fineract.client.models.PostLoanProductsRequest;
import org.apache.fineract.client.models.PostLoanProductsResponse;
import org.apache.fineract.infrastructure.event.external.data.ExternalEventResponse;
Expand Down Expand Up @@ -307,26 +306,19 @@ private void enableCOBBusinessStep(String... steps) {

}

public static String getExternalEventConfigurationsForUpdateJSON() {
Map<String, Map<String, Boolean>> configurationsForUpdate = new HashMap<>();
Map<String, Boolean> configurations = new HashMap<>();
configurations.put("CentersCreateBusinessEvent", true);
configurations.put("ClientActivateBusinessEvent", true);
configurationsForUpdate.put("externalEventConfigurations", configurations);
return new Gson().toJson(configurationsForUpdate);
}

private void enableLoanAccountCustomSnapshotBusinessEvent() {
final Map<String, Boolean> updatedConfigurations = ExternalEventConfigurationHelper.updateExternalEventConfigurations(requestSpec,
responseSpec, "{\"externalEventConfigurations\":{\"LoanAccountCustomSnapshotBusinessEvent\":true}}\n");
responseSpec, new ExternalEventConfigurationUpdateRequest()
.externalEventConfigurations(Map.of("LoanAccountCustomSnapshotBusinessEvent", true)));
Assertions.assertEquals(updatedConfigurations.size(), 1);
Assertions.assertTrue(updatedConfigurations.containsKey("LoanAccountCustomSnapshotBusinessEvent"));
Assertions.assertTrue(updatedConfigurations.get("LoanAccountCustomSnapshotBusinessEvent"));
}

private void disableLoanAccountCustomSnapshotBusinessEvent() {
final Map<String, Boolean> updatedConfigurations = ExternalEventConfigurationHelper.updateExternalEventConfigurations(requestSpec,
responseSpec, "{\"externalEventConfigurations\":{\"LoanAccountCustomSnapshotBusinessEvent\":false}}\n");
responseSpec, new ExternalEventConfigurationUpdateRequest()
.externalEventConfigurations(Map.of("LoanAccountCustomSnapshotBusinessEvent", false)));
Assertions.assertEquals(updatedConfigurations.size(), 1);
Assertions.assertTrue(updatedConfigurations.containsKey("LoanAccountCustomSnapshotBusinessEvent"));
Assertions.assertFalse(updatedConfigurations.get("LoanAccountCustomSnapshotBusinessEvent"));
Expand Down
Loading
Loading