From cbd7a095c0e8ab80a6b7cca1ae71641e74e8e2bc Mon Sep 17 00:00:00 2001 From: Dylan Baker Date: Thu, 2 Feb 2017 10:05:06 -0800 Subject: tests: Instruct pylint to ignore a large swath of warnings There are a number of things pylint warns on that absolutely make sense to fix in production code, but for unittests they either don't matter (like naming variables "foo"), can't be fixed (TestCase methods that don't use self because they use a mock assert), or the descriptive names violate PEP8. These are annoying and create noise, so tell pylint to ignore them. --- tests/commands/thread_test.py | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'tests/commands/thread_test.py') diff --git a/tests/commands/thread_test.py b/tests/commands/thread_test.py index bb8514d6..704039c1 100644 --- a/tests/commands/thread_test.py +++ b/tests/commands/thread_test.py @@ -10,6 +10,13 @@ import unittest from alot.commands import thread +# Good descriptive test names often don't fit PEP8, which is meant to cover +# functions meant to be called by humans. +# pylint: disable=invalid-name + +# These are tests, don't worry about names like "foo" and "bar" +# pylint: disable=blacklisted-name + class Test_ensure_unique_address(unittest.TestCase): -- cgit v1.2.3