qiuyanjun888 commented on issue #16479: URL: https://github.com/apache/dolphinscheduler/issues/16479#issuecomment-4676244722
I added one more master workflow integration test case for condition tasks with mixed predecessor states and opened a PR: https://github.com/apache/dolphinscheduler/pull/18337 The PR adds a YAML fixture where two predecessor LogicFakeTask nodes finish with SUCCESS/FAILURE, verifies the condition task follows the failed branch, and confirms the workflow finishes successfully. Verified locally with: - `./mvnw -B -ntp -pl dolphinscheduler-master -am clean test -Dtest=WorkflowStartConditionTestCase#testStartWorkflow_with_oneConditionTaskWithTwoFakePredecessors_mixedStates -Dsurefire.failIfNoSpecifiedTests=false` - `./mvnw -B -ntp -pl dolphinscheduler-master spotless:apply -DskipTests` -- 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]
