dailai commented on PR #8313: URL: https://github.com/apache/seatunnel/pull/8313#issuecomment-2566060967
> > I think we should add some e2e case. > > Hi @dailai , I think the logic already be tested by https://github.com/apache/seatunnel/pull/8313/files#diff-835463340eff995995f88a3c8b8a25e873860a751b91b4f32322224e0346fab1R89. Should we continue add e2e test case? PS: I think we shoud move some test case from e2e to unit test. The e2e test case is too much. WDYT? UT covers logic and is not a substitute for e2e testing. The test scenario is different. -- 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]
