marta-jankovics commented on code in PR #3471:
URL: https://github.com/apache/fineract/pull/3471#discussion_r1336861744
##########
integration-tests/src/test/java/org/apache/fineract/integrationtests/SavingsAccountTransactionTest.java:
##########
@@ -81,17 +101,86 @@ public void
verifySavingsTransactionSubmittedOnDateAndTransactionDate() throws J
LocalDate withdrawDate =
Utils.getDateAsLocalDate(withdrawDateString);
final Integer clientID =
ClientHelper.createClient(this.requestSpec, this.responseSpec, startDateString);
- Assertions.assertNotNull(clientID);
+ assertNotNull(clientID);
final Integer savingsId =
createSavingsAccountDailyPosting(clientID, startDateString);
- Assertions.assertNotNull(savingsId);
+ assertNotNull(savingsId);
performSavingsTransaction(savingsId, "100", depositDate, true);
performSavingsTransaction(savingsId, "50", withdrawDate, false);
} finally {
enableBusinessDate(requestSpec, responseSpec, false);
}
+ }
+
+ @Test
+ public void testConcurrentSavingsTransactions() {
+ final Integer clientID = ClientHelper.createClient(this.requestSpec,
this.responseSpec);
Review Comment:
The new test is to test the concurrent modification, return code 409 and how
it behaves in case of single requests, batch with and without enclosing
transactions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
