fix: remove deposit fee methods

Author: Insmael

src/main/java/com/cryptomarket/sdk/rest/CryptomarketRestClient.java

@@ -1388,7 +1388,6 @@ public String getEstimateWithdrawalFee(String currency, String amount, @Nullable
    */
   public List<Fee> getEstimateWithdrawalFees(List<FeeRequest> feeRequests) throws CryptomarketSDKException;
 
-
   /**
    * Get estimates of withdrawal fees
    * <p>
@@ -1402,7 +1401,6 @@ public String getEstimateWithdrawalFee(String currency, String amount, @Nullable
    */
   public List<Fee> getBulkEstimateWithdrawalFees(List<FeeRequest> feeRequests) throws CryptomarketSDKException;
 
-
   /**
    * Get an estimate of the deposit fee
    * <p>
@@ -1416,15 +1414,16 @@ public String getEstimateWithdrawalFee(String currency, String amount, @Nullable
    * @return The expected fee
    * @throws CryptomarketSDKException
    */
-  public String getEstimateDepositFee(String currency, String amount, @Nullable String networkCode)
-      throws CryptomarketSDKException;
+  // public String getEstimateDepositFee(String currency, String amount, @Nullable
+  // String networkCode) throws CryptomarketSDKException;
 
   /**
    * @see #getEstimateWithdrawalFee(String, String, String)
    * @param paramsBuilder
    * @throws CryptomarketSDKException
    */
-  public String getEstimateDepositFee(ParamsBuilder paramsBuilder) throws CryptomarketSDKException;
+  // public String getEstimateDepositFee(ParamsBuilder paramsBuilder) throws
+  // CryptomarketSDKException;
 
   /**
    * Get estimates of deposit fees
@@ -1437,7 +1436,8 @@ public String getEstimateDepositFee(String currency, String amount, @Nullable St
    * @return the list of fees
    * @throws CryptomarketSDKException
    */
-  public List<Fee> getBulkEstimateDepositFees(List<FeeRequest> feeRequests) throws CryptomarketSDKException;
+  // public List<Fee> getBulkEstimateDepositFees(List<FeeRequest> feeRequests)
+  // throws CryptomarketSDKException;
 
   /**
    * Converts between currencies

src/main/java/com/cryptomarket/sdk/rest/CryptomarketRestClientImpl.java

@@ -922,34 +922,34 @@ public List<Fee> getBulkEstimateWithdrawalFees(List<FeeRequest> feeRequests) thr
     return adapter.listFromJson(jsonResponse, Fee.class);
   }
 
-  @Override
-  public String getEstimateDepositFee(String currency, String amount, String networkCode)
-      throws CryptomarketSDKException {
-    return getEstimateDepositFee(new ParamsBuilder()
-        .currency(currency)
-        .networkCode(networkCode)
-        .amount(amount));
-  }
-
-  @Override
-  public String getEstimateDepositFee(ParamsBuilder paramsBuilder) throws CryptomarketSDKException {
-    paramsBuilder.checkRequired(Arrays.asList(
-        ArgNames.CURRENCY,
-        ArgNames.AMOUNT));
-    String jsonResponse = httpClient.get(
-        "wallet/crypto/fee/deposit/estimate",
-        paramsBuilder.build());
-    return adapter.objectFromJsonValue(jsonResponse, "fee", String.class);
-  }
-
-  @Override
-  public List<Fee> getBulkEstimateDepositFees(List<FeeRequest> feeRequests) throws CryptomarketSDKException {
-    var payload = adapter.listToJson(feeRequests, FeeRequest.class);
-    String jsonResponse = httpClient.post(
-        "wallet/crypto/fee/deposit/estimate/bulk",
-        payload);
-    return adapter.listFromJson(jsonResponse, Fee.class);
-  }
+  // @Override
+  // public String getEstimateDepositFee(String currency, String amount, String networkCode)
+  //     throws CryptomarketSDKException {
+  //   return getEstimateDepositFee(new ParamsBuilder()
+  //       .currency(currency)
+  //       .networkCode(networkCode)
+  //       .amount(amount));
+  // }
+
+  // @Override
+  // public String getEstimateDepositFee(ParamsBuilder paramsBuilder) throws CryptomarketSDKException {
+  //   paramsBuilder.checkRequired(Arrays.asList(
+  //       ArgNames.CURRENCY,
+  //       ArgNames.AMOUNT));
+  //   String jsonResponse = httpClient.get(
+  //       "wallet/crypto/fee/deposit/estimate",
+  //       paramsBuilder.build());
+  //   return adapter.objectFromJsonValue(jsonResponse, "fee", String.class);
+  // }
+
+  // @Override
+  // public List<Fee> getBulkEstimateDepositFees(List<FeeRequest> feeRequests) throws CryptomarketSDKException {
+  //   var payload = adapter.listToJson(feeRequests, FeeRequest.class);
+  //   String jsonResponse = httpClient.post(
+  //       "wallet/crypto/fee/deposit/estimate/bulk",
+  //       payload);
+  //   return adapter.listFromJson(jsonResponse, Fee.class);
+  // }
 
   @Override
   public List<String> convertBetweenCurrencies(

src/test/java/com/cryptomarket/sdk/TestRestClientWalletManagement.java

@@ -143,23 +143,23 @@ public void testGetEstimateWithdrawFee() throws CryptomarketSDKException {
     }
   }
 
-  @Test
-  public void testGetEstimateDepositFees() throws CryptomarketSDKException {
-    var fees = client
-        .getBulkEstimateDepositFees(List.of(new FeeRequest("EOS", "100"), new FeeRequest("ETH", "100")));
-    if (fees.size() != 2) {
-      fail("invalid amount of fees");
-    }
-    fees.forEach(Checker.checkFee);
-  }
-
-  @Test
-  public void testGetEstimateDepositFee() throws CryptomarketSDKException {
-    String estimate = client.getEstimateDepositFee("EOS", "100", null);
-    if (estimate.equals("")) {
-      fail();
-    }
-  }
+  // @Test
+  // public void testGetEstimateDepositFees() throws CryptomarketSDKException {
+  //   var fees = client
+  //       .getBulkEstimateDepositFees(List.of(new FeeRequest("EOS", "100"), new FeeRequest("ETH", "100")));
+  //   if (fees.size() != 2) {
+  //     fail("invalid amount of fees");
+  //   }
+  //   fees.forEach(Checker.checkFee);
+  // }
+
+  // @Test
+  // public void testGetEstimateDepositFee() throws CryptomarketSDKException {
+  //   String estimate = client.getEstimateDepositFee("EOS", "100", null);
+  //   if (estimate.equals("")) {
+  //     fail();
+  //   }
+  // }
 
   @Test
   public void testCryptoAddressBelongsToCurrentAccount() throws CryptomarketSDKException {