Enable pylint warnings on presubmit checks.

third_party/WebKit/Tools/Scripts/webkitpy/style/checkers/python_unittest.py

 # Copyright (C) 2010 Chris Jerdonek (cjerdonek@webkit.org)
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions
 # are met:
 # 1.  Redistributions of source code must retain the above copyright
 #     notice, this list of conditions and the following disclaimer.
 # 2.  Redistributions in binary form must reproduce the above copyright
 #     notice, this list of conditions and the following disclaimer in the
 #     documentation and/or other materials provided with the distribution.
@@ skipping 38 matching lines @@
 
         def _mock_handle_style_error(line_number, category, confidence,
                                      message):
             error = (line_number, category, confidence, message)
             errors.append(error)
 
         current_dir = os.path.dirname(__file__)
         file_path = os.path.join(current_dir, "python_unittest_input.py")
 
         checker = PythonChecker(file_path, _mock_handle_style_error)
-        checker.check(lines=[])
+        checker.check()
 
-        self.assertEqual(errors, [
-            (4, "pep8/W291", 5, "trailing whitespace"),
-            (4, "pylint/E1601(print-statement)", 5, "[] print statement used"),
-            (4, "pylint/E0602(undefined-variable)", 5, "[] Undefined variable 'error'"),
-        ])
+        print errors
+
+        self.assertEqual(
+            [
+                (2, 'pep8/W291', 5, 'trailing whitespace'),
+                (3, 'pep8/E261', 5, 'at least two spaces before inline comment'),
+                (3, 'pep8/E262', 5, "inline comment should start with '# '"),
+                (2, 'pylint/C0303(trailing-whitespace)', 5, '[] Trailing whitespace'),

[qyearsley, 2016/05/29 21:00:33]

Note that pylint and pep8 both print warnings about trailing whitespace; this is
redundant but not a problem.

+                (2, 'pylint/E1601(print-statement)', 5, '[] print statement used'),
+                (2, 'pylint/E0602(undefined-variable)', 5, u"[] Undefined variable 'error'"),
+                (3, 'pylint/W0611(unused-import)', 5, '[] Unused import math'),

[qyearsley, 2016/05/29 21:00:33]

After this CL, warnings about unused imports etc. should also be shown by this
python checker.

+            ],
+            errors)