thesmallstar commented on a change in pull request #1290: URL: https://github.com/apache/fineract/pull/1290#discussion_r491848617
########## File path: fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/ClientLoanIntegrationTest.java ########## @@ -65,7 +65,7 @@ * Client Loan Integration Test for checking Loan Application Repayments Schedule, loan charges, penalties, loan * repayments and verifying accounting transactions */ -@SuppressWarnings({ "rawtypes", "unchecked" }) +@SuppressWarnings({ "rawtypes", "unchecked", "AbbreviationAsWordInName" }) public class ClientLoanIntegrationTest { Review comment: Only this remains, I suppressed it here since there were 130-140 letter big function names and many of them, I thought it was really easy to read them in the way they are now. Also, I found those changes unnecessary since other errors in that file were solved already. test functions name should not matter much? What do you think? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org