fix style

liyanhui1228 · liyanhui1228 · commit d2d96287f064 · 2017-05-16T13:44:13.000-07:00
diff --git a/logging/google/cloud/logging/handlers/app_engine.py b/logging/google/cloud/logging/handlers/app_engine.py
@@ -23,49 +23,34 @@
 from google.cloud.logging.handlers.transports import BackgroundThreadTransport
 from google.cloud.logging.resource import Resource
 
+_DEFAULT_GAE_LOGGER_NAME = 'app'
+
 _GAE_PROJECT_ENV = 'GCLOUD_PROJECT'
 _GAE_SERVICE_ENV = 'GAE_SERVICE'
 _GAE_VERSION_ENV = 'GAE_VERSION'
 
 
 class AppEngineHandler(CloudLoggingHandler):
-    """A handler that directly makes Stackdriver logging API calls.
-
-    This handler can be used to route Python standard logging messages directly
-    to the Stackdriver Logging API.
-
-    This handler supports both an asynchronous and synchronous transport.
+    """A logging handler that sends App Engine-formatted logs to Stackdriver.
 
     :type client: :class:`google.cloud.logging.client`
     :param client: the authenticated Google Cloud Logging client for this
                    handler to use
 
-    :type name: str
-    :param name: the name of the custom log in Stackdriver Logging. Defaults
-                 to 'python'. The name of the Python logger will be represented
-                 in the ``python_logger`` field.
-
     :type transport: type
-    :param transport: Class for creating new transport objects. It should
-                      extend from the base :class:`.Transport` type and
-                      implement :meth`.Transport.send`. Defaults to
-                      :class:`.BackgroundThreadTransport`. The other
-                      option is :class:`.SyncTransport`.
-
-    :type resource: :class:`~google.cloud.logging.resource.Resource`
-    :param resource: Monitored resource of the entry, defaults
-                     to the global resource type.
+    :param transport: The transport class. It should be a subclass of :class:`.Transport`.
+                      If unspecified, :class:`.BackgroundThreadTransport` will be used.
     """
 
-    DEFAULT_LOGGER_NAME = 'app'
-
     def __init__(self, client,
                  transport=BackgroundThreadTransport):
-        super(AppEngineHandler, self).__init__(client, name=self.DEFAULT_LOGGER_NAME,
-                                               transport=transport, resource=self.gae_resource)
+        super(AppEngineHandler, self).__init__(
+            client,
+            name=_DEFAULT_GAE_LOGGER_NAME,
+            transport=transport,
+            resource=self.get_gae_resource())
 
-    @property
-    def gae_resource(self):
+    def get_gae_resource(self):
         gae_resource = Resource(
             type='gae_app',
             labels={
diff --git a/logging/google/cloud/logging/handlers/transports/sync.py b/logging/google/cloud/logging/handlers/transports/sync.py
@@ -18,7 +18,6 @@
 """
 
 from google.cloud.logging.handlers.transports.base import Transport
-from google.cloud.logging.resource import Resource
 
 
 class SyncTransport(Transport):
diff --git a/logging/tests/unit/handlers/test_app_engine.py b/logging/tests/unit/handlers/test_app_engine.py
@@ -17,7 +17,6 @@
 
 
 class TestAppEngineHandlerHandler(unittest.TestCase):
-
     PROJECT = 'PROJECT'
 
     def _get_target_class(self):
@@ -47,7 +46,7 @@ def test_ctor(self):
     def test_emit(self):
         client = _Client(self.PROJECT)
         handler = self._make_one(client, transport=_Transport)
-        gae_resource = handler.gae_resource
+        gae_resource = handler.get_gae_resource()
         logname = 'loggername'
         message = 'hello world'
         record = logging.LogRecord(logname, logging, None, None, message,
