Kengne-yvana commented on PR #5319:
URL: https://github.com/apache/fineract/pull/5319#issuecomment-3746617637
Hello @IOhacker ,
I have identified the cause of the failure in E2E Shard 10
(LoanAccrualActivity.feature - UC7). The test is failing with a mismatch error:
expected: 6 but was: 7 transaction lines.
Analysis:
My fix for FINERACT-2086 resolves a ClassCastException that previously
occurred when the system attempted to read the account association type.
Because this association is now retrieved correctly, the loan business logic is
successfully triggering a replayed Accrual transaction on 11 April 2025 that
was previously being skipped or failing silently.
Proposed Solution:
Instead of modifying the core logic to hide this transaction, I propose
updating the E2E feature file (LoanAccrualActivity.feature) to include this 7th
transaction line. This would confirm that the accounting for Merchant Issued
Refunds and Repayment reversals is now functioning with full integrity as a
result of this bug fix.
Plan:
Keep the current fix for the ClassCastException and the new unit tests.
Update the UC7 data table in the E2E suite to reflect the correct
7-transaction state.
Please let me know if you approve of this approach to resolving the Shard 10
failure, and I will proceed with the update.
--
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]
