code review

g2vinay · Jun 4, 2019 · b3ff055 · b3ff055
1 parent ad7ba3a
commit b3ff055
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/sdk/src/test/java/com/microsoft/azure/cosmosdb/rx/TestSuiteBase.java b/sdk/src/test/java/com/microsoft/azure/cosmosdb/rx/TestSuiteBase.java
@@ -700,7 +700,7 @@ static protected void safeCloseAsync(AsyncDocumentClient client) {
                 try {
                     client.close();
                 } catch (Exception e) {
-                    e.printStackTrace();
+                    logger.error("failed to close client", e);
                 }
             }).start();
         }
@@ -711,7 +711,7 @@ static protected void safeClose(AsyncDocumentClient client) {
             try {
                 client.close();
             } catch (Exception e) {
-                e.printStackTrace();
+                logger.error("failed to close client", e);
             }
         }
     }
@@ -893,7 +893,7 @@ public static Object[][] clientBuildersWithDirectSession() {
     }
 
     static Protocol[] toArray(List<Protocol> protocols) {
-        return protocols.toArray(new Protocol[0]);
+        return protocols.toArray(new Protocol[protocols.size()]);
     }
 
     private static Object[][] clientBuildersWithDirectSession(Protocol... protocols) {