Fix client.patch_table

tableId is a required argument of the patch method. Also, there's no
need to pass a tableReference in the body.

Signed-off-by: Yves Bastide <yves@botify.com>

Author: ybastide

bigquery/client.py

@@ -735,17 +735,13 @@ def patch_table(self, dataset, table, schema, project_id=None):
 
         body = {
             'schema': {'fields': schema},
-            'tableReference': {
-                'tableId': table,
-                'projectId': project_id,
-                'datasetId': dataset
-            }
         }
 
         try:
             result = self.bigquery.tables().patch(
                 projectId=project_id,
                 datasetId=dataset,
+                tableId=table,
                 body=body
             ).execute(num_retries=self.num_retries)
             if self.swallow_results:

bigquery/tests/test_client.py

@@ -1913,9 +1913,6 @@ def setUp(self):
         self.client = client.BigQueryClient(self.mock_bq_service, self.project)
         self.body = {
             'schema': {'fields': self.schema},
-            'tableReference': {
-                'tableId': self.table, 'projectId': self.project,
-                'datasetId': self.dataset}
         }
         self.expiration_time = 1437513693000
 
@@ -1941,7 +1938,8 @@ def test_table_patch_failed(self):
         self.client.swallow_results = True
 
         self.mock_tables.patch.assert_called_with(
-            projectId=self.project, datasetId=self.dataset, body=self.body)
+            projectId=self.project, datasetId=self.dataset,
+            tableId=self.table, body=self.body)
 
         self.mock_tables.patch.return_value.execute. \
             assert_called_with(num_retries=0)
@@ -1968,7 +1966,8 @@ def test_table_patch_success(self):
         self.client.swallow_results = True
 
         self.mock_tables.patch.assert_called_with(
-            projectId=self.project, datasetId=self.dataset, body=self.body)
+            projectId=self.project, datasetId=self.dataset,
+            tableId=self.table, body=self.body)
 
         self.mock_tables.patch.return_value.execute. \
             assert_called_with(num_retries=0)