Renaming JSONClient -> ClientWithProject.

dhermes · dhermes · commit b53be84e12a3 · 2017-01-26T14:36:16.000-08:00
Done via:
$ git grep -l JSONClient | xargs sed -i s/JSONClient/ClientWithProject/g

Also fixing test b0rken by previous commit.
diff --git a/bigquery/google/cloud/bigquery/client.py b/bigquery/google/cloud/bigquery/client.py
@@ -15,7 +15,7 @@
 """Client for interacting with the Google BigQuery API."""
 
 
-from google.cloud.client import JSONClient
+from google.cloud.client import ClientWithProject
 from google.cloud.bigquery._http import Connection
 from google.cloud.bigquery.dataset import Dataset
 from google.cloud.bigquery.job import CopyJob
@@ -50,7 +50,7 @@ def from_api_repr(cls, resource):
             resource['id'], resource['numericId'], resource['friendlyName'])
 
 
-class Client(JSONClient):
+class Client(ClientWithProject):
     """Client to bundle configuration needed for API requests.
 
     :type project: str
diff --git a/core/google/cloud/client.py b/core/google/cloud/client.py
@@ -129,11 +129,8 @@ def _determine_default(project):
         return _determine_default_project(project)
 
 
-class JSONClient(Client, _ClientProjectMixin):
-    """Client for Google JSON-based API.
-
-    Assumes such APIs use the ``project`` and the client needs to store this
-    value.
+class ClientWithProject(Client, _ClientProjectMixin):
+    """Client that also stores a project.
 
     :type project: str
     :param project: the project which the client acts on behalf of. If not
diff --git a/core/unit_tests/test__helpers.py b/core/unit_tests/test__helpers.py
@@ -789,7 +789,7 @@ def mock_channel(*args, **kwargs):
         credentials = object()
         user_agent = 'you-sir-age-int'
         host = 'localhost'
-        extra_options = {'extra_options': None}
+        extra_options = {'extra_options': ()}
         with _Monkey(MUT, make_secure_channel=mock_channel):
             stub = self._call_fut(credentials, user_agent,
                                   stub_class, host)
diff --git a/core/unit_tests/test_client.py b/core/unit_tests/test_client.py
@@ -104,13 +104,13 @@ def test_from_service_account_json_bad_args(self):
                 mock.sentinel.filename, credentials=mock.sentinel.credentials)
 
 
-class TestJSONClient(unittest.TestCase):
+class TestClientWithProject(unittest.TestCase):
 
     @staticmethod
     def _get_target_class():
-        from google.cloud.client import JSONClient
+        from google.cloud.client import ClientWithProject
 
-        return JSONClient
+        return ClientWithProject
 
     def _make_one(self, *args, **kw):
         return self._get_target_class()(*args, **kw)
diff --git a/dns/google/cloud/dns/client.py b/dns/google/cloud/dns/client.py
@@ -15,13 +15,13 @@
 """Client for interacting with the Google Cloud DNS API."""
 
 
-from google.cloud.client import JSONClient
+from google.cloud.client import ClientWithProject
 from google.cloud.dns.connection import Connection
 from google.cloud.dns.zone import ManagedZone
 from google.cloud.iterator import HTTPIterator
 
 
-class Client(JSONClient):
+class Client(ClientWithProject):
     """Client to bundle configuration needed for API requests.
 
     :type project: str
diff --git a/logging/google/cloud/logging/client.py b/logging/google/cloud/logging/client.py
@@ -29,7 +29,7 @@
 else:
     _HAVE_GAX = True
 
-from google.cloud.client import JSONClient
+from google.cloud.client import ClientWithProject
 from google.cloud.environment_vars import DISABLE_GRPC
 from google.cloud.logging._http import Connection
 from google.cloud.logging._http import _LoggingAPI as JSONLoggingAPI
@@ -59,7 +59,7 @@
 """Environment variable set in a Google Container Engine environment."""
 
 
-class Client(JSONClient):
+class Client(ClientWithProject):
     """Client to bundle configuration needed for API requests.
 
     :type project: str
diff --git a/monitoring/google/cloud/monitoring/client.py b/monitoring/google/cloud/monitoring/client.py
@@ -31,7 +31,7 @@
 import datetime
 
 from google.cloud._helpers import _datetime_to_rfc3339
-from google.cloud.client import JSONClient
+from google.cloud.client import ClientWithProject
 from google.cloud.monitoring.connection import Connection
 from google.cloud.monitoring.group import Group
 from google.cloud.monitoring.metric import Metric
@@ -47,7 +47,7 @@
 _UTCNOW = datetime.datetime.utcnow  # To be replaced by tests.
 
 
-class Client(JSONClient):
+class Client(ClientWithProject):
     """Client to bundle configuration needed for API requests.
 
     :type project: str
diff --git a/pubsub/google/cloud/pubsub/client.py b/pubsub/google/cloud/pubsub/client.py
@@ -16,7 +16,7 @@
 
 import os
 
-from google.cloud.client import JSONClient
+from google.cloud.client import ClientWithProject
 from google.cloud.environment_vars import DISABLE_GRPC
 from google.cloud.pubsub._http import Connection
 from google.cloud.pubsub._http import _PublisherAPI as JSONPublisherAPI
@@ -43,7 +43,7 @@
 _USE_GAX = _HAVE_GAX and not _DISABLE_GAX
 
 
-class Client(JSONClient):
+class Client(ClientWithProject):
     """Client to bundle configuration needed for API requests.
 
     :type project: str
diff --git a/runtimeconfig/google/cloud/runtimeconfig/client.py b/runtimeconfig/google/cloud/runtimeconfig/client.py
@@ -15,12 +15,12 @@
 """Client for interacting with the Google Cloud RuntimeConfig API."""
 
 
-from google.cloud.client import JSONClient
+from google.cloud.client import ClientWithProject
 from google.cloud.runtimeconfig.connection import Connection
 from google.cloud.runtimeconfig.config import Config
 
 
-class Client(JSONClient):
+class Client(ClientWithProject):
     """Client to bundle configuration needed for API requests.
 
     :type project: str
diff --git a/storage/google/cloud/storage/client.py b/storage/google/cloud/storage/client.py
@@ -16,15 +16,15 @@
 
 
 from google.cloud._helpers import _LocalStack
-from google.cloud.client import JSONClient
+from google.cloud.client import ClientWithProject
 from google.cloud.exceptions import NotFound
 from google.cloud.iterator import HTTPIterator
 from google.cloud.storage._http import Connection
 from google.cloud.storage.batch import Batch
 from google.cloud.storage.bucket import Bucket
 
 
-class Client(JSONClient):
+class Client(ClientWithProject):
     """Client to bundle configuration needed for API requests.
 
     :type project: str
diff --git a/vision/google/cloud/vision/client.py b/vision/google/cloud/vision/client.py
@@ -16,7 +16,7 @@
 
 import os
 
-from google.cloud.client import JSONClient
+from google.cloud.client import ClientWithProject
 from google.cloud.environment_vars import DISABLE_GRPC
 
 from google.cloud.vision._gax import _GAPICVisionAPI
@@ -28,7 +28,7 @@
 _USE_GAX = not os.getenv(DISABLE_GRPC, False)
 
 
-class Client(JSONClient):
+class Client(ClientWithProject):
     """Client to bundle configuration needed for API requests.
 
     :type project: str
