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

Check pytest-dependency with test classes #6

Closed
RKrahl opened this issue May 28, 2017 · 1 comment
Closed

Check pytest-dependency with test classes #6

RKrahl opened this issue May 28, 2017 · 1 comment
Labels
documentation Improvements or additions to documentation
Milestone

Comments

@RKrahl
Copy link
Owner

RKrahl commented May 28, 2017

I did not experiment much with test classes by now. Need to verify that the plugin also works for test classes, should add appropriate tests to the test suite and add the usage for test classes to the documentation.

@RKrahl RKrahl added documentation Improvements or additions to documentation question Further information is requested labels May 28, 2017
@RKrahl RKrahl added this to the 0.3 milestone May 28, 2017
@RKrahl
Copy link
Owner Author

RKrahl commented Sep 16, 2017

It turns out that pytest-dependency works just fine with test classes and the usage is straight forward.

There is only one issue: for methods of test classes, the default name is just the method name. This may cause conflicts if there is a function having the same name outside the class or if there is another class with a method of the same name. Such conflicts may have unexpected results. To mend this, I will prepend the class name to the default test name.

@RKrahl RKrahl removed the question Further information is requested label Sep 16, 2017
@RKrahl RKrahl closed this as completed in c767eca Sep 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

1 participant