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

pytest: support for node ids in general #15

Open
blueyed opened this issue Dec 4, 2017 · 0 comments
Open

pytest: support for node ids in general #15

blueyed opened this issue Dec 4, 2017 · 0 comments

Comments

@blueyed
Copy link
Contributor

blueyed commented Dec 4, 2017

Currently only ::method is matched, but there is also ::Class::method, and e.g. test_pytest_param_example.py::test_eval[1+7-8].

As for matching the ::Class I think using the normal command / instead of search() might work - using two patterns (/^\s*class Class;/\s*def method).
Currently s:setpos only supports calling a function though - are you up for changing this?
(I do not think that we can assume execute() to be available)
Would an internal method wrapping :normal make sense - maybe using execute() if it is available?

As for the parametrized suffix: this should get matched, but ignored - it all goes to the same method.

@kopischke kopischke self-assigned this Dec 4, 2017
@kopischke kopischke added this to the Future plans milestone Mar 4, 2018
@kopischke kopischke removed their assignment May 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants