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

Specify paramterized node in command line - Issue #649 #1452

Conversation

palaviv
Copy link
Contributor

@palaviv palaviv commented Mar 13, 2016

This PR solves the problem specified in Issue #649.
I added the test to test_mark but i am not so sure that is the right place.

@RonnyPfannschmidt
Copy link
Member

good work, its mergable as is

as for nitpicking, could you please add a todo comment to remove the workaround once collection structure properly contains parameterization

in any case i'll merge after work

@nicoddemus
Copy link
Member

(I gave a 👍 to the PR, does anyone receive a notification for that?)

@The-Compiler
Copy link
Member

@nicoddemus No, I think that's the whole point of those reactions - I think they got added because for bigger projects, dozens of 👍 comments were too noisy

@palaviv
Copy link
Contributor Author

palaviv commented Mar 14, 2016

Thanks for the good reviews :).
I will not be infront a computer until tommorow evening. I can add the TODO commnet then if you wish to wait.
Is there a PR or Issue that regard the planned change in the collection structure?

@RonnyPfannschmidt
Copy link
Member

no pr or issue, i have it on my personal agenda for the sprint

@palaviv
Copy link
Contributor Author

palaviv commented Mar 16, 2016

I added the TODO comment

@RonnyPfannschmidt
Copy link
Member

well done, thanks

RonnyPfannschmidt added a commit that referenced this pull request Mar 16, 2016
…mmand-line

Specify paramterized node in command line - fixes #649
@RonnyPfannschmidt RonnyPfannschmidt merged commit d70596d into pytest-dev:master Mar 16, 2016
@david-fliguer-ws
Copy link

Hi, how do I use this ?

I have parametrized tests which name are WST_6,WST_7,WST_61,WST_62 and I want a way to run specifically test WST_6, if I use the -k parameter then it will run also WST_61 and WST_62.

How can I solve this ?

@RonnyPfannschmidt
Copy link
Member

@david-fliguer-ws based on the test names you provided they cant be based on pytest parametrization, so this doesn't apply to you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants