Merge pull request #238 from dhermes/large-style-changes

Large number of style-related changes to pass pylint

Author: dhermes

gcloud/__init__.py

@@ -1,2 +1,4 @@
 """GCloud API access in idiomatic Python."""
+
+
 __version__ = '0.02.2'

gcloud/connection.py

@@ -1,4 +1,5 @@
 """ Shared implementation of connections to API servers."""
+
 from pkg_resources import get_distribution
 
 import httplib2
@@ -28,6 +29,7 @@ def __init__(self, credentials=None):
         :type credentials: :class:`oauth2client.client.OAuth2Credentials`
         :param credentials: The OAuth2 Credentials to use for this connection.
         """
+        self._http = None
         self._credentials = credentials
 
     @property
@@ -45,7 +47,7 @@ def http(self):
         :rtype: :class:`httplib2.Http`
         :returns: A Http object used to transport data.
         """
-        if not hasattr(self, '_http'):
+        if self._http is None:
             self._http = httplib2.Http()
             if self._credentials:
                 self._http = self._credentials.authorize(self._http)

gcloud/datastore/_helpers.py

@@ -3,7 +3,7 @@
 These functions are *not* part of the API.
 """
 import calendar
-from datetime import datetime, timedelta
+import datetime
 
 from google.protobuf.internal.type_checkers import Int64ValueChecker
 import pytz
@@ -43,7 +43,7 @@ def _get_protobuf_attribute_and_value(val):
     :returns: A tuple of the attribute name and proper value type.
     """
 
-    if isinstance(val, datetime):
+    if isinstance(val, datetime.datetime):
         name = 'timestamp_microseconds'
         # If the datetime is naive (no timezone), consider that it was
         # intended to be UTC and replace the tzinfo to that effect.
@@ -91,8 +91,8 @@ def _get_value_from_value_pb(value_pb):
     result = None
     if value_pb.HasField('timestamp_microseconds_value'):
         microseconds = value_pb.timestamp_microseconds_value
-        naive = (datetime.utcfromtimestamp(0) +
-                 timedelta(microseconds=microseconds))
+        naive = (datetime.datetime.utcfromtimestamp(0) +
+                 datetime.timedelta(microseconds=microseconds))
         result = naive.replace(tzinfo=pytz.utc)
 
     elif value_pb.HasField('key_value'):
@@ -163,9 +163,9 @@ def _set_protobuf_value(value_pb, val):
         key = val.key()
         if key is not None:
             e_pb.key.CopyFrom(key.to_protobuf())
-        for k, v in val.items():
+        for item_key, value in val.iteritems():
             p_pb = e_pb.property.add()
-            p_pb.name = k
-            _set_protobuf_value(p_pb.value, v)
+            p_pb.name = item_key
+            _set_protobuf_value(p_pb.value, value)
     else:  # scalar, just assign
         setattr(value_pb, attr, val)

gcloud/datastore/connection.py

@@ -1,4 +1,5 @@
 """Connections to gcloud datastore API servers."""
+
 from gcloud import connection
 from gcloud.datastore import datastore_v1_pb2 as datastore_pb
 from gcloud.datastore import _helpers
@@ -23,7 +24,7 @@ class Connection(connection.Connection):
     """A template for the URL of a particular API call."""
 
     def __init__(self, credentials=None):
-        self._credentials = credentials
+        super(Connection, self).__init__(credentials=credentials)
         self._current_transaction = None
 
     def _request(self, dataset_id, method, data):
@@ -240,11 +241,12 @@ def run_query(self, dataset_id, query_pb, namespace=None):
         request.query.CopyFrom(query_pb)
         response = self._rpc(dataset_id, 'runQuery', request,
                              datastore_pb.RunQueryResponse)
-        return ([e.entity for e in response.batch.entity_result],
-                response.batch.end_cursor,
-                response.batch.more_results,
-                response.batch.skipped_results,
-                )
+        return (
+            [e.entity for e in response.batch.entity_result],
+            response.batch.end_cursor,
+            response.batch.more_results,
+            response.batch.skipped_results,
+        )
 
     def lookup(self, dataset_id, key_pbs):
         """Lookup keys from a dataset in the Cloud Datastore.

gcloud/datastore/dataset.py

@@ -70,6 +70,7 @@ def query(self, *args, **kwargs):
         :rtype: :class:`gcloud.datastore.query.Query`
         :returns: a new Query instance, bound to this dataset.
         """
+        # This import is here to avoid circular references.
         from gcloud.datastore.query import Query
         kwargs['dataset'] = self
         return Query(*args, **kwargs)
@@ -83,6 +84,7 @@ def entity(self, kind):
         :rtype: :class:`gcloud.datastore.entity.Entity`
         :returns: a new Entity instance, bound to this dataset.
         """
+        # This import is here to avoid circular references.
         from gcloud.datastore.entity import Entity
         return Entity(dataset=self, kind=kind)
 
@@ -96,6 +98,7 @@ def transaction(self, *args, **kwargs):
         :rtype: :class:`gcloud.datastore.transaction.Transaction`
         :returns: a new Transaction instance, bound to this dataset.
         """
+        # This import is here to avoid circular references.
         from gcloud.datastore.transaction import Transaction
         kwargs['dataset'] = self
         return Transaction(*args, **kwargs)

gcloud/datastore/entity.py

@@ -23,7 +23,7 @@ class NoKey(RuntimeError):
     """Exception raised by Entity methods which require a key."""
 
 
