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

Reply via email to