jdailey commented on code in PR #6352:
URL: https://github.com/apache/fineract/pull/6352#discussion_r3953352250
##########
fineract-progressive-loan/src/test/java/org/apache/fineract/portfolio/loanproduct/calc/ProgressiveEMICalculatorTest.java:
##########
@@ -5316,6 +5313,153 @@ public void test_principalGraceForProgressiveSchedule()
{
checkPeriod(interestSchedule, 5, 25.35, 0.15, 25.20, 0.0, false);
}
+ /**
+ * Control case for the bullet-loan boundary defect.
+ *
+ * With N=8 repayments and graceOnPrincipalPayment = N-2 = 6, the schedule
behaves correctly: installments 1..6 are
+ * interest-only and the principal is amortized across the final two
installments (7 and 8). This test passes on the
+ * current engine and is here to demonstrate that the defect exercised by
+ * {@link #test_principalGrace_nMinus1_shouldProduceBulletLoan()} is
specific to the grace = N-1 boundary.
+ */
+ @Test
+ public void test_principalGrace_nMinus2_deferralWorks() {
Review Comment:
Agreed. E2E test is needed.
--
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]
