GangLiCN commented on PR #6435: URL: https://github.com/apache/seatunnel/pull/6435#issuecomment-1982759687
> > > > > Please add test in e2e. > > > > > > > > > > > > Hi, please tell me the full detailed steps for "add test in e2e" since I'm fresh on this. > > > > Bases on my understanding, it seems that it's unnecessary to add new test cases because current test case is enoguh to cover the bug fix. As I mentioned before, we can verify the fix like below: " The fix can be tested via general windows specific flink/spark starter script. Flink: start-seatunnel-flink-13(15)-connector-v2.cmd Spark: start-seatunnel-spark-2(3)-connector-v2.cmd > > > > For conf file, just using the example conf file from "seatunnel_home/config" dir. v2.batch.config.template, v2.streaming.template " > > > > > > > > > Oh I see. How about add some UT to verify whether the output command is right. Our UT will exeucted both on windows and linux. > > > > > > What does UT mean ? user testcase ? > > unit test. you can refer > > https://github.com/apache/seatunnel/blob/ec533ecd9a7d94d1c3e6d6ad7cc0440315d897e5/seatunnel-core/seatunnel-flink-starter/seatunnel-flink-starter-common/src/test/java/org/apache/seatunnel/core/starter/flink/TestFlinkParameter.java#L37 > > so on I checked the source code of "TestFlinkParameter.java" and feel a bit confused. It imports two new class: import org.apache.seatunnel.core.starter.flink.args.FlinkCommandArgs; import org.apache.seatunnel.core.starter.flink.utils.EnvironmentUtil; I can't find these classes in dev branch of base repo(apache/seatunnel)'s code line, Has these two java classes added recently ? About this PR, Is it also necessary to introduce some new java classes to finish a simple unit test case ? Do we have a guideline or standard rule to submit unit test case ? -- 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]
