-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Have workflow engine test suites run all flows #1743
Labels
Comments
Andrew-S-Rosen
changed the title
Have workflow engine test suites run all tests
Have workflow engine test suites run all jobs and flows
Feb 24, 2024
Andrew-S-Rosen
changed the title
Have workflow engine test suites run all jobs and flows
Have workflow engine test suites run all flows
Feb 29, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What new feature would you like to see?
We should figure out some way to refactor the workflow engine test suites so they run every flow. This will help catch bugs earlier, like #1742. The challenge is we want to avoid copy/paste, but they all are called in different ways, e.g.
Client()
for Dask,.result()
calls for some and not others. Still, I'd rather do copy/paste for now than have untested code altogether.The text was updated successfully, but these errors were encountered: