Repository navigation
[DSIP-64] Add more workflow Integration test cases in master. #16479
Description
Search before asking
- I had searched in the DSIP and found no similar DSIP.
Motivation
After #16423, we can easily add integration case for master.
Some cases are already added at #16423, but the workflow might have many structure, and some task might influence the workflow runtime graph, so we need to add more cases.
- Switch task case [Improvement][Master] Adding integration testcases for switch task case in Master #16725
- Sub workflow task case
- Condition task case
- More failover case
- More workflow operation case
- Master event bus benchmark
Design Detail
No response
Compatibility, Deprecation, and Migration Plan
No response
Test Plan
No response
Code of Conduct
- I agree to follow this project's Code of Conduct
Activity
[-][DSIP-] Add more workflow Integration case in master.[/-][+][DSIP-64] Add more workflow Integration case in master.[/+][-][DSIP-64] Add more workflow Integration case in master.[/-][+][DSIP-64] Add more workflow Integration test cases in master.[/+]binitshrest commentedon Oct 14, 2024
@ruanwenjun I am interest to work on this issue.
ruanwenjun commentedon Oct 14, 2024
Great, you can submit a PR to try, and you can find some example code under master/test
qiuyanjun888 commentedon Jun 11, 2026
Hi @ruanwenjun , I would like to work on this issue by adding one workflow integration test case first. Could you please assign it to me?
qiuyanjun888 commentedon Jun 11, 2026
@ruanwenjun I added one more master workflow integration test case for condition tasks with mixed predecessor states and opened a PR: #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 -DskipTestsLeonxLJX commentedon Aug 28, 2026
Hi! I'd like to work on this issue. Please let me know if I can be assigned. Thanks!
LeonxLJX commentedon Sep 1, 2026
Hi! I'd like to add more workflow Integration test cases in master (DSIP-64). Plan: add the test cases, wire into CI, verify. May I be assigned?
ruanwenjun commentedon Sep 1, 2026
@LeonxLJX You can directly submit PR.