mariiaKraievska commented on code in PR #6138:
URL: https://github.com/apache/fineract/pull/6138#discussion_r3629300878


##########
fineract-e2e-tests-runner/src/test/resources/features/WorkingCapitalDelinquencyReset.feature:
##########
@@ -485,3 +485,205 @@ Feature: Working Capital Delinquency Reset Action
     #    --- Close loan ---
     Then Admin closes the Working Capital loan with a full repayment on "11 
April 2026"
     And Working Capital loan status will be "CLOSED_OBLIGATIONS_MET"
+
+  @TestRailId:C89749
+  Scenario: Verify Working Capital delinquency reset / delinquency reset undo 
- UC9: delinquency reset with start new period

Review Comment:
   @peter-kovacs-dpc UC9–UC12 include "delinquency reset undo" in the scenario 
title, but none of them actually tests undo. Only UC1–UC3 cover undo today. 
Should we either rename these scenarios or add step that verifies undo after 
reset with startNewPeriod=true? 
   
   @somasorosdpc As I see undo currently does not restore truncated period 
dates or remove newly created periods. Could you please help me understand: is 
this the intended behaviour, is it out of scope for this task, or should it be 
fixed?



-- 
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]

Reply via email to