Repository navigation
Update testing framework documentation - #5446
Conversation
- Rename `py.test` to `pytest` where ever it appears
- Add example showing the `--args` argument to pass arguments
to pytest
- Remove discussion of running `pytest` directly
pllim
left a comment
pllim
left a comment
There was a problem hiding this comment.
Thanks. You need to fix Sphinx error Unknown target name: "pytest". for the document to compile properly.
| through with the ``--args`` argument:: | ||
|
|
||
| .. note:: | ||
| > python setup.py test --args "-x" |
There was a problem hiding this comment.
Pretty sure it is --args="-x" (with equal sign), as that is what I use in my own package. Also, you can pass in multiple options using --args='-x --y="xyz"', if you want to make the document even more detailed.
There was a problem hiding this comment.
Either one is valid, actually.
|
My find-and-replace missed a spot. Should be fixed now! |
|
Looks good to me. Thanks! I'll leave this open for now in case others want to review. |
|
p.s. I'll just tag this as 1.3 since its release is coming up anyway, but feel free to re-milestone if needed. |
|
This looks good to me. For a bit more background, this came out of the efforts in #5435, which make it look like it may be very difficult to make the @astrofrog or @bsipocz, do you have any thoughts on this (and maybe @embray depending on how much he's watching his notifications)? I.e., do you see push-back to just saying directly invoking |
|
I haven't seen any objections in 2 weeks, so merging. |
|
@eteq - It seems I never noticed this notification. Anyway it looks to be a good improvement, and I don't see much problems with the removal of direct py.test support (that isn't working anyway and should have a small enough priority to get fixed any time soon). |
4 participants
py.test.