adamsaghy commented on PR #3881:
URL: https://github.com/apache/fineract/pull/3881#issuecomment-2105929613

   > @adamsaghy this commit basically replaces the wrong table column in the 
sql query with the right one. Please guide me on the best way to write this 
this test.
   
   First of all thank you for the fix. Like I said in my earlier comment you 
should write an integration test which call the mentioned APIs and they are not 
failing (response is HTTP 200). This way we can make sure in the future if the 
query or the data model got changed we will know about it.
   
   You may find plenty example in the "integration-test" directory.
   
   Please let me know if you are facing any trouble!


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