Skip to content
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

[Tests] Add workflow run basic spec test #2059

Open
chrismclarke opened this issue Aug 23, 2023 · 0 comments · May be fixed by #2060
Open

[Tests] Add workflow run basic spec test #2059

chrismclarke opened this issue Aug 23, 2023 · 0 comments · May be fixed by #2060
Assignees
Labels
feature Work on app features/modules

Comments

@chrismclarke
Copy link
Member

What?
Following work in #2041, identified it would be useful to have some basic spec tests that cover workflow functionality

Why?
Catch regression issues

How?
Add .spec to scripts test in packages\scripts\src\commands\workflow to ensure workflow can be triggered with child workflows, and that all receive and evaluate options as expected

Other details
Whilst trying locally I've had quite a few issues getting the tests to run without executing commander.js commands (or passing additional args to them). Will likely require some extra refactoring on top of writing tests.

@chrismclarke chrismclarke added the feature Work on app features/modules label Aug 23, 2023
This was referenced Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Work on app features/modules
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants