pydrive2: use isinstance() instead of type() (#291)

Looks like this was an anti pattern that stuck around from some early
days.

Fixes iterative/dvc-gdrive#31

Author: efiop

pydrive2/files.py

@@ -285,7 +285,7 @@ def GetContentString(
         """
         if (
             self.content is None
-            or type(self.content) is not io.BytesIO
+            or not isinstance(self.content, io.BytesIO)
             or self.has_bom == remove_bom
         ):
             self.FetchContent(mimetype, remove_bom)

pydrive2/settings.py

@@ -167,15 +167,15 @@ def _ValidateSettingsElement(data, struct, key):
         else:
             data[key] = default
     # If data exists, Check type of the data
-    elif type(value) is not data_type:
+    elif not isinstance(value, data_type):
         raise InvalidConfigError(f"Setting {key} should be type {data_type}")
     # If type of this data is dict, check if structure of the data is valid.
     if data_type is dict:
         _ValidateSettingsStruct(data[key], struct[key]["struct"])
     # If type of this data is list, check if all values in the list is valid.
     elif data_type is list:
         for element in data[key]:
-            if type(element) is not struct[key]["struct"]:
+            if not isinstance(element, struct[key]["struct"]):
                 raise InvalidConfigError(
                     "Setting %s should be list of %s"
                     % (key, struct[key]["struct"])