Samer-Melhem-FOO commented on PR #6165: URL: https://github.com/apache/fineract/pull/6165#issuecomment-5069239510
@adamsaghy I've pushed a fix addressing the test coverage request: - Added 3 new test cases to `ClientSearchTest.java` covering case-insensitive search by display name, external ID, and client identifier document key. Also looked into the 2 failing CI checks (`run-e2e-tests` Shard 6 and `run-integrations-tests-on-mysql` Shard 4) — both are unrelated to this change (Working Capital Loan charges/delinquency tests failing on timeouts/500s, nothing to do with client search). checkstyle and spotless are green. Please take another look when you get a chance. -- 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]
