Fix some coverage and pep8 issues.

Author: eriknw

toolz/functoolz.py

@@ -329,14 +329,14 @@ def __reduce__(self):
         func = self.func
         modname = getattr(func, '__module__', None)
         qualname = getattr(func, '__qualname__', None)
-        if qualname is None:
+        if qualname is None:  # pragma: py3 no cover
             qualname = getattr(func, '__name__', None)
         is_decorated = None
         if modname and qualname:
             attrs = []
             obj = import_module(modname)
             for attr in qualname.split('.'):
-                if isinstance(obj, curry):
+                if isinstance(obj, curry):  # pragma: py2 no cover
                     attrs.append('func')
                     obj = obj.func
                 obj = getattr(obj, attr, None)
@@ -351,7 +351,8 @@ def __reduce__(self):
         # functools.partial objects can't be pickled
         userdict = tuple((k, v) for k, v in self.__dict__.items()
                          if k != '_partial')
-        state = (type(self), func, self.args, self.keywords, userdict, is_decorated)
+        state = (type(self), func, self.args, self.keywords, userdict,
+                 is_decorated)
         return (_restore_curry, state)
 
 

toolz/tests/test_serialization.py

@@ -3,6 +3,7 @@
 import toolz.curried.exceptions
 import pickle
 from toolz.compatibility import PY3
+from toolz.utils import raises
 
 
 def test_compose():
@@ -177,3 +178,11 @@ def preserves_identity(obj):
     func2 = pickle.loads(pickle.dumps(func1))
     assert func1 is not func2
     assert func1(4) == func2(4) == 10
+
+
+def test_curried_bad_qualname():
+    @toolz.curry
+    class Bad(object):
+        __qualname__ = 'toolz.functoolz.not.a.valid.path'
+
+    assert raises(pickle.PicklingError, lambda: pickle.dumps(Bad))