-
Notifications
You must be signed in to change notification settings - Fork 14
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
allow for TESTARGS
in invoke test
#62
Comments
I started looking at this and noticed the following:
Adding
Oddly enough, kwargs={"kwargs": "a b --foo --bar"} So for some reason it will be wrapped with a dict!
So, even though you added My proposal in this scenario would be to add One downside of this approach is that it can clash with Cc @kjaymiller @aiven-sal @ahmedsobeh. WDYT? |
@mkmkme I think that could work, would that change also be available in Most of the testing that I want to do is for smaller tests while implementing. |
@kjaymiller yes it will. I'll create a PR with |
Version: 6.0.0b1
Platform: MacOS Arm M2
Description: Description of your issue, stack traces from errors and code that reproduces the issue
Currently the
invoke test
command pre-configures some commands but pushes you down a few paths:I propose that we add the following to all the tests functions in
task.py
either:TESTARGS
parameters that allows for the user to apply their own args to the testtest_path
,-v (verbosity)
,-k (keyword)
The text was updated successfully, but these errors were encountered: