azure-quantum 2.5.0.dev0__py3-none-any.whl → 2.5.0.dev2__py3-none-any.whl

azure/quantum/_client/_serialization.py

@@ -1,4 +1,3 @@
-# pylint: disable=too-many-lines
 # --------------------------------------------------------------------------
 #
 # Copyright (c) Microsoft Corporation. All rights reserved.
@@ -25,6 +24,7 @@
 #
 # --------------------------------------------------------------------------
 
+# pylint: skip-file
 # pyright: reportUnnecessaryTypeIgnoreComment=false
 
 from base64 import b64decode, b64encode
@@ -52,6 +52,7 @@ from typing import (
     MutableMapping,
     Type,
     List,
+    Mapping,
 )
 
 try:
@@ -90,8 +91,6 @@ class RawDeserializer:
         :param data: Input, could be bytes or stream (will be decoded with UTF8) or text
         :type data: str or bytes or IO
         :param str content_type: The content type.
-        :return: The deserialized data.
-        :rtype: object
         """
         if hasattr(data, "read"):
             # Assume a stream
@@ -113,7 +112,7 @@ class RawDeserializer:
             try:
                 return json.loads(data_as_str)
             except ValueError as err:
-                raise DeserializationError("JSON is invalid: {}".format(err), err) from err
+                raise DeserializationError("JSON is invalid: {}".format(err), err)
         elif "xml" in (content_type or []):
             try:
 
@@ -145,8 +144,6 @@ class RawDeserializer:
                 # context otherwise.
                 _LOGGER.critical("Wasn't XML not JSON, failing")
                 raise DeserializationError("XML is invalid") from err
-        elif content_type.startswith("text/"):
-            return data_as_str
         raise DeserializationError("Cannot deserialize content-type: {}".format(content_type))
 
     @classmethod
@@ -156,11 +153,6 @@ class RawDeserializer:
         Use bytes and headers to NOT use any requests/aiohttp or whatever
         specific implementation.
         Headers will tested for "content-type"
-
-        :param bytes body_bytes: The body of the response.
-        :param dict headers: The headers of the response.
-        :returns: The deserialized data.
-        :rtype: object
         """
         # Try to use content-type from headers if available
         content_type = None
@@ -178,6 +170,13 @@ class RawDeserializer:
         return None
 
 
+try:
+    basestring  # type: ignore
+    unicode_str = unicode  # type: ignore
+except NameError:
+    basestring = str
+    unicode_str = str
+
 _LOGGER = logging.getLogger(__name__)
 
 try:
@@ -190,30 +189,15 @@ class UTC(datetime.tzinfo):
     """Time Zone info for handling UTC"""
 
     def utcoffset(self, dt):
-        """UTF offset for UTC is 0.
-
-        :param datetime.datetime dt: The datetime
-        :returns: The offset
-        :rtype: datetime.timedelta
-        """
+        """UTF offset for UTC is 0."""
         return datetime.timedelta(0)
 
     def tzname(self, dt):
-        """Timestamp representation.
-
-        :param datetime.datetime dt: The datetime
-        :returns: The timestamp representation
-        :rtype: str
-        """
+        """Timestamp representation."""
         return "Z"
 
     def dst(self, dt):
-        """No daylight saving for UTC.
-
-        :param datetime.datetime dt: The datetime
-        :returns: The daylight saving time
-        :rtype: datetime.timedelta
-        """
+        """No daylight saving for UTC."""
         return datetime.timedelta(hours=1)
 
 
@@ -227,7 +211,7 @@ except ImportError:  # Python 2.7
         :param datetime.timedelta offset: offset in timedelta format
         """
 
-        def __init__(self, offset) -> None:
+        def __init__(self, offset):
             self.__offset = offset
 
         def utcoffset(self, dt):
@@ -256,26 +240,24 @@ except ImportError:
 _FLATTEN = re.compile(r"(?<!\\)\.")
 
 
-def attribute_transformer(key, attr_desc, value):  # pylint: disable=unused-argument
+def attribute_transformer(key, attr_desc, value):
     """A key transformer that returns the Python attribute.
 
     :param str key: The attribute name
     :param dict attr_desc: The attribute metadata
     :param object value: The value
     :returns: A key using attribute name
-    :rtype: str
     """
     return (key, value)
 
 
-def full_restapi_key_transformer(key, attr_desc, value):  # pylint: disable=unused-argument
+def full_restapi_key_transformer(key, attr_desc, value):
     """A key transformer that returns the full RestAPI key path.
 
-    :param str key: The attribute name
+    :param str _: The attribute name
     :param dict attr_desc: The attribute metadata
     :param object value: The value
     :returns: A list of keys using RestAPI syntax.
-    :rtype: list
     """
     keys = _FLATTEN.split(attr_desc["key"])
     return ([_decode_attribute_map_key(k) for k in keys], value)
@@ -288,26 +270,19 @@ def last_restapi_key_transformer(key, attr_desc, value):
     :param dict attr_desc: The attribute metadata
     :param object value: The value
     :returns: The last RestAPI key.
-    :rtype: str
     """
     key, value = full_restapi_key_transformer(key, attr_desc, value)
     return (key[-1], value)
 
 
 def _create_xml_node(tag, prefix=None, ns=None):
-    """Create a XML node.
-
-    :param str tag: The tag name
-    :param str prefix: The prefix
-    :param str ns: The namespace
-    :return: The XML node
-    :rtype: xml.etree.ElementTree.Element
-    """
+    """Create a XML node."""
     if prefix and ns:
         ET.register_namespace(prefix, ns)
     if ns:
         return ET.Element("{" + ns + "}" + tag)
-    return ET.Element(tag)
+    else:
+        return ET.Element(tag)
 
 
 class Model(object):
@@ -321,7 +296,7 @@ class Model(object):
 
     def __init__(self, **kwargs: Any) -> None:
         self.additional_properties: Optional[Dict[str, Any]] = {}
-        for k in kwargs:  # pylint: disable=consider-using-dict-items
+        for k in kwargs:
             if k not in self._attribute_map:
                 _LOGGER.warning("%s is not a known attribute of class %s and will be ignored", k, self.__class__)
             elif k in self._validation and self._validation[k].get("readonly", False):
@@ -330,23 +305,13 @@ class Model(object):
                 setattr(self, k, kwargs[k])
 
     def __eq__(self, other: Any) -> bool:
-        """Compare objects by comparing all attributes.
-
-        :param object other: The object to compare
-        :returns: True if objects are equal
-        :rtype: bool
-        """
+        """Compare objects by comparing all attributes."""
         if isinstance(other, self.__class__):
             return self.__dict__ == other.__dict__
         return False
 
     def __ne__(self, other: Any) -> bool:
-        """Compare objects by comparing all attributes.
-
-        :param object other: The object to compare
-        :returns: True if objects are not equal
-        :rtype: bool
-        """
+        """Compare objects by comparing all attributes."""
         return not self.__eq__(other)
 
     def __str__(self) -> str:
@@ -366,11 +331,7 @@ class Model(object):
 
     @classmethod
     def _create_xml_node(cls):
-        """Create XML node.
-
-        :returns: The XML node
-        :rtype: xml.etree.ElementTree.Element
-        """
+        """Create XML node."""
         try:
             xml_map = cls._xml_map  # type: ignore
         except AttributeError:
@@ -390,9 +351,7 @@ class Model(object):
         :rtype: dict
         """
         serializer = Serializer(self._infer_class_models())
-        return serializer._serialize(  # type: ignore # pylint: disable=protected-access
-            self, keep_readonly=keep_readonly, **kwargs
-        )
+        return serializer._serialize(self, keep_readonly=keep_readonly, **kwargs)  # type: ignore
 
     def as_dict(
         self,
@@ -426,15 +385,12 @@ class Model(object):
 
         If you want XML serialization, you can pass the kwargs is_xml=True.
 
-        :param bool keep_readonly: If you want to serialize the readonly attributes
         :param function key_transformer: A key transformer function.
         :returns: A dict JSON compatible object
         :rtype: dict
         """
         serializer = Serializer(self._infer_class_models())
-        return serializer._serialize(  # type: ignore # pylint: disable=protected-access
-            self, key_transformer=key_transformer, keep_readonly=keep_readonly, **kwargs
-        )
+        return serializer._serialize(self, key_transformer=key_transformer, keep_readonly=keep_readonly, **kwargs)  # type: ignore
 
     @classmethod
     def _infer_class_models(cls):
@@ -444,7 +400,7 @@ class Model(object):
             client_models = {k: v for k, v in models.__dict__.items() if isinstance(v, type)}
             if cls.__name__ not in client_models:
                 raise ValueError("Not Autorest generated code")
-        except Exception:  # pylint: disable=broad-exception-caught
+        except Exception:
             # Assume it's not Autorest generated (tests?). Add ourselves as dependencies.
             client_models = {cls.__name__: cls}
         return client_models
@@ -457,7 +413,6 @@ class Model(object):
         :param str content_type: JSON by default, set application/xml if XML.
         :returns: An instance of this model
         :raises: DeserializationError if something went wrong
-        :rtype: ModelType
         """
         deserializer = Deserializer(cls._infer_class_models())
         return deserializer(cls.__name__, data, content_type=content_type)  # type: ignore
@@ -476,11 +431,9 @@ class Model(object):
         and last_rest_key_case_insensitive_extractor)
 
         :param dict data: A dict using RestAPI structure
-        :param function key_extractors: A key extractor function.
         :param str content_type: JSON by default, set application/xml if XML.
         :returns: An instance of this model
         :raises: DeserializationError if something went wrong
-        :rtype: ModelType
         """
         deserializer = Deserializer(cls._infer_class_models())
         deserializer.key_extractors = (  # type: ignore
@@ -500,25 +453,21 @@ class Model(object):
             return {}
         result = dict(cls._subtype_map[key])
         for valuetype in cls._subtype_map[key].values():
-            result.update(objects[valuetype]._flatten_subtype(key, objects))  # pylint: disable=protected-access
+            result.update(objects[valuetype]._flatten_subtype(key, objects))
         return result
 
     @classmethod
     def _classify(cls, response, objects):
         """Check the class _subtype_map for any child classes.
         We want to ignore any inherited _subtype_maps.
-
-        :param dict response: The initial data
-        :param dict objects: The class objects
-        :returns: The class to be used
-        :rtype: class
+        Remove the polymorphic key from the initial data.
         """
         for subtype_key in cls.__dict__.get("_subtype_map", {}).keys():
             subtype_value = None
 
             if not isinstance(response, ET.Element):
                 rest_api_response_key = cls._get_rest_key_parts(subtype_key)[-1]
-                subtype_value = response.get(rest_api_response_key, None) or response.get(subtype_key, None)
+                subtype_value = response.pop(rest_api_response_key, None) or response.pop(subtype_key, None)
             else:
                 subtype_value = xml_key_extractor(subtype_key, cls._attribute_map[subtype_key], response)
             if subtype_value:
@@ -557,13 +506,11 @@ def _decode_attribute_map_key(key):
     inside the received data.
 
     :param str key: A key string from the generated code
-    :returns: The decoded key
-    :rtype: str
     """
     return key.replace("\\.", ".")
 
 
-class Serializer(object):  # pylint: disable=too-many-public-methods
+class Serializer(object):
     """Request object model serializer."""
 
     basic_types = {str: "str", int: "int", bool: "bool", float: "float"}
@@ -598,7 +545,7 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
         "multiple": lambda x, y: x % y != 0,
     }
 
-    def __init__(self, classes: Optional[Mapping[str, type]] = None) -> None:
+    def __init__(self, classes: Optional[Mapping[str, Type[ModelType]]] = None):
         self.serialize_type = {
             "iso-8601": Serializer.serialize_iso,
             "rfc-1123": Serializer.serialize_rfc,
@@ -614,20 +561,17 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
             "[]": self.serialize_iter,
             "{}": self.serialize_dict,
         }
-        self.dependencies: Dict[str, type] = dict(classes) if classes else {}
+        self.dependencies: Dict[str, Type[ModelType]] = dict(classes) if classes else {}
         self.key_transformer = full_restapi_key_transformer
         self.client_side_validation = True
 
-    def _serialize(  # pylint: disable=too-many-nested-blocks, too-many-branches, too-many-statements, too-many-locals
-        self, target_obj, data_type=None, **kwargs
-    ):
+    def _serialize(self, target_obj, data_type=None, **kwargs):
         """Serialize data into a string according to type.
 
-        :param object target_obj: The data to be serialized.
+        :param target_obj: The data to be serialized.
         :param str data_type: The type to be serialized from.
         :rtype: str, dict
         :raises: SerializationError if serialization fails.
-        :returns: The serialized data.
         """
         key_transformer = kwargs.get("key_transformer", self.key_transformer)
         keep_readonly = kwargs.get("keep_readonly", False)
@@ -653,14 +597,12 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
 
         serialized = {}
         if is_xml_model_serialization:
-            serialized = target_obj._create_xml_node()  # pylint: disable=protected-access
+            serialized = target_obj._create_xml_node()
         try:
-            attributes = target_obj._attribute_map  # pylint: disable=protected-access
+            attributes = target_obj._attribute_map
             for attr, attr_desc in attributes.items():
                 attr_name = attr
-                if not keep_readonly and target_obj._validation.get(  # pylint: disable=protected-access
-                    attr_name, {}
-                ).get("readonly", False):
+                if not keep_readonly and target_obj._validation.get(attr_name, {}).get("readonly", False):
                     continue
 
                 if attr_name == "additional_properties" and attr_desc["key"] == "":
@@ -696,8 +638,7 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
                         if isinstance(new_attr, list):
                             serialized.extend(new_attr)  # type: ignore
                         elif isinstance(new_attr, ET.Element):
-                            # If the down XML has no XML/Name,
-                            # we MUST replace the tag with the local tag. But keeping the namespaces.
+                            # If the down XML has no XML/Name, we MUST replace the tag with the local tag. But keeping the namespaces.
                             if "name" not in getattr(orig_attr, "_xml_map", {}):
                                 splitted_tag = new_attr.tag.split("}")
                                 if len(splitted_tag) == 2:  # Namespace
@@ -708,7 +649,7 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
                         else:  # That's a basic type
                             # Integrate namespace if necessary
                             local_node = _create_xml_node(xml_name, xml_prefix, xml_ns)
-                            local_node.text = str(new_attr)
+                            local_node.text = unicode_str(new_attr)
                             serialized.append(local_node)  # type: ignore
                     else:  # JSON
                         for k in reversed(keys):  # type: ignore
@@ -728,17 +669,17 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
         except (AttributeError, KeyError, TypeError) as err:
             msg = "Attribute {} in object {} cannot be serialized.\n{}".format(attr_name, class_name, str(target_obj))
             raise SerializationError(msg) from err
-        return serialized
+        else:
+            return serialized
 
     def body(self, data, data_type, **kwargs):
         """Serialize data intended for a request body.
 
-        :param object data: The data to be serialized.
+        :param data: The data to be serialized.
         :param str data_type: The type to be serialized from.
         :rtype: dict
         :raises: SerializationError if serialization fails.
         :raises: ValueError if data is None
-        :returns: The serialized request body
         """
 
         # Just in case this is a dict
@@ -767,7 +708,7 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
                         attribute_key_case_insensitive_extractor,
                         last_rest_key_case_insensitive_extractor,
                     ]
-                data = deserializer._deserialize(data_type, data)  # pylint: disable=protected-access
+                data = deserializer._deserialize(data_type, data)
             except DeserializationError as err:
                 raise SerializationError("Unable to build a model: " + str(err)) from err
 
@@ -776,11 +717,9 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
     def url(self, name, data, data_type, **kwargs):
         """Serialize data intended for a URL path.
 
-        :param str name: The name of the URL path parameter.
-        :param object data: The data to be serialized.
+        :param data: The data to be serialized.
         :param str data_type: The type to be serialized from.
         :rtype: str
-        :returns: The serialized URL path
         :raises: TypeError if serialization fails.
         :raises: ValueError if data is None
         """
@@ -794,20 +733,21 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
                 output = output.replace("{", quote("{")).replace("}", quote("}"))
             else:
                 output = quote(str(output), safe="")
-        except SerializationError as exc:
-            raise TypeError("{} must be type {}.".format(name, data_type)) from exc
-        return output
+        except SerializationError:
+            raise TypeError("{} must be type {}.".format(name, data_type))
+        else:
+            return output
 
     def query(self, name, data, data_type, **kwargs):
         """Serialize data intended for a URL query.
 
-        :param str name: The name of the query parameter.
-        :param object data: The data to be serialized.
+        :param data: The data to be serialized.
         :param str data_type: The type to be serialized from.
+        :keyword bool skip_quote: Whether to skip quote the serialized result.
+        Defaults to False.
         :rtype: str, list
         :raises: TypeError if serialization fails.
         :raises: ValueError if data is None
-        :returns: The serialized query parameter
         """
         try:
             # Treat the list aside, since we don't want to encode the div separator
@@ -824,20 +764,19 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
                 output = str(output)
             else:
                 output = quote(str(output), safe="")
-        except SerializationError as exc:
-            raise TypeError("{} must be type {}.".format(name, data_type)) from exc
-        return str(output)
+        except SerializationError:
+            raise TypeError("{} must be type {}.".format(name, data_type))
+        else:
+            return str(output)
 
     def header(self, name, data, data_type, **kwargs):
         """Serialize data intended for a request header.
 
-        :param str name: The name of the header.
-        :param object data: The data to be serialized.
+        :param data: The data to be serialized.
         :param str data_type: The type to be serialized from.
         :rtype: str
         :raises: TypeError if serialization fails.
         :raises: ValueError if data is None
-        :returns: The serialized header
         """
         try:
             if data_type in ["[str]"]:
@@ -846,20 +785,21 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
             output = self.serialize_data(data, data_type, **kwargs)
             if data_type == "bool":
                 output = json.dumps(output)
-        except SerializationError as exc:
-            raise TypeError("{} must be type {}.".format(name, data_type)) from exc
-        return str(output)
+        except SerializationError:
+            raise TypeError("{} must be type {}.".format(name, data_type))
+        else:
+            return str(output)
 
     def serialize_data(self, data, data_type, **kwargs):
         """Serialize generic data according to supplied data type.
 
-        :param object data: The data to be serialized.
+        :param data: The data to be serialized.
         :param str data_type: The type to be serialized from.
+        :param bool required: Whether it's essential that the data not be
+         empty or None
         :raises: AttributeError if required data is None.
         :raises: ValueError if data is None
         :raises: SerializationError if serialization fails.
-        :returns: The serialized data.
-        :rtype: str, int, float, bool, dict, list
         """
         if data is None:
             raise ValueError("No value for given attribute")
@@ -870,7 +810,7 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
             if data_type in self.basic_types.values():
                 return self.serialize_basic(data, data_type, **kwargs)
 
-            if data_type in self.serialize_type:
+            elif data_type in self.serialize_type:
                 return self.serialize_type[data_type](data, **kwargs)
 
             # If dependencies is empty, try with current data class
@@ -886,10 +826,11 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
         except (ValueError, TypeError) as err:
             msg = "Unable to serialize value: {!r} as type: {!r}."
             raise SerializationError(msg.format(data, data_type)) from err
-        return self._serialize(data, **kwargs)
+        else:
+            return self._serialize(data, **kwargs)
 
     @classmethod
-    def _get_custom_serializers(cls, data_type, **kwargs):  # pylint: disable=inconsistent-return-statements
+    def _get_custom_serializers(cls, data_type, **kwargs):
         custom_serializer = kwargs.get("basic_types_serializers", {}).get(data_type)
         if custom_serializer:
             return custom_serializer
@@ -905,26 +846,23 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
         - basic_types_serializers dict[str, callable] : If set, use the callable as serializer
         - is_xml bool : If set, use xml_basic_types_serializers
 
-        :param obj data: Object to be serialized.
+        :param data: Object to be serialized.
         :param str data_type: Type of object in the iterable.
-        :rtype: str, int, float, bool
-        :return: serialized object
         """
         custom_serializer = cls._get_custom_serializers(data_type, **kwargs)
         if custom_serializer:
             return custom_serializer(data)
         if data_type == "str":
             return cls.serialize_unicode(data)
-        return eval(data_type)(data)  # nosec # pylint: disable=eval-used
+        return eval(data_type)(data)  # nosec
 
     @classmethod
     def serialize_unicode(cls, data):
         """Special handling for serializing unicode strings in Py2.
         Encode to UTF-8 if unicode, otherwise handle as a str.
 
-        :param str data: Object to be serialized.
+        :param data: Object to be serialized.
         :rtype: str
-        :return: serialized object
         """
         try:  # If I received an enum, return its value
             return data.value
@@ -938,7 +876,8 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
                 return data
         except NameError:
             return str(data)
-        return str(data)
+        else:
+            return str(data)
 
     def serialize_iter(self, data, iter_type, div=None, **kwargs):
         """Serialize iterable.
@@ -948,13 +887,15 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
           serialization_ctxt['type'] should be same as data_type.
         - is_xml bool : If set, serialize as XML
 
-        :param list data: Object to be serialized.
+        :param list attr: Object to be serialized.
         :param str iter_type: Type of object in the iterable.
+        :param bool required: Whether the objects in the iterable must
+         not be None or empty.
         :param str div: If set, this str will be used to combine the elements
          in the iterable into a combined string. Default is 'None'.
+        :keyword bool do_quote: Whether to quote the serialized result of each iterable element.
         Defaults to False.
         :rtype: list, str
-        :return: serialized iterable
         """
         if isinstance(data, str):
             raise SerializationError("Refuse str type as a valid iter type.")
@@ -1009,8 +950,9 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
 
         :param dict attr: Object to be serialized.
         :param str dict_type: Type of object in the dictionary.
+        :param bool required: Whether the objects in the dictionary must
+         not be None or empty.
         :rtype: dict
-        :return: serialized dictionary
         """
         serialization_ctxt = kwargs.get("serialization_ctxt", {})
         serialized = {}
@@ -1034,7 +976,7 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
 
         return serialized
 
-    def serialize_object(self, attr, **kwargs):  # pylint: disable=too-many-return-statements
+    def serialize_object(self, attr, **kwargs):
         """Serialize a generic object.
         This will be handled as a dictionary. If object passed in is not
         a basic type (str, int, float, dict, list) it will simply be
@@ -1042,7 +984,6 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
 
         :param dict attr: Object to be serialized.
         :rtype: dict or str
-        :return: serialized object
         """
         if attr is None:
             return None
@@ -1053,7 +994,7 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
             return self.serialize_basic(attr, self.basic_types[obj_type], **kwargs)
         if obj_type is _long_type:
             return self.serialize_long(attr)
-        if obj_type is str:
+        if obj_type is unicode_str:
             return self.serialize_unicode(attr)
         if obj_type is datetime.datetime:
             return self.serialize_iso(attr)
@@ -1067,7 +1008,7 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
             return self.serialize_decimal(attr)
 
         # If it's a model or I know this dependency, serialize as a Model
-        if obj_type in self.dependencies.values() or isinstance(attr, Model):
+        elif obj_type in self.dependencies.values() or isinstance(attr, Model):
             return self._serialize(attr)
 
         if obj_type == dict:
@@ -1098,61 +1039,56 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
         try:
             enum_obj(result)  # type: ignore
             return result
-        except ValueError as exc:
+        except ValueError:
             for enum_value in enum_obj:  # type: ignore
                 if enum_value.value.lower() == str(attr).lower():
                     return enum_value.value
             error = "{!r} is not valid value for enum {!r}"
-            raise SerializationError(error.format(attr, enum_obj)) from exc
+            raise SerializationError(error.format(attr, enum_obj))
 
     @staticmethod
-    def serialize_bytearray(attr, **kwargs):  # pylint: disable=unused-argument
+    def serialize_bytearray(attr, **kwargs):
         """Serialize bytearray into base-64 string.
 
-        :param str attr: Object to be serialized.
+        :param attr: Object to be serialized.
         :rtype: str
-        :return: serialized base64
         """
         return b64encode(attr).decode()
 
     @staticmethod
-    def serialize_base64(attr, **kwargs):  # pylint: disable=unused-argument
+    def serialize_base64(attr, **kwargs):
         """Serialize str into base-64 string.
 
-        :param str attr: Object to be serialized.
+        :param attr: Object to be serialized.
         :rtype: str
-        :return: serialized base64
         """
         encoded = b64encode(attr).decode("ascii")
         return encoded.strip("=").replace("+", "-").replace("/", "_")
 
     @staticmethod
-    def serialize_decimal(attr, **kwargs):  # pylint: disable=unused-argument
+    def serialize_decimal(attr, **kwargs):
         """Serialize Decimal object to float.
 
-        :param decimal attr: Object to be serialized.
+        :param attr: Object to be serialized.
         :rtype: float
-        :return: serialized decimal
         """
         return float(attr)
 
     @staticmethod
-    def serialize_long(attr, **kwargs):  # pylint: disable=unused-argument
+    def serialize_long(attr, **kwargs):
         """Serialize long (Py2) or int (Py3).
 
-        :param int attr: Object to be serialized.
+        :param attr: Object to be serialized.
         :rtype: int/long
-        :return: serialized long
         """
         return _long_type(attr)
 
     @staticmethod
-    def serialize_date(attr, **kwargs):  # pylint: disable=unused-argument
+    def serialize_date(attr, **kwargs):
         """Serialize Date object into ISO-8601 formatted string.
 
         :param Date attr: Object to be serialized.
         :rtype: str
-        :return: serialized date
         """
         if isinstance(attr, str):
             attr = isodate.parse_date(attr)
@@ -1160,12 +1096,11 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
         return t
 
     @staticmethod
-    def serialize_time(attr, **kwargs):  # pylint: disable=unused-argument
+    def serialize_time(attr, **kwargs):
         """Serialize Time object into ISO-8601 formatted string.
 
         :param datetime.time attr: Object to be serialized.
         :rtype: str
-        :return: serialized time
         """
         if isinstance(attr, str):
             attr = isodate.parse_time(attr)
@@ -1175,32 +1110,30 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
         return t
 
     @staticmethod
-    def serialize_duration(attr, **kwargs):  # pylint: disable=unused-argument
+    def serialize_duration(attr, **kwargs):
         """Serialize TimeDelta object into ISO-8601 formatted string.
 
         :param TimeDelta attr: Object to be serialized.
         :rtype: str
-        :return: serialized duration
         """
         if isinstance(attr, str):
             attr = isodate.parse_duration(attr)
         return isodate.duration_isoformat(attr)
 
     @staticmethod
-    def serialize_rfc(attr, **kwargs):  # pylint: disable=unused-argument
+    def serialize_rfc(attr, **kwargs):
         """Serialize Datetime object into RFC-1123 formatted string.
 
         :param Datetime attr: Object to be serialized.
         :rtype: str
         :raises: TypeError if format invalid.
-        :return: serialized rfc
         """
         try:
             if not attr.tzinfo:
                 _LOGGER.warning("Datetime with no tzinfo will be considered UTC.")
             utc = attr.utctimetuple()
-        except AttributeError as exc:
-            raise TypeError("RFC1123 object must be valid Datetime object.") from exc
+        except AttributeError:
+            raise TypeError("RFC1123 object must be valid Datetime object.")
 
         return "{}, {:02} {} {:04} {:02}:{:02}:{:02} GMT".format(
             Serializer.days[utc.tm_wday],
@@ -1213,13 +1146,12 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
         )
 
     @staticmethod
-    def serialize_iso(attr, **kwargs):  # pylint: disable=unused-argument
+    def serialize_iso(attr, **kwargs):
         """Serialize Datetime object into ISO-8601 formatted string.
 
         :param Datetime attr: Object to be serialized.
         :rtype: str
         :raises: SerializationError if format invalid.
-        :return: serialized iso
         """
         if isinstance(attr, str):
             attr = isodate.parse_datetime(attr)
@@ -1245,14 +1177,13 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
             raise TypeError(msg) from err
 
     @staticmethod
-    def serialize_unix(attr, **kwargs):  # pylint: disable=unused-argument
+    def serialize_unix(attr, **kwargs):
         """Serialize Datetime object into IntTime format.
         This is represented as seconds.
 
         :param Datetime attr: Object to be serialized.
         :rtype: int
         :raises: SerializationError if format invalid
-        :return: serialied unix
         """
         if isinstance(attr, int):
             return attr
@@ -1260,11 +1191,11 @@ class Serializer(object):  # pylint: disable=too-many-public-methods
             if not attr.tzinfo:
                 _LOGGER.warning("Datetime with no tzinfo will be considered UTC.")
             return int(calendar.timegm(attr.utctimetuple()))
-        except AttributeError as exc:
-            raise TypeError("Unix time object must be valid Datetime object.") from exc
+        except AttributeError:
+            raise TypeError("Unix time object must be valid Datetime object.")
 
 
-def rest_key_extractor(attr, attr_desc, data):  # pylint: disable=unused-argument
+def rest_key_extractor(attr, attr_desc, data):
     key = attr_desc["key"]
     working_data = data
 
@@ -1285,9 +1216,7 @@ def rest_key_extractor(attr, attr_desc, data):  # pylint: disable=unused-argumen
     return working_data.get(key)
 
 
-def rest_key_case_insensitive_extractor(  # pylint: disable=unused-argument, inconsistent-return-statements
-    attr, attr_desc, data
-):
+def rest_key_case_insensitive_extractor(attr, attr_desc, data):
     key = attr_desc["key"]
     working_data = data
 
@@ -1308,29 +1237,17 @@ def rest_key_case_insensitive_extractor(  # pylint: disable=unused-argument, inc
         return attribute_key_case_insensitive_extractor(key, None, working_data)
 
 
-def last_rest_key_extractor(attr, attr_desc, data):  # pylint: disable=unused-argument
-    """Extract the attribute in "data" based on the last part of the JSON path key.
-
-    :param str attr: The attribute to extract
-    :param dict attr_desc: The attribute description
-    :param dict data: The data to extract from
-    :rtype: object
-    :returns: The extracted attribute
-    """
+def last_rest_key_extractor(attr, attr_desc, data):
+    """Extract the attribute in "data" based on the last part of the JSON path key."""
     key = attr_desc["key"]
     dict_keys = _FLATTEN.split(key)
     return attribute_key_extractor(dict_keys[-1], None, data)
 
 
-def last_rest_key_case_insensitive_extractor(attr, attr_desc, data):  # pylint: disable=unused-argument
+def last_rest_key_case_insensitive_extractor(attr, attr_desc, data):
     """Extract the attribute in "data" based on the last part of the JSON path key.
 
     This is the case insensitive version of "last_rest_key_extractor"
-    :param str attr: The attribute to extract
-    :param dict attr_desc: The attribute description
-    :param dict data: The data to extract from
-    :rtype: object
-    :returns: The extracted attribute
     """
     key = attr_desc["key"]
     dict_keys = _FLATTEN.split(key)
@@ -1367,7 +1284,7 @@ def _extract_name_from_internal_type(internal_type):
     return xml_name
 
 
-def xml_key_extractor(attr, attr_desc, data):  # pylint: disable=unused-argument,too-many-return-statements
+def xml_key_extractor(attr, attr_desc, data):
     if isinstance(data, dict):
         return None
 
@@ -1419,21 +1336,22 @@ def xml_key_extractor(attr, attr_desc, data):  # pylint: disable=unused-argument
         if is_iter_type:
             if is_wrapped:
                 return None  # is_wrapped no node, we want None
-            return []  # not wrapped, assume empty list
+            else:
+                return []  # not wrapped, assume empty list
         return None  # Assume it's not there, maybe an optional node.
 
     # If is_iter_type and not wrapped, return all found children
     if is_iter_type:
         if not is_wrapped:
             return children
-        # Iter and wrapped, should have found one node only (the wrap one)
-        if len(children) != 1:
-            raise DeserializationError(
-                "Tried to deserialize an array not wrapped, and found several nodes '{}'. Maybe you should declare this array as wrapped?".format(  # pylint: disable=line-too-long
-                    xml_name
+        else:  # Iter and wrapped, should have found one node only (the wrap one)
+            if len(children) != 1:
+                raise DeserializationError(
+                    "Tried to deserialize an array not wrapped, and found several nodes '{}'. Maybe you should declare this array as wrapped?".format(
+                        xml_name
+                    )
                 )
-            )
-        return list(children[0])  # Might be empty list and that's ok.
+            return list(children[0])  # Might be empty list and that's ok.
 
     # Here it's not a itertype, we should have found one element only or empty
     if len(children) > 1:
@@ -1450,9 +1368,9 @@ class Deserializer(object):
 
     basic_types = {str: "str", int: "int", bool: "bool", float: "float"}
 
-    valid_date = re.compile(r"\d{4}[-]\d{2}[-]\d{2}T\d{2}:\d{2}:\d{2}\.?\d*Z?[-+]?[\d{2}]?:?[\d{2}]?")
+    valid_date = re.compile(r"\d{4}[-]\d{2}[-]\d{2}T\d{2}:\d{2}:\d{2}" r"\.?\d*Z?[-+]?[\d{2}]?:?[\d{2}]?")
 
-    def __init__(self, classes: Optional[Mapping[str, type]] = None) -> None:
+    def __init__(self, classes: Optional[Mapping[str, Type[ModelType]]] = None):
         self.deserialize_type = {
             "iso-8601": Deserializer.deserialize_iso,
             "rfc-1123": Deserializer.deserialize_rfc,
@@ -1472,7 +1390,7 @@ class Deserializer(object):
             "duration": (isodate.Duration, datetime.timedelta),
             "iso-8601": (datetime.datetime),
         }
-        self.dependencies: Dict[str, type] = dict(classes) if classes else {}
+        self.dependencies: Dict[str, Type[ModelType]] = dict(classes) if classes else {}
         self.key_extractors = [rest_key_extractor, xml_key_extractor]
         # Additional properties only works if the "rest_key_extractor" is used to
         # extract the keys. Making it to work whatever the key extractor is too much
@@ -1490,12 +1408,11 @@ class Deserializer(object):
         :param str content_type: Swagger "produces" if available.
         :raises: DeserializationError if deserialization fails.
         :return: Deserialized object.
-        :rtype: object
         """
         data = self._unpack_content(response_data, content_type)
         return self._deserialize(target_obj, data)
 
-    def _deserialize(self, target_obj, data):  # pylint: disable=inconsistent-return-statements
+    def _deserialize(self, target_obj, data):
         """Call the deserializer on a model.
 
         Data needs to be already deserialized as JSON or XML ElementTree
@@ -1504,13 +1421,12 @@ class Deserializer(object):
         :param object data: Object to deserialize.
         :raises: DeserializationError if deserialization fails.
         :return: Deserialized object.
-        :rtype: object
         """
         # This is already a model, go recursive just in case
         if hasattr(data, "_attribute_map"):
             constants = [name for name, config in getattr(data, "_validation", {}).items() if config.get("constant")]
             try:
-                for attr, mapconfig in data._attribute_map.items():  # pylint: disable=protected-access
+                for attr, mapconfig in data._attribute_map.items():
                     if attr in constants:
                         continue
                     value = getattr(data, attr)
@@ -1527,15 +1443,15 @@ class Deserializer(object):
 
         response, class_name = self._classify_target(target_obj, data)
 
-        if isinstance(response, str):
+        if isinstance(response, basestring):
             return self.deserialize_data(data, response)
-        if isinstance(response, type) and issubclass(response, Enum):
+        elif isinstance(response, type) and issubclass(response, Enum):
             return self.deserialize_enum(data, response)
 
-        if data is None or data is CoreNull:
+        if data is None:
             return data
         try:
-            attributes = response._attribute_map  # type: ignore # pylint: disable=protected-access
+            attributes = response._attribute_map  # type: ignore
             d_attrs = {}
             for attr, attr_desc in attributes.items():
                 # Check empty string. If it's not empty, someone has a real "additionalProperties"...
@@ -1565,8 +1481,9 @@ class Deserializer(object):
         except (AttributeError, TypeError, KeyError) as err:
             msg = "Unable to deserialize to object: " + class_name  # type: ignore
             raise DeserializationError(msg) from err
-        additional_properties = self._build_additional_properties(attributes, data)
-        return self._instantiate_model(response, d_attrs, additional_properties)
+        else:
+            additional_properties = self._build_additional_properties(attributes, data)
+            return self._instantiate_model(response, d_attrs, additional_properties)
 
     def _build_additional_properties(self, attribute_map, data):
         if not self.additional_properties_detection:
@@ -1593,20 +1510,18 @@ class Deserializer(object):
 
         :param str target: The target object type to deserialize to.
         :param str/dict data: The response data to deserialize.
-        :return: The classified target object and its class name.
-        :rtype: tuple
         """
         if target is None:
             return None, None
 
-        if isinstance(target, str):
+        if isinstance(target, basestring):
             try:
                 target = self.dependencies[target]
             except KeyError:
                 return target, target
 
         try:
-            target = target._classify(data, self.dependencies)  # type: ignore # pylint: disable=protected-access
+            target = target._classify(data, self.dependencies)
         except AttributeError:
             pass  # Target is not a Model, no classify
         return target, target.__class__.__name__  # type: ignore
@@ -1621,12 +1536,10 @@ class Deserializer(object):
         :param str target_obj: The target object type to deserialize to.
         :param str/dict data: The response data to deserialize.
         :param str content_type: Swagger "produces" if available.
-        :return: Deserialized object.
-        :rtype: object
         """
         try:
             return self(target_obj, data, content_type=content_type)
-        except:  # pylint: disable=bare-except
+        except:
             _LOGGER.debug(
                 "Ran into a deserialization error. Ignoring since this is failsafe deserialization", exc_info=True
             )
@@ -1644,12 +1557,10 @@ class Deserializer(object):
 
         If raw_data is something else, bypass all logic and return it directly.
 
-        :param obj raw_data: Data to be processed.
-        :param str content_type: How to parse if raw_data is a string/bytes.
+        :param raw_data: Data to be processed.
+        :param content_type: How to parse if raw_data is a string/bytes.
         :raises JSONDecodeError: If JSON is requested and parsing is impossible.
         :raises UnicodeDecodeError: If bytes is not UTF8
-        :rtype: object
-        :return: Unpacked content.
         """
         # Assume this is enough to detect a Pipeline Response without importing it
         context = getattr(raw_data, "context", {})
@@ -1666,42 +1577,31 @@ class Deserializer(object):
         if hasattr(raw_data, "_content_consumed"):
             return RawDeserializer.deserialize_from_http_generics(raw_data.text, raw_data.headers)
 
-        if isinstance(raw_data, (str, bytes)) or hasattr(raw_data, "read"):
+        if isinstance(raw_data, (basestring, bytes)) or hasattr(raw_data, "read"):
             return RawDeserializer.deserialize_from_text(raw_data, content_type)  # type: ignore
         return raw_data
 
     def _instantiate_model(self, response, attrs, additional_properties=None):
         """Instantiate a response model passing in deserialized args.
 
-        :param Response response: The response model class.
-        :param dict attrs: The deserialized response attributes.
-        :param dict additional_properties: Additional properties to be set.
-        :rtype: Response
-        :return: The instantiated response model.
+        :param response: The response model class.
+        :param d_attrs: The deserialized response attributes.
         """
         if callable(response):
             subtype = getattr(response, "_subtype_map", {})
             try:
-                readonly = [
-                    k
-                    for k, v in response._validation.items()  # pylint: disable=protected-access  # type: ignore
-                    if v.get("readonly")
-                ]
-                const = [
-                    k
-                    for k, v in response._validation.items()  # pylint: disable=protected-access  # type: ignore
-                    if v.get("constant")
-                ]
+                readonly = [k for k, v in response._validation.items() if v.get("readonly")]
+                const = [k for k, v in response._validation.items() if v.get("constant")]
                 kwargs = {k: v for k, v in attrs.items() if k not in subtype and k not in readonly + const}
                 response_obj = response(**kwargs)
                 for attr in readonly:
                     setattr(response_obj, attr, attrs.get(attr))
                 if additional_properties:
-                    response_obj.additional_properties = additional_properties  # type: ignore
+                    response_obj.additional_properties = additional_properties
                 return response_obj
             except TypeError as err:
                 msg = "Unable to deserialize {} into model {}. ".format(kwargs, response)  # type: ignore
-                raise DeserializationError(msg + str(err)) from err
+                raise DeserializationError(msg + str(err))
         else:
             try:
                 for attr, value in attrs.items():
@@ -1710,16 +1610,15 @@ class Deserializer(object):
             except Exception as exp:
                 msg = "Unable to populate response model. "
                 msg += "Type: {}, Error: {}".format(type(response), exp)
-                raise DeserializationError(msg) from exp
+                raise DeserializationError(msg)
 
-    def deserialize_data(self, data, data_type):  # pylint: disable=too-many-return-statements
+    def deserialize_data(self, data, data_type):
         """Process data for deserialization according to data type.
 
         :param str data: The response string to be deserialized.
         :param str data_type: The type to deserialize to.
         :raises: DeserializationError if deserialization fails.
         :return: Deserialized object.
-        :rtype: object
         """
         if data is None:
             return data
@@ -1733,11 +1632,7 @@ class Deserializer(object):
                 if isinstance(data, self.deserialize_expected_types.get(data_type, tuple())):
                     return data
 
-                is_a_text_parsing_type = lambda x: x not in [  # pylint: disable=unnecessary-lambda-assignment
-                    "object",
-                    "[]",
-                    r"{}",
-                ]
+                is_a_text_parsing_type = lambda x: x not in ["object", "[]", r"{}"]
                 if isinstance(data, ET.Element) and is_a_text_parsing_type(data_type) and not data.text:
                     return None
                 data_val = self.deserialize_type[data_type](data)
@@ -1757,14 +1652,14 @@ class Deserializer(object):
             msg = "Unable to deserialize response data."
             msg += " Data: {}, {}".format(data, data_type)
             raise DeserializationError(msg) from err
-        return self._deserialize(obj_type, data)
+        else:
+            return self._deserialize(obj_type, data)
 
     def deserialize_iter(self, attr, iter_type):
         """Deserialize an iterable.
 
         :param list attr: Iterable to be deserialized.
         :param str iter_type: The type of object in the iterable.
-        :return: Deserialized iterable.
         :rtype: list
         """
         if attr is None:
@@ -1781,7 +1676,6 @@ class Deserializer(object):
         :param dict/list attr: Dictionary to be deserialized. Also accepts
          a list of key, value pairs.
         :param str dict_type: The object type of the items in the dictionary.
-        :return: Deserialized dictionary.
         :rtype: dict
         """
         if isinstance(attr, list):
@@ -1792,12 +1686,11 @@ class Deserializer(object):
             attr = {el.tag: el.text for el in attr}
         return {k: self.deserialize_data(v, dict_type) for k, v in attr.items()}
 
-    def deserialize_object(self, attr, **kwargs):  # pylint: disable=too-many-return-statements
+    def deserialize_object(self, attr, **kwargs):
         """Deserialize a generic object.
         This will be handled as a dictionary.
 
         :param dict attr: Dictionary to be deserialized.
-        :return: Deserialized object.
         :rtype: dict
         :raises: TypeError if non-builtin datatype encountered.
         """
@@ -1806,7 +1699,7 @@ class Deserializer(object):
         if isinstance(attr, ET.Element):
             # Do no recurse on XML, just return the tree as-is
             return attr
-        if isinstance(attr, str):
+        if isinstance(attr, basestring):
             return self.deserialize_basic(attr, "str")
         obj_type = type(attr)
         if obj_type in self.basic_types:
@@ -1832,10 +1725,11 @@ class Deserializer(object):
                     pass
             return deserialized
 
-        error = "Cannot deserialize generic object with type: "
-        raise TypeError(error + str(obj_type))
+        else:
+            error = "Cannot deserialize generic object with type: "
+            raise TypeError(error + str(obj_type))
 
-    def deserialize_basic(self, attr, data_type):  # pylint: disable=too-many-return-statements
+    def deserialize_basic(self, attr, data_type):
         """Deserialize basic builtin data type from string.
         Will attempt to convert to str, int, float and bool.
         This function will also accept '1', '0', 'true' and 'false' as
@@ -1843,7 +1737,6 @@ class Deserializer(object):
 
         :param str attr: response string to be deserialized.
         :param str data_type: deserialization data type.
-        :return: Deserialized basic type.
         :rtype: str, int, float or bool
         :raises: TypeError if string format is not valid.
         """
@@ -1855,23 +1748,24 @@ class Deserializer(object):
                 if data_type == "str":
                     # None or '', node <a/> is empty string.
                     return ""
-                # None or '', node <a/> with a strong type is None.
-                # Don't try to model "empty bool" or "empty int"
-                return None
+                else:
+                    # None or '', node <a/> with a strong type is None.
+                    # Don't try to model "empty bool" or "empty int"
+                    return None
 
         if data_type == "bool":
             if attr in [True, False, 1, 0]:
                 return bool(attr)
-            if isinstance(attr, str):
+            elif isinstance(attr, basestring):
                 if attr.lower() in ["true", "1"]:
                     return True
-                if attr.lower() in ["false", "0"]:
+                elif attr.lower() in ["false", "0"]:
                     return False
             raise TypeError("Invalid boolean value: {}".format(attr))
 
         if data_type == "str":
             return self.deserialize_unicode(attr)
-        return eval(data_type)(attr)  # nosec # pylint: disable=eval-used
+        return eval(data_type)(attr)  # nosec
 
     @staticmethod
     def deserialize_unicode(data):
@@ -1879,7 +1773,6 @@ class Deserializer(object):
         as a string.
 
         :param str data: response string to be deserialized.
-        :return: Deserialized string.
         :rtype: str or unicode
         """
         # We might be here because we have an enum modeled as string,
@@ -1893,7 +1786,8 @@ class Deserializer(object):
                 return data
         except NameError:
             return str(data)
-        return str(data)
+        else:
+            return str(data)
 
     @staticmethod
     def deserialize_enum(data, enum_obj):
@@ -1905,7 +1799,6 @@ class Deserializer(object):
         :param str data: Response string to be deserialized. If this value is
          None or invalid it will be returned as-is.
         :param Enum enum_obj: Enum object to deserialize to.
-        :return: Deserialized enum object.
         :rtype: Enum
         """
         if isinstance(data, enum_obj) or data is None:
@@ -1916,9 +1809,9 @@ class Deserializer(object):
             # Workaround. We might consider remove it in the future.
             try:
                 return list(enum_obj.__members__.values())[data]
-            except IndexError as exc:
+            except IndexError:
                 error = "{!r} is not a valid index for enum {!r}"
-                raise DeserializationError(error.format(data, enum_obj)) from exc
+                raise DeserializationError(error.format(data, enum_obj))
         try:
             return enum_obj(str(data))
         except ValueError:
@@ -1934,7 +1827,6 @@ class Deserializer(object):
         """Deserialize string into bytearray.
 
         :param str attr: response string to be deserialized.
-        :return: Deserialized bytearray
         :rtype: bytearray
         :raises: TypeError if string format invalid.
         """
@@ -1947,7 +1839,6 @@ class Deserializer(object):
         """Deserialize base64 encoded string into string.
 
         :param str attr: response string to be deserialized.
-        :return: Deserialized base64 string
         :rtype: bytearray
         :raises: TypeError if string format invalid.
         """
@@ -1963,9 +1854,8 @@ class Deserializer(object):
         """Deserialize string into Decimal object.
 
         :param str attr: response string to be deserialized.
-        :return: Deserialized decimal
+        :rtype: Decimal
         :raises: DeserializationError if string format invalid.
-        :rtype: decimal
         """
         if isinstance(attr, ET.Element):
             attr = attr.text
@@ -1980,7 +1870,6 @@ class Deserializer(object):
         """Deserialize string into long (Py2) or int (Py3).
 
         :param str attr: response string to be deserialized.
-        :return: Deserialized int
         :rtype: long or int
         :raises: ValueError if string format invalid.
         """
@@ -1993,7 +1882,6 @@ class Deserializer(object):
         """Deserialize ISO-8601 formatted string into TimeDelta object.
 
         :param str attr: response string to be deserialized.
-        :return: Deserialized duration
         :rtype: TimeDelta
         :raises: DeserializationError if string format invalid.
         """
@@ -2004,14 +1892,14 @@ class Deserializer(object):
         except (ValueError, OverflowError, AttributeError) as err:
             msg = "Cannot deserialize duration object."
             raise DeserializationError(msg) from err
-        return duration
+        else:
+            return duration
 
     @staticmethod
     def deserialize_date(attr):
         """Deserialize ISO-8601 formatted string into Date object.
 
         :param str attr: response string to be deserialized.
-        :return: Deserialized date
         :rtype: Date
         :raises: DeserializationError if string format invalid.
         """
@@ -2027,7 +1915,6 @@ class Deserializer(object):
         """Deserialize ISO-8601 formatted string into time object.
 
         :param str attr: response string to be deserialized.
-        :return: Deserialized time
         :rtype: datetime.time
         :raises: DeserializationError if string format invalid.
         """
@@ -2042,7 +1929,6 @@ class Deserializer(object):
         """Deserialize RFC-1123 formatted string into Datetime object.
 
         :param str attr: response string to be deserialized.
-        :return: Deserialized RFC datetime
         :rtype: Datetime
         :raises: DeserializationError if string format invalid.
         """
@@ -2058,14 +1944,14 @@ class Deserializer(object):
         except ValueError as err:
             msg = "Cannot deserialize to rfc datetime object."
             raise DeserializationError(msg) from err
-        return date_obj
+        else:
+            return date_obj
 
     @staticmethod
     def deserialize_iso(attr):
         """Deserialize ISO-8601 formatted string into Datetime object.
 
         :param str attr: response string to be deserialized.
-        :return: Deserialized ISO datetime
         :rtype: Datetime
         :raises: DeserializationError if string format invalid.
         """
@@ -2095,7 +1981,8 @@ class Deserializer(object):
         except (ValueError, OverflowError, AttributeError) as err:
             msg = "Cannot deserialize datetime object."
             raise DeserializationError(msg) from err
-        return date_obj
+        else:
+            return date_obj
 
     @staticmethod
     def deserialize_unix(attr):
@@ -2103,7 +1990,6 @@ class Deserializer(object):
         This is represented as seconds.
 
         :param int attr: Object to be serialized.
-        :return: Deserialized datetime
         :rtype: Datetime
         :raises: DeserializationError if format invalid
         """
@@ -2115,4 +2001,5 @@ class Deserializer(object):
         except ValueError as err:
             msg = "Cannot deserialize to unix datetime object."
             raise DeserializationError(msg) from err
-        return date_obj
+        else:
+            return date_obj