diff options
| author | Benjamin Schubert <ben.c.schubert@gmail.com> | 2019-06-05 18:19:58 +0100 |
|---|---|---|
| committer | bst-marge-bot <marge-bot@buildstream.build> | 2019-06-06 15:05:21 +0000 |
| commit | 504069c35ea334666cc25455e32a9ad9e3bf0070 (patch) | |
| tree | 2f4a52b3d697119c3b18bb6cfb06df66395f09b1 | |
| parent | ff0555ab712d13ec3a7b8388684fdd2ea4e82926 (diff) | |
| download | buildstream-504069c35ea334666cc25455e32a9ad9e3bf0070.tar.gz | |
tests/artifactcache: Silence all 'redefined-outer-name' pylint errors
This is due to pytest fixtures having to be named the same as the test
arguments.
This is a pre-requisite to enable pylint on this directory
We need to do this per file as we can't blanket disable for directories.
See upstream issue: https://github.com/PyCQA/pylint/issues/618
| -rw-r--r-- | tests/artifactcache/cache_size.py | 3 | ||||
| -rw-r--r-- | tests/artifactcache/config.py | 3 | ||||
| -rw-r--r-- | tests/artifactcache/expiry.py | 3 | ||||
| -rw-r--r-- | tests/artifactcache/junctions.py | 3 | ||||
| -rw-r--r-- | tests/artifactcache/pull.py | 3 | ||||
| -rw-r--r-- | tests/artifactcache/push.py | 3 |
6 files changed, 18 insertions, 0 deletions
diff --git a/tests/artifactcache/cache_size.py b/tests/artifactcache/cache_size.py index 09f0f25d0..250094746 100644 --- a/tests/artifactcache/cache_size.py +++ b/tests/artifactcache/cache_size.py @@ -1,3 +1,6 @@ +# Pylint doesn't play well with fixtures and dependency injection from pytest +# pylint: disable=redefined-outer-name + import os from unittest import mock diff --git a/tests/artifactcache/config.py b/tests/artifactcache/config.py index 8adbb3274..39d8b9da3 100644 --- a/tests/artifactcache/config.py +++ b/tests/artifactcache/config.py @@ -1,3 +1,6 @@ +# Pylint doesn't play well with fixtures and dependency injection from pytest +# pylint: disable=redefined-outer-name + import pytest import itertools diff --git a/tests/artifactcache/expiry.py b/tests/artifactcache/expiry.py index 6aa518f1a..c67884046 100644 --- a/tests/artifactcache/expiry.py +++ b/tests/artifactcache/expiry.py @@ -17,6 +17,9 @@ # Authors: Tristan Maat <tristan.maat@codethink.co.uk> # +# Pylint doesn't play well with fixtures and dependency injection from pytest +# pylint: disable=redefined-outer-name + import os import re from unittest import mock diff --git a/tests/artifactcache/junctions.py b/tests/artifactcache/junctions.py index e1bbb31fd..8e38f26e1 100644 --- a/tests/artifactcache/junctions.py +++ b/tests/artifactcache/junctions.py @@ -1,3 +1,6 @@ +# Pylint doesn't play well with fixtures and dependency injection from pytest +# pylint: disable=redefined-outer-name + import os import shutil import pytest diff --git a/tests/artifactcache/pull.py b/tests/artifactcache/pull.py index cdadf583a..eafcec64b 100644 --- a/tests/artifactcache/pull.py +++ b/tests/artifactcache/pull.py @@ -1,3 +1,6 @@ +# Pylint doesn't play well with fixtures and dependency injection from pytest +# pylint: disable=redefined-outer-name + import multiprocessing import os import signal diff --git a/tests/artifactcache/push.py b/tests/artifactcache/push.py index 4f31148b7..a49ee32d2 100644 --- a/tests/artifactcache/push.py +++ b/tests/artifactcache/push.py @@ -1,3 +1,6 @@ +# Pylint doesn't play well with fixtures and dependency injection from pytest +# pylint: disable=redefined-outer-name + import multiprocessing import os import signal |