lint: clean up old statements
We can delete a number of disable statements that are no longer relevant,
replace the W/E/F constants with their human readable names, and scope a
few to where they're needed instead of disabling in the entire module.
BUG=None
TEST=`cros lint` is unchanged
Change-Id: Ic7ac1ab410917b075212b38412f8a1e381b8b915
Reviewed-on: https://chromium-review.googlesource.com/1135635
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Don Garrett <dgarrett@chromium.org>
diff --git a/cli/command_unittest.py b/cli/command_unittest.py
index f228e11..1c1abf1 100644
--- a/cli/command_unittest.py
+++ b/cli/command_unittest.py
@@ -49,7 +49,7 @@
def testBadUseOfCommandDecorator(self):
"""Tests that our decorator correctly rejects bad test commands."""
try:
- # pylint: disable=W0612
+ # pylint: disable=unused-variable
@command.CommandDecorator('bad')
class BadTestCommand(object):
"""A command that wasn't implemented correctly."""
@@ -97,7 +97,7 @@
class CommandTest(cros_test_lib.MockTestCase):
"""This test class tests that we can load modules correctly."""
- # pylint: disable=W0212
+ # pylint: disable=protected-access
def testFindModules(self):
"""Tests that we can return modules correctly when mocking out glob."""