Drop redundant method overrides, left over from Pub/Sub.

Addresses:
#1560 (comment)
#1560 (comment)

Author: tseaver

gcloud/logging/connection.py

@@ -46,40 +46,3 @@ class Connection(base_connection.JSONConnection):
              'https://www.googleapis.com/auth/logging.admin',
              'https://www.googleapis.com/auth/cloud-platform')
     """The scopes required for authenticating as a Cloud Logging consumer."""
-
-    def __init__(self, credentials=None, http=None, api_base_url=None):
-        super(Connection, self).__init__(credentials=credentials, http=http)
-        if api_base_url is None:
-            api_base_url = self.__class__.API_BASE_URL
-        self.api_base_url = api_base_url
-
-    def build_api_url(self, path, query_params=None,
-                      api_base_url=None, api_version=None):
-        """Construct an API url given a few components, some optional.
-
-        Typically, you shouldn't need to use this method.
-
-        :type path: string
-        :param path: The path to the resource.
-
-        :type query_params: dict
-        :param query_params: A dictionary of keys and values to insert into
-                             the query string of the URL.
-
-        :type api_base_url: string
-        :param api_base_url: The base URL for the API endpoint.
-                             Typically you won't have to provide this.
-
-        :type api_version: string
-        :param api_version: The version of the API to call.
-                            Typically you shouldn't provide this and instead
-                            use the default for the library.
-
-        :rtype: string
-        :returns: The URL assembled from the pieces provided.
-        """
-        if api_base_url is None:
-            api_base_url = self.api_base_url
-        return super(Connection, self.__class__).build_api_url(
-            path, query_params=query_params,
-            api_base_url=api_base_url, api_version=api_version)

gcloud/logging/test_connection.py

@@ -27,45 +27,3 @@ def _makeOne(self, *args, **kw):
     def test_default_url(self):
         conn = self._makeOne()
         klass = self._getTargetClass()
-        self.assertEqual(conn.api_base_url, klass.API_BASE_URL)
-
-    def test_custom_url_from_constructor(self):
-        HOST = object()
-        conn = self._makeOne(api_base_url=HOST)
-
-        klass = self._getTargetClass()
-        self.assertNotEqual(conn.api_base_url, klass.API_BASE_URL)
-        self.assertEqual(conn.api_base_url, HOST)
-
-    def test_build_api_url_no_extra_query_params(self):
-        conn = self._makeOne()
-        URI = '/'.join([
-            conn.API_BASE_URL,
-            conn.API_VERSION,
-            'foo',
-        ])
-        self.assertEqual(conn.build_api_url('/foo'), URI)
-
-    def test_build_api_url_w_extra_query_params(self):
-        from six.moves.urllib.parse import parse_qsl
-        from six.moves.urllib.parse import urlsplit
-        conn = self._makeOne()
-        uri = conn.build_api_url('/foo', {'bar': 'baz'})
-        scheme, netloc, path, qs, _ = urlsplit(uri)
-        self.assertEqual('%s://%s' % (scheme, netloc), conn.API_BASE_URL)
-        self.assertEqual(path,
-                         '/'.join(['', conn.API_VERSION, 'foo']))
-        parms = dict(parse_qsl(qs))
-        self.assertEqual(parms['bar'], 'baz')
-
-    def test_build_api_url_w_base_url_override(self):
-        base_url1 = 'api-base-url1'
-        base_url2 = 'api-base-url2'
-        conn = self._makeOne(api_base_url=base_url1)
-        URI = '/'.join([
-            base_url2,
-            conn.API_VERSION,
-            'foo',
-        ])
-        self.assertEqual(
-            conn.build_api_url('/foo', api_base_url=base_url2), URI)