Repository navigation
Top level tests directory #51
Description
tests [..] should be placed in a submodule of funniest. so that they can be imported, but won't pollute the global namespace.
There's a good case to be made that tests should not live inside your package, but instead inside a separate top-level tests/. This is how one of the PyPA core devs does it. This is also what https://packaging.python.org/tutorials/packaging-projects/ (which makes sense, it's written by the same group). The reason for this is it lets you cull tests from your binary distribution, since people installing from binary almost never care to run your tests or have them take up space -- though you need to go out of your way to make sure to re-include them in your sdist, but that's a one-line addition.
https://stackoverflow.com/questions/61151/where-do-the-python-unit-tests-go covers the three main options:
- Side-by-side your code (and says they prefer it; to be honest, so do I, it feels more right)
- In a top level tests/
- In a tests/ subpackage
The current recommendation in this doc is for 3. PyPA thinks you should use 2. What are your thoughts?
Activity
merwok commentedon Jul 9, 2020
Side-by-sidewith the code only works if you are not using ansrcdirectory to isolate your package source (see https://blog.ganssle.io/articles/2019/08/test-as-installed.html)Top-level
testsis common. Depending on your build tool (setuptools or flit or poetry or other), you may need configuration to make sure that the tests are present in sdists, but not installed.kousu commentedon Jul 12, 2020
Now I'm veering off-topic, but I wish there wasn't a distinction between installed/not installed. It isn't necessary in Javascript or Scheme or lots of other languages. I've been thinking about it and I think the reason comes down to the rule that submodules locate themselves -- and relative imports -- by
__name__, splitting on "."s instead of by__file__, splitting on "/"s or better yet following "..". Can you think of any other reason?Just to spread this knowledge around since it's going to get buried, PyPA now seems to be advocating either
or using https://pypi.org/project/setuptools-scm/. But that seems weird to me; the manifest seems fine.