ruanwenjun commented on PR #16616:
URL: 
https://github.com/apache/dolphinscheduler/pull/16616#issuecomment-2363175614

   > > It's needed to add integration test case to test the whole runtime case 
rather than ut.
   > 
   > > > It's needed to add integration test case to test the whole runtime 
case rather than ut.
   > > 
   > > 
   > > Okay, but can you tell me how to run these integration test cases? They 
all throw errors when I run them on my local IDEA. Is it necessary to depend on 
some specific local environments?
   > 
   > I found the real reason for the integration test error. Some fields in the 
YAML file need to be renamed, for example: processDefinitionCode needs to be 
replaced with workflowDefinitionCode, and processDefinitionVersion needs to be 
replaced with workflowDefinitionVersion, etc. @ruanwenjun
   > 
   > And there are some errors in the assertions as well.
   
   It's better to submit another PR to fix these IT.


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