-
-
Notifications
You must be signed in to change notification settings - Fork 49
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
Support snippets of test configuration to be merged into settings #457
Comments
That's nice -- it wouldn't support complete ad hoc test settings (per my original discussion on Slack) but it would make it much less verbose to support a specific set of test settings which should be sufficient, and it would be no big deal to add more |
Nice! I'll finish up my work on seancorfield/polylith-external-test-runner#5 and seancorfield/polylith-external-test-runner#6 and document usage per this new machinery in Polylith. |
There is a need for test configuration to be merged into the settings, e.g. when running the external test runner, see this comment.
We could solve this by adding a
:test-configs
key toworkspace.edn
, e.g.The Kaocha test runner can now be selected with:
Or the External test runner, without the integration tests:
The idea is that the selected configuration snippets will be merged to the
:test
key in the workspace configuration, and if the value lives in a collection (e.g. vector or hash map) it will be added to existing values, e.g.:Will end up as:
If we run commands from a shell, we will also get autocomplete for the
with
option.The text was updated successfully, but these errors were encountered: