27

Is it possible to disable an inspection for the whole file in PyCharm?

The reason this is needed is when dealing with py.test. It uses fixtures which appear to shadow function parameters, and at the same time cause unresolved references. e.g.:

from myfixtures import user  # Unused import statement warning

def test_is_awesome(user):  # Shadows name 'user' from outer scope warning
    assert user.is_awesome()

There is also other warnings from py.test, such as using pytest.raises() causes a "Can not find reference 'raises'" in pytest.py.

Maybe there's another way to fix these problems? Maybe I'm using py.test incorrectly?

gak
  • 30,970
  • 26
  • 115
  • 152

3 Answers3

24

Is it possible to disable an inspection for the whole file in PyCharm?

Yes. This answer is for this question only (and not about "Maybe there's another way to fix these problems? Maybe I'm using py.test incorrectly?").

  1. "Settings | Scopes"
  2. Create new scope that would include such "unwanted" file(s)
  3. "Settings | Inspections"
  4. Find "problematic" inspection
  5. Right click and choose "Add scope"
  6. Disable that inspection for that specific scope

Alternatively (may work or may not: depends on actual inspection .. and I'm not sure if it actually works in PyCharm this way -- not a PyCharm user myself unfortunately)

  1. Alt + Enter while caret is standing on error/warning place in your code
  2. Select correct entry from appeared popup menu
  3. Using Arrow Right key expand submenu
  4. Look for "Suppress inspection" option

This is how it looks in PhpStorm (screenshot shows "suppress for statement" option and not "suppress for whole file"):

enter image description here

Related: https://stackoverflow.com/a/20803118/783119

Community
  • 1
  • 1
LazyOne
  • 148,457
  • 42
  • 363
  • 369
  • 2
    Thanks. This is extra handy since you can share these settings over source control. – gak Apr 09 '14 at 21:26
6

To answer the "Maybe I'm using py.test incorrectly?" question:

Importing fixtures is not the best pattern to follow. Instead it is better to put fixtures in a conftest.py file of the package which needs them. If a fixture is used in two packages just put a conftest.py in their parent directory and put the fixture in there. This should get rid of the unused import and shadowing warnings.

As for the pytest.raises namespace issue, I don't think there's a solution to this currently. This is something that pylint suffers as well (and I think there's an effort to create a py.test plugin to pylint to address these things). So I think at the end of the day the linter will still need to know a little about py.test.

flub
  • 5,457
  • 22
  • 23
1

I found that putting the per statement inspection:

    ...
    # noinspection PyUnusedLocal
    instance = mommy.make(Contact)

at the start of the file suppressed it for the whole file:

# noinspection PyUnusedLocal
from django.urls import reverse
hum3
  • 1,333
  • 13
  • 17
  • 1
    Using PyCharm 2020.2 (Professional Edition) Build #PY-202.6397.98, built on July 27, 2020 and putting `# noinspection PyUnusedLocal` as the very first line of the file appears to work for me, but the file is an auto-generated settings file for jupyter. All of the lines are comments or commented out defaults, but for the one setting I had edited somewhere in the middle. Once comments are removed, that is the next statement. After editing more lines, those later lines show as having an issue. – hlongmore Aug 20 '20 at 20:11
  • 1
    Here is a reference for the different inspections that you can disable via `noinspection`: https://www.jetbrains.com/help/pycharm/disabling-and-enabling-inspections.html#comments-ref – Konstantin Aug 19 '21 at 07:44