avikganguly01 commented on pull request #1956: URL: https://github.com/apache/fineract/pull/1956#issuecomment-961669013
@ptuomola (cc @awasum @fynmanoj @vidakovic ): Please find 2 new additions to the list of flaky tests :- testCenterReschedulingLoansWithInterestRecalculationEnabled testCenterReschedulingMultiTrancheLoansWithInterestRecalculationEnabled Merging the PR as it's not the committer's responsibility to get a proposed PR to pass the build when there are 25+ flaky tests. Committer has checked with gradle test command that all non flaky tests are passing. The data points you are referring to in the other thread to deny the existence of flaky tests is not valid as they are manipulated. Perhaps a way of not stepping on each other's feet is to separate the services as you were discussing with Aleksander. I think Batch API module is a good place to start as it's already decoupled at the service and domain layer. -- 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]
