CODEWITHSUNDEEP
pythonflake8
Chris B.
Chris B.

Reputation: 90211

How do I get Flake8 to work with F811 errors?

We're using flake8 to test our code, and we're using pytest with fixtures. The following code:

from staylists.tests.fixtures import fixture1  # noqa: F401

def test_case(fixture1):  # noqa: F811
    # Test goes here
    assert 1 == 1

Generates a lib/python/test.py:3:1: F811 redefinition of unused 'fixture1' from line 1 error during linting.

Upvotes: 11

Views: 6293

Answers (2)

anthony sottile
anthony sottile

Reputation: 69924

There are two "best practices" for sharing fixtures:

  1. define them in a conftest above both test modules
    • pytest will automatically share them between the two tests
  2. define a pytest plugin which exposes fixtures
    • pytest will make your fixtures available to all tests

bringing a fixture into a scope via import side-effects will trigger the issues you're seeing and is not recommended

Upvotes: 4

fafl
fafl

Reputation: 7385

The F401 and F811 errors can be avoided by moving all fixtures into the conftest.py file. Pytest loads this file automatically and makes all fixtures inside available in all tests, even without explicit import statements.

More discussion about the file can be found here: In py.test, what is the use of conftest.py files?

Upvotes: 11

Related Questions