pylint

src/greentest/greentest/errorhandler.py

@@ -20,9 +20,11 @@
 
 from functools import wraps
 
+
 def wrap_error_fatal(method):
     import gevent
-    SYSTEM_ERROR = gevent.get_hub().SYSTEM_ERROR
+    system_error = gevent.get_hub().SYSTEM_ERROR
+
     @wraps(method)
     def wrapper(self, *args, **kwargs):
         # XXX should also be able to do gevent.SYSTEM_ERROR = object
@@ -32,13 +34,14 @@ def wrap_error_fatal(method):
         try:
             return method(self, *args, **kwargs)
         finally:
-            gevent.get_hub().SYSTEM_ERROR = SYSTEM_ERROR
+            gevent.get_hub().SYSTEM_ERROR = system_error
     return wrapper
 
 
 def wrap_restore_handle_error(method):
     import gevent
     old = gevent.get_hub().handle_error
+
     @wraps(method)
     def wrapper(self, *args, **kwargs):
         try:

src/greentest/greentest/leakcheck.py

@@ -18,9 +18,11 @@
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 # THE SOFTWARE.
 
+import sys
 import gc
 import collections
 import types
+from functools import wraps
 
 import gevent.core
 
@@ -68,7 +70,7 @@ def wrap_refcount(method):
         return diff
 
     @wraps(method)
-    def wrapper(self, *args, **kwargs):
+    def wrapper(self, *args, **kwargs): # pylint:disable=too-many-branches
         gc.collect()
         gc.collect()
         gc.collect()

src/greentest/greentest/six.py

 import sys
+# pylint:disable=unused-argument,import-error
 
 PY3 = sys.version_info[0] >= 3
 
@@ -18,7 +19,7 @@ if PY3:
         raise value
 
     xrange = range
-    string_types = str,
+    string_types = (str,)
     text_type = str
 
 else:
@@ -36,7 +37,7 @@ else:
 
     import __builtin__ as builtins
     xrange = builtins.xrange
-    string_types = builtins.basestring,
+    string_types = (builtins.basestring,)
     text_type = builtins.unicode
 
     exec_("""def reraise(tp, value, tb=None):

src/greentest/greentest/skipping.py

@@ -23,12 +23,15 @@ import unittest
 
 from greentest import sysinfo
 
+
 def _do_not_skip(reason):
     assert reason
+
     def dec(f):
         return f
     return dec
 
+
 if sysinfo.WIN:
     skipOnWindows = unittest.skip
 else: