-
Notifications
You must be signed in to change notification settings - Fork 38
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
The need to specify a test suite in ${DYNAMICTEST(S)} is somewhat cumbersome #43
Comments
Hi @nizwiz Can you tell me a bit more about the use case? is has not been designed to be used by humans. The suite can only be skipped if exactly one suite is used. |
With use case i mean in which situation are you using that and why can you not use tags or other things. |
Well, the use case is pretty much as described in section "Selection of test cases by name" in Readme.rst. |
Hm.... Let me think about this. but i think a better approach for you would be a Variablefile! define a yaml variable file with the test case names as a list and hand it over to robot. that should make this ugly command line better. |
DYNAMICTESTS:
- MyDataDrivenTestSuiteWithParticularlyLongName.0101
- MyDataDrivenTestSuiteWithParticularlyLongName.0102
- MyDataDrivenTestSuiteWithParticularlyLongName.0103 |
Oh wait, I'm actually also using pabot with --testlevelsplit (because each test case spends a considerable amount of time just waiting for a result to be computed). I've double-checked and, in deed, when using pabot together with DYNAMICTESTS, there doesn't seem to be any parallelization. I guess I'll have to use tags anyhow, then. |
Good point! |
I will let it open as a maker for me to do that addition to docs! thanks for your contribution!!! |
Well, thank you for this excellent RF extension! It's really indispensable. |
Just imagine having to type this:
Yeah you can copy'n'paste, but in a real world example you would want to copy'n'paste the test names as well, and having to insert the suite name in between test names is cumbersome (and in the frequent case that one specifies a single robot file on the command line, it's also redundant).
I think it would be great if the test suite part were optional.
I'd be willing to contribute a pull request.
The text was updated successfully, but these errors were encountered: