Fix linting error on type checks

pypuppetdb/QueryBuilder.py

@@ -71,10 +71,10 @@ def __init__(self, operator):
         self.operations = []
 
     def add(self, query):
-        if type(query) == list:
+        if isinstance(query, list):
             for i in query:
                 self.add(i)
-        elif type(query) == str:
+        elif isinstance(query, str):
             self.operations.append(json.loads(query))
         elif isinstance(query, (BinaryOperator, InOperator, BooleanOperator)):
             self.operations.append(query.json_data())
@@ -652,7 +652,7 @@ class NullOperator(BinaryOperator):
     """
 
     def __init__(self, field, value):
-        if type(value) != bool:
+        if not isinstance(value, bool):
             raise APIError("NullOperator value must be boolean")
 
         super().__init__("null?", field, value)

pypuppetdb/api/query.py

@@ -54,7 +54,7 @@ def nodes(self, unreported=2, with_status=False, with_event_numbers=True, **kwar
         # If we happen to only get one node back it
         # won't be inside a list so iterating over it
         # goes boom. Therefor we wrap a list around it.
-        if type(nodes) == dict:
+        if isinstance(nodes, dict):
             nodes = [
                 nodes,
             ]
@@ -239,7 +239,7 @@ def catalogs(self, **kwargs):
 
         catalogs = self._query("catalogs", **kwargs)
 
-        if type(catalogs) == dict:
+        if isinstance(catalogs, dict):
             catalogs = [
                 catalogs,
             ]

tests/test_api_pql.py

@@ -56,7 +56,7 @@ def test_pql_casting(self, api):
 
         nodes = list(api.pql(pql_query))
 
-        assert type(nodes[0]) == Node
+        assert isinstance(nodes[0], Node)
         assert nodes[0].name == "greenserver.vm"
 
         httpretty.disable()