Skip to content

Commit ca5a1fc

Browse files
i
FINERACT-2571: Migrate charge creation helper to feign client Migrate charge creation helper to feign client FINERACT-2571: Migrate charge creation helper to feign client
1 parent 89eb49c commit ca5a1fc

2 files changed

Lines changed: 11 additions & 6 deletions

File tree

integration-tests/src/test/java/org/apache/fineract/integrationtests/common/NotificationHelper.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
import java.time.Duration;
2727
import lombok.extern.slf4j.Slf4j;
2828
import org.apache.fineract.client.models.GetNotificationsResponse;
29+
import org.apache.fineract.client.util.Calls;
2930
import org.apache.fineract.client.util.JSON;
3031

3132
@Slf4j
@@ -42,10 +43,10 @@ private NotificationHelper() {}
4243
@Deprecated(forRemoval = true)
4344
public static GetNotificationsResponse getNotifications(final RequestSpecification requestSpec,
4445
final ResponseSpecification responseSpec) {
45-
final String GET_NOTIFICATIONS_URL = NOTIFICATION_API_URL;
46+
4647
log.info("-----------------------------GET NOTIFICATIONS-----------------------------------");
47-
String response = Utils.performServerGet(requestSpec, responseSpec, GET_NOTIFICATIONS_URL);
48-
return GSON.fromJson(response, GetNotificationsResponse.class);
48+
49+
return Calls.ok(() -> FineractClientHelper.getFineractClient().notifications().getAllNotifications(null, 1, 0, null, null));
4950
}
5051

5152
// TODO: Rewrite to use fineract-client instead!

integration-tests/src/test/java/org/apache/fineract/integrationtests/common/charges/ChargesHelper.java

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -599,7 +599,9 @@ public static HashMap<String, Object> populateDefaultsShareRedeemFlatCharge() {
599599
@Deprecated(forRemoval = true)
600600
public static Integer createCharges(final RequestSpecification requestSpec, final ResponseSpecification responseSpec,
601601
final String request) {
602-
return Utils.performServerPost(requestSpec, responseSpec, CREATE_CHARGES_URL, request, "resourceId");
602+
ChargeRequest chargeRequest = GSON.fromJson(request, ChargeRequest.class);
603+
PostChargesResponse response = Calls.ok(FineractClientHelper.getFineractClient().charges.createCharge(chargeRequest));
604+
return response.getResourceId().intValue();
603605
}
604606

605607
// TODO: Rewrite to use fineract-client instead!
@@ -608,8 +610,10 @@ public static Integer createCharges(final RequestSpecification requestSpec, fina
608610
@Deprecated(forRemoval = true)
609611
public static PostChargesResponse createLoanCharge(final RequestSpecification requestSpec, final ResponseSpecification responseSpec,
610612
final String payload) {
611-
final String response = Utils.performServerPost(requestSpec, responseSpec, CREATE_CHARGES_URL, payload, null);
612-
return GSON.fromJson(response, PostChargesResponse.class);
613+
614+
ChargeRequest chargeRequest = GSON.fromJson(payload, ChargeRequest.class);
615+
616+
return Calls.ok(FineractClientHelper.getFineractClient().charges().createCharge(chargeRequest));
613617
}
614618

615619
// TODO: Rewrite to use fineract-client instead!

0 commit comments

Comments
 (0)