AshharAhmadKhan commented on PR #6325: URL: https://github.com/apache/fineract/pull/6325#issuecomment-5429685734
> Now we have a working API, lets write an integration or E2E test for it ;) Hey Adam, I don't want to add tests to this PR. I'd rather create a separate PR for that. I confirmed there's no integration or E2E test coverage for /v1/email at any layer right now, so I'll write a single integration test covering the full lifecycle, CREATE, RETRIEVE, UPDATE, DELETE, against /v1/email. All four operations share the same setup, so testing them together in one test validates the real end to end flow rather than isolated fragments. I'll do this once #6330 is merged, since CREATE needs the validator fix in that PR to accept the real request params. 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
