Merge pull request #1311 from tseaver/bigquery-fix_list_jobs_projection

Fix 'Client.list_jobs' to use 'full' projection.
googleapis · Dec 21, 2015 · 4734112 · 4734112
2 parents 4861e7e + ceed0c0
commit 4734112
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/gcloud/bigquery/client.py b/gcloud/bigquery/client.py
@@ -159,7 +159,7 @@ def list_jobs(self, max_results=None, page_token=None, all_users=None,
                   retrieved with another call, passing that value as
                   ``page_token``).
         """
-        params = {}
+        params = {'projection': 'full'}
 
         if max_results is not None:
             params['maxResults'] = max_results

diff --git a/gcloud/bigquery/test_client.py b/gcloud/bigquery/test_client.py
@@ -260,7 +260,7 @@ def test_list_jobs_defaults(self):
         req = conn._requested[0]
         self.assertEqual(req['method'], 'GET')
         self.assertEqual(req['path'], '/%s' % PATH)
-        self.assertEqual(req['query_params'], {})
+        self.assertEqual(req['query_params'], {'projection': 'full'})
 
     def test_list_jobs_explicit_empty(self):
         PROJECT = 'PROJECT'
@@ -282,7 +282,8 @@ def test_list_jobs_explicit_empty(self):
         self.assertEqual(req['method'], 'GET')
         self.assertEqual(req['path'], '/%s' % PATH)
         self.assertEqual(req['query_params'],
-                         {'maxResults': 1000,
+                         {'projection': 'full',
+                          'maxResults': 1000,
                           'pageToken': TOKEN,
                           'allUsers': True,
                           'stateFilter': 'done'})