Add optional params for list transfers api

Author: kushal

bitgo-java-api/src/main/java/com/bitso/bitgo/v2/BitGoClient.java

@@ -83,7 +83,7 @@ Optional<SendCoinsResponse> sendMany(String coin, String walletId, String wallet
     Optional<Map<String, Object>> getWalletTransferSeqId(String coin, String walletId, String sequenceId) throws IOException;
 
     /**
-     * See https://www.bitgo.com/api/v2/#list-wallet-transactions
+     * See https://www.bitgo.com/api/v2/#operation/v2.wallet.listtransfers
      *
      * @param coin
      * @param walletId
@@ -92,7 +92,7 @@ Optional<SendCoinsResponse> sendMany(String coin, String walletId, String wallet
      * @return
      * @throws IOException
      */
-    WalletTransferResponse listWalletTransfers(String coin, String walletId, String nextBatchPrevId, int limit) throws IOException;
+    WalletTransferResponse listWalletTransfers(String coin, String walletId, String nextBatchPrevId, int limit, Map<String, Object> optionalParameters) throws IOException;
 
     /**
      * See https://www.bitgo.com/api/v2/?shell#unlock

bitgo-java-api/src/main/java/com/bitso/bitgo/v2/BitGoClientImpl.java

@@ -238,17 +238,23 @@ public Optional<Map<String, Object>> getWalletTransferSeqId(String coin, String
     }
 
     @Override
-    public WalletTransferResponse listWalletTransfers(String coin, String walletId, String prevId, int limit) throws IOException {
+    public WalletTransferResponse listWalletTransfers(String coin, String walletId, String prevId, int limit, Map<String, Object> optionalParameters) throws IOException {
         if (limit > 250) limit = 250;
         if (limit < 0) limit = 0;
         String url = baseUrl + LIST_WALLET_TRANSFER_URL.replace("$COIN", coin).replace("$WALLET", walletId) + "?limit=" + limit;
 
-        Map<String, String> reqPropMap = null;
+        Map<String, String> props = null;
         if (prevId != null) {
-            reqPropMap = new HashMap<>();
-            reqPropMap.put("prevId", prevId);
+            props = new HashMap<>();
+            props.put("prevId", prevId);
         }
-        HttpURLConnection conn = httpGetRetry500(url, reqPropMap);
+        if (optionalParameters != null && optionalParameters.size() > 0) {
+            if (props == null) props = new HashMap<>();
+            for (final Map.Entry<String, Object> entry : optionalParameters.entrySet()) {
+                props.put(entry.getKey(), entry.getValue().toString());
+            }
+        }
+        HttpURLConnection conn = httpGetRetry500(url, props);
 
         final WalletTransferResponse resp = SerializationUtil.mapper.readValue(conn.getInputStream(), WalletTransferResponse.class);
         log.trace("listWalletTransactions response: {}", resp);

bitgo-java-api/src/test/java/com/bitso/bitgo/util/TestClientV2.java

@@ -23,6 +23,8 @@
 /**
  * Simple client tests.
  *
+ * docker run -it -p 3080:3080 bitgosdk/express:latest
+ *
  * @author Enrique Zamudio
  * Date: 5/8/17 8:04 PM
  */
@@ -124,9 +126,9 @@ public void currentUserProfile() throws IOException {
 
     @Test
     public void listWalletTransfers() throws IOException {
-        final WalletTransferResponse resp = client.listWalletTransfers(COIN, WALLET_ID, null, 250);
+        final WalletTransferResponse resp = client.listWalletTransfers(COIN, WALLET_ID, null, 250, Map.of("dateGte", 1536189990165L));
         System.out.println("list.size() = " + resp.getTransfers().size());
-        assertTrue(resp.getTransfers().size() > 10);
+        assertTrue(resp.getTransfers().size() == 3);
 //        System.out.println(resp);
 //        for (int i = 0; i < 30; i++) {
 //            resp.getTransfers().remove(0);