qiuyanjun888 opened a new pull request, #18337:
URL: https://github.com/apache/dolphinscheduler/pull/18337
## Was this PR generated or assisted by AI?
Yes. Assisted by AI for code navigation, test-case construction, local
verification, and PR preparation.
## Purpose of the pull request
Part of #16479.
Add one more master workflow integration test for a condition task whose two
predecessor tasks finish with mixed states. The case verifies the condition
task evaluates the combined predecessor states and follows the failed branch
while the workflow still finishes successfully.
## Brief change log
- Added a new condition workflow YAML fixture with two root `LogicFakeTask`
predecessors:
- `A` succeeds
- `B` fails
- `C` condition requires both predecessors to be `SUCCESS`
- failed branch runs `E`; success branch `D` is not triggered
- Added a `WorkflowStartConditionTestCase` test that asserts the workflow
succeeds and only `A`, `B`, `C`, and `E` task instances are created with the
expected states.
## Verify this pull request
- `./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]
