fix: Add labels to converter for listTables method (#3735)

Author: Tomasz Los

google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/spi/v2/HttpBigQueryRpc.java

@@ -457,7 +457,8 @@ public Table apply(TableList.Tables tablePb) {
                     .setCreationTime(tablePb.getCreationTime())
                     .setTimePartitioning(tablePb.getTimePartitioning())
                     .setRangePartitioning(tablePb.getRangePartitioning())
-                    .setClustering(tablePb.getClustering());
+                    .setClustering(tablePb.getClustering())
+                    .setLabels(tablePb.getLabels());
               }
             }));
   }

google-cloud-bigquery/src/test/java/com/google/cloud/bigquery/BigQueryImplTest.java

@@ -155,6 +155,9 @@ public class BigQueryImplTest {
       TableInfo.of(TABLE_ID, TABLE_DEFINITION_WITH_RANGE_PARTITIONING);
   private static final TableInfo TABLE_INFO = TableInfo.of(TABLE_ID, TABLE_DEFINITION);
   private static final TableInfo OTHER_TABLE_INFO = TableInfo.of(OTHER_TABLE_ID, TABLE_DEFINITION);
+  private static final TableInfo OTHER_TABLE_WITH_LABELS_INFO = TableInfo.newBuilder(OTHER_TABLE_ID, TABLE_DEFINITION)
+      .setLabels(ImmutableMap.of("key", "value"))
+      .build();
   private static final TableInfo TABLE_INFO_WITH_PROJECT =
       TableInfo.of(TABLE_ID_WITH_PROJECT, TABLE_DEFINITION);
   private static final TableInfo MODEL_TABLE_INFO_WITH_PROJECT =
@@ -1151,6 +1154,23 @@ public void testListTablesFromDatasetIdWithProject() throws IOException {
         .listTablesSkipExceptionTranslation(OTHER_PROJECT, DATASET, EMPTY_RPC_OPTIONS);
   }
 
+  @Test
+  public void testListTablesWithLabels() throws IOException {
+    bigquery = options.getService();
+    ImmutableList<Table> tableList =
+        ImmutableList.of(
+            new Table(bigquery, new TableInfo.BuilderImpl(OTHER_TABLE_WITH_LABELS_INFO)));
+    Tuple<String, Iterable<com.google.api.services.bigquery.model.Table>> result =
+        Tuple.of(CURSOR, Iterables.transform(tableList, TableInfo.TO_PB_FUNCTION));
+    when(bigqueryRpcMock.listTablesSkipExceptionTranslation(PROJECT, DATASET, EMPTY_RPC_OPTIONS))
+        .thenReturn(result);
+    Page<Table> page = bigquery.listTables(DATASET);
+    assertEquals(CURSOR, page.getNextPageToken());
+    assertArrayEquals(tableList.toArray(), Iterables.toArray(page.getValues(), Table.class));
+    verify(bigqueryRpcMock).listTablesSkipExceptionTranslation(PROJECT, DATASET, EMPTY_RPC_OPTIONS);
+    assertEquals(ImmutableMap.of("key", "value"), page.getValues().iterator().next().getLabels());
+  }
+
   @Test
   public void testListTablesWithOptions() throws IOException {
     bigquery = options.getService();