pytest needed as (runtime) requirement?

pytest is now listed as install_requires in setup.py, and also included as ‘run requirement’ in the conda recipe. I think it’s better to use tests_require (see http://setuptools.readthedocs.io/en/latest/setuptools.html) for setup.py, and not include it in the recipe. In general I don’t think test packages should be dependencies, people that want to test will either have this in their environment already, or will read the documentation.

[mwcraig]

Well, we also need it for the Testrunner class, and I vaguely remember that we had issues with the recipe/building the conda package when it didn’t include it in the run requirements: https://travis-ci.org/conda-forge/astropy-feedstock/jobs/251188788

That should be fixable in the conda recipe, which has a separate section for test requirements (that actually works as opposed to tests_require in setup.py).

[drdavella]

@mwcraig, @bsipocz if you’re willing to wait until #6606 is merged, then we can simply add pytest-astropy to the conda recipe test requirements.