adamsaghy commented on code in PR #4142: URL: https://github.com/apache/fineract/pull/4142#discussion_r1827599754
########## integration-tests/src/test/java/org/apache/fineract/integrationtests/AdvancedPaymentAllocationLoanRepaymentScheduleTest.java: ########## @@ -5473,6 +5475,45 @@ public void uc149() { }); } + // UC150: Validate interestRateFrequencyType to set as Whole Term at the Loan account level Review Comment: Would you enhance this test case or create a new one which validate the proper interest rate is set and used? I would say validating whether the proper annual interest rate is on the loan and the calculated interest is matching would be sufficient -- 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: commits-unsubscr...@fineract.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org