Warn about `__init__.py` in tests about flake8-pytest-style HOT 6 OPEN

3. When you have duplicate test names

from flake8-pytest-style.

Hi @sobolevn,

I'm not sure I agree that this rule should be enforced. Per the pytest docs, layout without __init__.py files is only one of supported layouts, and an alternative layout with multiple packages and __init__.py files is also supported. I've seen both in my experience, both have pros and cons, and I don't really feel that either of them should be enforced for all projects.

from flake8-pytest-style.

That's why I have added a note about a configuration:

It is probably a good idea to make this configurable:

tests path
style: with __init__.py or without

from flake8-pytest-style.

My bad, must've missed it :(
I'll keep this in mind, thanks

from flake8-pytest-style.

Having this configurable is a good idea. While pytest emulates the presence of __init__.py, there are cases when this may break (especially when you also have to support Python 2).

I personally like not having __init__.py, but FTR here's some corner cases that I met:

1. various tooling does not support/understand implicit namespaces or such support is implemented poorly
2. when the custom import machinery is implemented, it may conflict with what pytest does and it has to be disabled (this is the case I saw in Ansible, in particular)

It's probably worth mentioning the presence of the exceptional cases in the doc for the rule, once implemented.

from flake8-pytest-style.

unless importlib mode is used, its wrong to drop them

from flake8-pytest-style.