Merge pull request #801 from tseaver/799-work_around_cannot_bind_fields_query_param

Fix #799: Don't pass 'fields' query param to possibly non-existing subscription.

Author: tseaver

gcloud/pubsub/subscription.py

@@ -107,9 +107,7 @@ def exists(self):
         """
         conn = self.topic.connection
         try:
-            conn.api_request(method='GET',
-                             path=self.path,
-                             query_params={'fields': 'name'})
+            conn.api_request(method='GET', path=self.path)
         except NotFound:
             return False
         else:

gcloud/pubsub/test_subscription.py

@@ -171,7 +171,7 @@ def test_exists_miss(self):
         req = conn._requested[0]
         self.assertEqual(req['method'], 'GET')
         self.assertEqual(req['path'], '/%s' % SUB_PATH)
-        self.assertEqual(req['query_params'], {'fields': 'name'})
+        self.assertEqual(req.get('query_params'), None)
 
     def test_exists_hit(self):
         PROJECT = 'PROJECT'
@@ -187,7 +187,7 @@ def test_exists_hit(self):
         req = conn._requested[0]
         self.assertEqual(req['method'], 'GET')
         self.assertEqual(req['path'], '/%s' % SUB_PATH)
-        self.assertEqual(req['query_params'], {'fields': 'name'})
+        self.assertEqual(req.get('query_params'), None)
 
     def test_reload(self):
         PROJECT = 'PROJECT'