[cli] Enable job test command to use the --config-file option #180
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
klio job test
currently takes in the--config-file
option but does nothing with it, since therequires_config_file
flag is set toFalse
.The config file option was also ignored in
klio-exec
, but that was fixed via PR 177.This PR sets enables the use of the
--config-file
option by doing 2 things:TestPipeline().requires_config_file
toTrue
.TestPipeline
handles command generation so that the--config-file
option is appended after thetest
command, but before pytest arguments are added.Before this change, the generated command would be:
klio job test -- py_arg1 pyarg2 ... pyargN --config-file config.yaml
This causes the
--config-file
option to be ignored.After this change, the generated command would be:
klio job test --config-file config.yaml -- py_arg1 pyarg 2 .. pyargN
Checklist for PR author(s)
[cli] Fixes bugs in 'klio job fake-cmd'
.docs/src
.docs/src/reference/<package>/changelog.rst
.