Fixed and added Tables unit tests.

sdk/tables/azure-data-tables/pom.xml

@@ -32,10 +32,6 @@ Licensed under the MIT License.
     <tag>HEAD</tag>
   </scm>
 
-  <properties>
-    <jacoco.skip.coverage.check>true</jacoco.skip.coverage.check>
-  </properties>
-
   <dependencies>
     <dependency>
       <groupId>com.azure</groupId>

sdk/tables/azure-data-tables/src/main/java/com/azure/data/tables/TableServiceAsyncClient.java

@@ -72,6 +72,7 @@
 @ServiceClient(builder = TableServiceClientBuilder.class, isAsync = true)
 public final class TableServiceAsyncClient {
     private final ClientLogger logger = new ClientLogger(TableServiceAsyncClient.class);
+    private final SerializerAdapter serializerAdapter;
     private final AzureTableImpl implementation;
     private final String accountName;
     private final HttpPipeline pipeline;
@@ -88,6 +89,7 @@ public final class TableServiceAsyncClient {
             throw logger.logExceptionAsError(ex);
         }
 
+        this.serializerAdapter = serializerAdapter;
         this.implementation = new AzureTableImplBuilder()
             .serializerAdapter(serializerAdapter)
             .url(url)
@@ -142,6 +144,15 @@ ClientLogger getLogger() {
         return this.logger;
     }
 
+    /**
+     * Gets this client's {@link SerializerAdapter}.
+     *
+     * @return This client's {@link SerializerAdapter}.
+     */
+    SerializerAdapter getSerializerAdapter() {
+        return this.serializerAdapter;
+    }
+
     /**
      * Gets the REST API version used by this client.
      *

sdk/tables/azure-data-tables/src/main/java/com/azure/data/tables/TableServiceClient.java

@@ -6,11 +6,15 @@
 import com.azure.core.annotation.ServiceClient;
 import com.azure.core.annotation.ServiceMethod;
 import com.azure.core.credential.AzureNamedKeyCredential;
+import com.azure.core.http.HttpPipeline;
 import com.azure.core.http.HttpResponse;
 import com.azure.core.http.rest.PagedIterable;
 import com.azure.core.http.rest.Response;
 import com.azure.core.http.rest.SimpleResponse;
 import com.azure.core.util.Context;
+import com.azure.core.util.logging.ClientLogger;
+import com.azure.core.util.serializer.SerializerAdapter;
+import com.azure.data.tables.implementation.AzureTableImpl;
 import com.azure.data.tables.implementation.TableUtils;
 import com.azure.data.tables.implementation.models.ResponseFormat;
 import com.azure.data.tables.implementation.models.TableProperties;
@@ -73,6 +77,24 @@ public TableServiceVersion getServiceVersion() {
         return client.getServiceVersion();
     }
 
+    /**
+     * Gets the {@link HttpPipeline} powering this client.
+     *
+     * @return This client's {@link HttpPipeline}.
+     */
+    HttpPipeline getHttpPipeline() {
+        return client.getHttpPipeline();
+    }
+
+    /**
+     * Gets this client's {@link SerializerAdapter}.
+     *
+     * @return This client's {@link SerializerAdapter}.
+     */
+    SerializerAdapter getSerializerAdapter() {
+        return client.getSerializerAdapter();
+    }
+
     /**
      * Generates an account SAS for the Azure Storage account using the specified
      * {@link TableAccountSasSignatureValues}.

sdk/tables/azure-data-tables/src/main/java/com/azure/data/tables/implementation/models/TableServiceProperties.java

@@ -5,10 +5,16 @@
 package com.azure.data.tables.implementation.models;
 
 import com.azure.core.annotation.Fluent;
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
+import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlProperty;
+import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlRootElement;
+
+import java.util.ArrayList;
 import java.util.List;
 
 /** Table Service Properties. */
+@JacksonXmlRootElement(localName = "StorageServiceProperties")
 @Fluent
 public final class TableServiceProperties {
     /*
@@ -31,11 +37,20 @@ public final class TableServiceProperties {
     @JsonProperty(value = "MinuteMetrics")
     private Metrics minuteMetrics;
 
+    private static final class CorsWrapper {
+        @JacksonXmlProperty(localName = "CorsRule")
+        private final List<CorsRule> items;
+
+        @JsonCreator
+        private CorsWrapper(@JacksonXmlProperty(localName = "CorsRule") List<CorsRule> items) {
+            this.items = items;
+        }
+    }
     /*
      * The set of CORS rules.
      */
     @JsonProperty(value = "Cors")
-    private List<CorsRule> cors;
+    private CorsWrapper cors;
 
     /**
      * Get the logging property: Azure Analytics Logging settings.
@@ -103,7 +118,11 @@ public TableServiceProperties setMinuteMetrics(Metrics minuteMetrics) {
      * @return the cors value.
      */
     public List<CorsRule> getCors() {
-        return this.cors;
+        if (this.cors == null) {
+            this.cors = new CorsWrapper(new ArrayList<CorsRule>());
+        }
+
+        return this.cors.items;
     }
 
     /**
@@ -113,7 +132,8 @@ public List<CorsRule> getCors() {
      * @return the TableServiceProperties object itself.
      */
     public TableServiceProperties setCors(List<CorsRule> cors) {
-        this.cors = cors;
+        this.cors = new CorsWrapper(cors);
+
         return this;
     }
 }

sdk/tables/azure-data-tables/src/main/java/com/azure/data/tables/implementation/models/TableServiceStats.java

@@ -6,8 +6,10 @@
 
 import com.azure.core.annotation.Fluent;
 import com.fasterxml.jackson.annotation.JsonProperty;
+import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlRootElement;
 
 /** Stats for the service. */
+@JacksonXmlRootElement(localName = "StorageServiceStats")
 @Fluent
 public final class TableServiceStats {
     /*

sdk/tables/azure-data-tables/src/test/java/com/azure/data/tables/TableAsyncClientTest.java

@@ -224,7 +224,7 @@ void createEntityWithAllSupportedDataTypesAsync() {
         StepVerifier.create(tableClient.getEntityWithResponse(partitionKeyValue, rowKeyValue, null))
             .assertNext(response -> {
                 final TableEntity entity = response.getValue();
-                Map<String, Object> properties = entity.getProperties();
+                final Map<String, Object> properties = entity.getProperties();
                 assertTrue(properties.get("BinaryTypeProperty") instanceof byte[]);
                 assertTrue(properties.get("BooleanTypeProperty") instanceof Boolean);
                 assertTrue(properties.get("DateTypeProperty") instanceof OffsetDateTime);
@@ -493,7 +493,7 @@ void getEntityWithResponseSubclassAsync() {
         String s = "Test";
         SampleEntity.Color color = SampleEntity.Color.GREEN;
 
-        Map<String, Object> props = new HashMap<>();
+        final Map<String, Object> props = new HashMap<>();
         props.put("ByteField", bytes);
         props.put("BooleanField", b);
         props.put("DateTimeField", dateTime);
@@ -607,7 +607,7 @@ void updateEntityWithResponseSubclassAsync() {
 
         StepVerifier.create(tableClient.getEntity(partitionKeyValue, rowKeyValue))
             .assertNext(entity -> {
-                Map<String, Object> properties = entity.getProperties();
+                final Map<String, Object> properties = entity.getProperties();
                 assertTrue(properties.containsKey("SubclassProperty"));
                 assertEquals("UpdatedValue", properties.get("SubclassProperty"));
             })