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

Add documentation #2

Closed
RKrahl opened this issue Apr 21, 2016 · 5 comments
Closed

Add documentation #2

RKrahl opened this issue Apr 21, 2016 · 5 comments
Labels
enhancement New feature or request
Milestone

Comments

@RKrahl
Copy link
Owner

RKrahl commented Apr 21, 2016

Add more documentation. Need at least a text explaining the basic concept. What is this good for, statement of the problem that this plugin solves. How to use it.

@RKrahl RKrahl added the enhancement New feature or request label Apr 21, 2016
@Midnighter
Copy link

I can't believe pytest hasn't had this so far... I would love to use this feature, could you add a simple example to the readme on how to mark dependencies between two tests or is your code still very much in flux?

@RKrahl
Copy link
Owner Author

RKrahl commented Feb 2, 2017

Sure I could and should add more documentation, including examples. I don't think that the stability of the API is an issue that could restrain this. The only problem is … well, I need to get around to do it. That is what this open issue is about. I can only work in my spare time on this project.

In the mean time, have I look into the tests. tests/test_02_simple_dependency.py provides pretty generic examples for the basic usage (watch for the multi line string arguments to ctestdir.makepyfile()). tests/test_03_param.py shows how to combine the dependency marks with parameterizing tests. A rather complex example of a real application can be found in another project using pytest-dependency, here: icatproject/python-icat:tests/test_06_ids.py (look out for the comment reading "Enriched versions of testdatafiles and testdatasets, decorated with appropriate dependency markers, …").

@Midnighter
Copy link

Thank you very much that's really helpful. And believe me, I know the pain of developing something that meets your own high standards in your spare time.

@RKrahl RKrahl added this to the 0.2 milestone May 14, 2017
@RKrahl
Copy link
Owner Author

RKrahl commented May 26, 2017

Obviously, documentation is never finished. Now I wrote something, I hope will cover the basic stuff. At least I believe it's enough to close this issue for now.

@Midnighter: if you got some time, maybe you could have a look at the online documentation and drop some comments. For the package author, it's always difficult to assess what is obvious to the user and what needs more explanations.

@RKrahl RKrahl closed this as completed in 0de07ad May 26, 2017
@Midnighter
Copy link

The docs look good to me. Thank you for the addition. I'll let you know if I come across any uncertainties/missing descriptions.

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

No branches or pull requests

2 participants