-class Entity(dict):  # pylint: disable=too-many-public-methods
+class Entity(dict):
     """:type dataset: :class:`gcloud.datastore.dataset.Dataset`
     :param dataset: The dataset in which this entity belongs.
 
@@ -95,7 +95,9 @@ def key(self, key=None):
         :type key: :class:`glcouddatastore.key.Key`
         :param key: The key you want to set on the entity.
 
-        :returns: Either the current key or the :class:`Entity`.
+        :rtype: :class:`gcloud.datastore.key.Key` or :class:`Entity`.
+        :returns: Either the current key (on get) or the current
+                  object (on set).
 
         >>> entity.key(my_other_key)  # This returns the original entity.
         <Entity[{'kind': 'OtherKeyKind', 'id': 1234}] {'property': 'value'}>
@@ -141,7 +143,7 @@ def from_key(cls, key):
         return cls().key(key)
 
     @classmethod
-    def from_protobuf(cls, pb, dataset=None):  # pylint: disable=invalid-name
+    def from_protobuf(cls, pb, dataset=None):
         """Factory method for creating an entity based on a protobuf.
 
         The protobuf should be one returned from the Cloud Datastore

gcloud/datastore/key.py

@@ -159,7 +159,7 @@ def is_partial(self):
         :returns: True if the last element of the key's path does not have
                   an 'id' or a 'name'.
         """
-        return (self.id_or_name() is None)
+        return self.id_or_name() is None
 
     def dataset(self, dataset=None):
         """Dataset setter / getter.
@@ -231,19 +231,19 @@ def kind(self, kind=None):
         elif self.path():
             return self._path[-1]['kind']
 
-    def id(self, id=None):
+    def id(self, id_to_set=None):
         """ID setter / getter.  Based on the last element of path.
 
-        :type kind: :class:`str`
-        :param kind: The new kind for the key.
+        :type id_to_set: :class:`int`
+        :param id_to_set: The new ID for the key.
 
         :rtype: :class:`Key` (for setter); or :class:`int` (for getter)
         :returns: a new key, cloned from self., with the given id (setter);
-                 or self's id (getter).
+                  or self's id (getter).
         """
-        if id:
+        if id_to_set:
             clone = self._clone()
-            clone._path[-1]['id'] = id
+            clone._path[-1]['id'] = id_to_set
             return clone
         elif self.path():
             return self._path[-1].get('id')

gcloud/datastore/query.py

@@ -315,11 +315,9 @@ def fetch(self, limit=None):
         if limit:
             clone = self.limit(limit)
 
-        (entity_pbs,
-         end_cursor,
-         more_results,
-         skipped_results) = self.dataset().connection().run_query(
+        query_results = self.dataset().connection().run_query(
             query_pb=clone.to_protobuf(), dataset_id=self.dataset().id())
+        entity_pbs, end_cursor = query_results[:2]
 
         self._cursor = end_cursor
         return [Entity.from_protobuf(entity, dataset=self.dataset())
@@ -379,14 +377,14 @@ def order(self, *properties):
         """
         clone = self._clone()
 
-        for p in properties:
+        for prop in properties:
             property_order = clone._pb.order.add()
 
-            if p.startswith('-'):
-                property_order.property.name = p[1:]
+            if prop.startswith('-'):
+                property_order.property.name = prop[1:]
                 property_order.direction = property_order.DESCENDING
             else:
-                property_order.property.name = p
+                property_order.property.name = prop
                 property_order.direction = property_order.ASCENDING
 
         return clone

gcloud/datastore/test___init__.py

@@ -25,11 +25,12 @@ def test_it(self):
                 found = self._callFUT(CLIENT_EMAIL, f.name)
         self.assertTrue(isinstance(found, Connection))
         self.assertTrue(found._credentials is client._signed)
-        self.assertEqual(client._called_with,
-                         {'service_account_name': CLIENT_EMAIL,
-                          'private_key': PRIVATE_KEY,
-                          'scope': SCOPE,
-                          })
+        expected_called_with = {
+            'service_account_name': CLIENT_EMAIL,
+            'private_key': PRIVATE_KEY,
+            'scope': SCOPE,
+        }
+        self.assertEqual(client._called_with, expected_called_with)
 
 
 class Test_get_dataset(unittest2.TestCase):
@@ -59,8 +60,9 @@ def test_it(self):
         self.assertTrue(isinstance(found, Dataset))
         self.assertTrue(isinstance(found.connection(), Connection))
         self.assertEqual(found.id(), DATASET_ID)
-        self.assertEqual(client._called_with,
-                         {'service_account_name': CLIENT_EMAIL,
-                          'private_key': PRIVATE_KEY,
-                          'scope': SCOPE,
-                          })
+        expected_called_with = {
+            'service_account_name': CLIENT_EMAIL,
+            'private_key': PRIVATE_KEY,
+            'scope': SCOPE,
+        }
+        self.assertEqual(client._called_with, expected_called_with)

gcloud/datastore/test__helpers.py

@@ -182,7 +182,7 @@ def test_unknown(self):
         from gcloud.datastore.datastore_v1_pb2 import Value
 
         pb = Value()
-        self.assertEqual(self._callFUT(pb), None)  # XXX desirable?
+        self.assertEqual(self._callFUT(pb), None)
 
 
 class Test__get_value_from_property_pb(unittest2.TestCase):