chore: remove deprecated functions

Author: Justintime50

CHANGELOG.md

@@ -8,6 +8,13 @@
   - Removed the unused `Error` model
   - Added an explicit `AddressVerificationFieldError` model
   - The `BetaPaymentRefund` now uses a list of `FieldError` instead of `Error` for the `errors` field
+- Removes deprecated functions
+  - `TimeInTransit.getSmartRateAccuracy` (use `TimeInTransit.getSmartrateAccuracy` instead)
+  - `paymentMethod.all` (use `billing.retrievePaymentMethods` instead)
+  - `shipment.getSmartrates` (use `shipment.smartrates` instead)
+  - String overload for `shipment.lowestSmartRate`, 3rd param requires a valid `SmartrateAccuracy`
+  - `user.apiKeys` (use `apiKey.retrieveApiKeysForUser` instead)
+  - `utilities.getLowestSmartRate` (use `utilities.findLowestSmartrate` instead)
 - Corrects payload wrapping for updating a webhook
 - Bumps dependencies
 

src/main/java/com/easypost/model/TimeInTransit.java

@@ -24,37 +24,6 @@ public final class TimeInTransit {
     @SerializedName("percentile_99")
     private Integer percentile99;
 
-    /**
-     * Get the delivery accuracy of a specific percentile of this TimeInTransit.
-     *
-     * @param percentile the percentile to find the corresponding accuracy for
-     * @return the delivery accuracy of the specified percentile
-     * @throws EasyPostException when the percentile is not valid
-     * @deprecated Use {@link #getBySmartrateAccuracy(SmartrateAccuracy)} instead.
-     * Deprecated: v5.5.0 - v7.0.0
-     */
-    @Deprecated
-    public int getSmartRateAccuracy(final String percentile) throws EasyPostException {
-        switch (percentile) {
-            case "percentile_50":
-                return this.percentile50;
-            case "percentile_75":
-                return this.percentile75;
-            case "percentile_85":
-                return this.percentile85;
-            case "percentile_90":
-                return this.percentile90;
-            case "percentile_95":
-                return this.percentile95;
-            case "percentile_97":
-                return this.percentile97;
-            case "percentile_99":
-                return this.percentile99;
-            default:
-                throw new InvalidParameterError("percentile");
-        }
-    }
-
     /**
      * Get the delivery accuracy of a specific percentile of this TimeInTransit.
      *

src/main/java/com/easypost/service/PaymentMethodService.java

@@ -18,26 +18,4 @@ public class PaymentMethodService {
     PaymentMethodService(EasyPostClient client) {
         this.client = client;
     }
-
-    /**
-     * List all payment methods.
-     *
-     * @return Billing object.
-     * @throws EasyPostException when the request fails.
-     * @deprecated Use {@link com.easypost.service.BillingService#retrievePaymentMethods()} instead.
-     * Deprecated: v5.5.0 - v7.0.0
-     */
-    @Deprecated
-    public PaymentMethod all() throws EasyPostException {
-        String endpoint = "payment_methods";
-
-        PaymentMethod response =
-                Requestor.request(RequestMethod.GET, endpoint, null, PaymentMethod.class, client);
-
-        if (response.getId() == null) {
-            throw new InvalidObjectError(Constants.ErrorMessages.NO_PAYMENT_METHODS);
-        }
-
-        return response;
-    }
 }

src/main/java/com/easypost/service/ShipmentService.java

@@ -139,23 +139,6 @@ public Shipment newRates(final String id, final Map<String, Object> params) thro
         return Requestor.request(RequestMethod.POST, endpoint, params, Shipment.class, client);
     }
 
-    /**
-     * Get SmartRates for this Shipment.
-     *
-     * @param params The options for the query.
-     * @param id     The ID of shipment.
-     * @return List of SmartRate objects
-     * @throws EasyPostException when the request fails.
-     * @deprecated Use {@link #smartrates(String, Map)} instead.
-     * Deprecated: v5.5.0 - v7.0.0
-     */
-    @Deprecated
-    @InlineMe(replacement = "this.smartrates(id, params)")
-    public final List<SmartRate> getSmartrates(final String id, final Map<String, Object> params)
-        throws EasyPostException {
-        return this.smartrates(id, params);
-    }
-
     /**
      * Get SmartRate for this Shipment.
      *
@@ -300,28 +283,6 @@ public Shipment insure(final String id, final Map<String, Object> params) throws
         return Requestor.request(RequestMethod.POST, endpoint, params, Shipment.class, client);
     }
 
-    /**
-     * Get the lowest SmartRate for this Shipment.
-     *
-     * @param id               The ID of shipment.
-     * @param deliveryDay      Delivery days restriction to use when filtering.
-     * @param deliveryAccuracy Delivery days accuracy restriction to use when
-     *                         filtering.
-     * @return lowest SmartRate object
-     * @throws EasyPostException when the request fails.
-     * @deprecated use {@link #lowestSmartRate(String, int, SmartrateAccuracy)} instead.
-     * Deprecated: v5.5.0 - v7.0.0
-     */
-    @Deprecated
-    @InlineMe(
-        replacement = "this.lowestSmartRate(id, deliveryDay, SmartrateAccuracy.getByKeyName(deliveryAccuracy))",
-        imports = "com.easypost.model.SmartrateAccuracy"
-    )
-    public final SmartRate lowestSmartRate(final String id, int deliveryDay, String deliveryAccuracy)
-            throws EasyPostException {
-        return this.lowestSmartRate(id, deliveryDay, SmartrateAccuracy.getByKeyName(deliveryAccuracy));
-    }
-
     /**
      * Get the lowest SmartRate for this Shipment.
      *
@@ -341,43 +302,6 @@ public SmartRate lowestSmartRate(final String id, final int deliveryDay, Smartra
         return lowestSmartrate;
     }
 
-    /**
-     * Get SmartRates for this Shipment.
-     *
-     * @param id The ID of shipment.
-     * @return List of SmartRate objects
-     * @throws EasyPostException when the request fails.
-     * @deprecated Use {@link #smartrates(String, Map)} instead.
-     * Deprecated: v5.5.0 - v7.0.0
-     */
-    @Deprecated
-    @InlineMe(replacement = "this.smartrates(id, null)")
-    public final List<SmartRate> getSmartrates(final String id) throws EasyPostException {
-        return this.smartrates(id, null);
-    }
-
-    /**
-     * Get the lowest SmartRate from a list of SmartRates.
-     *
-     * @param smartRates       List of SmartRates to filter from.
-     * @param deliveryDay      Delivery days restriction to use when filtering.
-     * @param deliveryAccuracy Delivery days accuracy restriction to use when
-     *                         filtering.
-     * @return lowest SmartRate object
-     * @throws EasyPostException when the request fails.
-     * @deprecated Use {@link #findLowestSmartrate(List, int, SmartrateAccuracy)} instead.
-     * Deprecated: v5.5.0 - v7.0.0
-     */
-    @Deprecated
-    @InlineMe(replacement =
-        "this.findLowestSmartrate(smartRates, deliveryDay, SmartrateAccuracy.getByKeyName(deliveryAccuracy))",
-        imports = "com.easypost.model.SmartrateAccuracy"
-    )
-    public final SmartRate getLowestSmartRate(final List<SmartRate> smartRates,
-            int deliveryDay, String deliveryAccuracy) throws EasyPostException {
-        return findLowestSmartrate(smartRates, deliveryDay, SmartrateAccuracy.getByKeyName(deliveryAccuracy));
-    }
-
     /**
      * Find the lowest SmartRate from a list of SmartRates.
      *

src/main/java/com/easypost/service/UserService.java

@@ -102,30 +102,6 @@ public void delete(final String id) throws EasyPostException {
         Requestor.request(RequestMethod.DELETE, endpoint, null, User.class, client);
     }
 
-    /**
-     * Get this User's API keys.
-     *
-     * @deprecated Use {@link ApiKeyService#retrieveApiKeysForUser(String)} instead.
-     * @param id The ID of the user.
-     * @return List of ApiKey objects.
-     * @throws EasyPostException when the request fails.
-     */
-    public List<ApiKey> apiKeys(final String id) throws EasyPostException {
-        ApiKeys parentKeys = client.apiKey.all();
-
-        if (Objects.equals(id, parentKeys.getId())) {
-            return parentKeys.getKeys();
-        }
-
-        for (int i = 0; i < parentKeys.getChildren().size(); i++) {
-            if (id.equals(parentKeys.getChildren().get(i).getId())) {
-                return parentKeys.getChildren().get(i).getKeys();
-            }
-        }
-
-        throw new FilteringError(String.format(Constants.ErrorMessages.NO_OBJECT_FOUND, "API keys"));
-    }
-
     /**
      * Update the user brand.
      *

src/main/java/com/easypost/utils/Utilities.java

@@ -139,27 +139,6 @@ public static Event validateWebhook(byte[] eventBody, Map<String, Object> header
         }
     }
 
-    /**
-     * Get the lowest Smartrate from a list of Smartrates.
-     *
-     * @param smartrates       List of Smartrates to filter from.
-     * @param deliveryDay      Delivery days restriction to use when filtering.
-     * @param deliveryAccuracy Delivery days accuracy restriction to use when
-     *                         filtering.
-     * @return lowest Smartrate object
-     * @throws EasyPostException when the request fails.
-     * @deprecated Use {@link #findLowestSmartrate(List, int, SmartrateAccuracy)} instead.
-     * Deprecated: v5.5.0 - v7.0.0
-     */
-    @Deprecated
-    @InlineMe(replacement =
-        "Utilities.findLowestSmartrate(smartrates, deliveryDay, SmartrateAccuracy.getByKeyName(deliveryAccuracy))",
-        imports = {"com.easypost.model.SmartrateAccuracy", "com.easypost.utils.Utilities"})
-    public static SmartRate getLowestSmartRate(final List<SmartRate> smartrates, int deliveryDay,
-            String deliveryAccuracy) throws EasyPostException {
-        return findLowestSmartrate(smartrates, deliveryDay, SmartrateAccuracy.getByKeyName(deliveryAccuracy));
-    }
-
     /**
      * Find the lowest Smartrate from a list of Smartrates.
      *

src/test/java/com/easypost/ShipmentTest.java

@@ -392,9 +392,6 @@ public void testSmartRate() throws EasyPostException {
         assertNotNull(smartRate.getTimeInTransit().getPercentile95());
         assertNotNull(smartRate.getTimeInTransit().getPercentile97());
         assertNotNull(smartRate.getTimeInTransit().getPercentile99());
-
-        assertThrows(InvalidParameterError.class,
-                () -> smartRate.getTimeInTransit().getSmartRateAccuracy("percentile_100"));
     }
 
     /**
@@ -536,20 +533,6 @@ public void testInstanceLowestSmartRate() throws EasyPostException {
         assertThrows(EasyPostException.class, () -> {
             vcr.client.shipment.lowestSmartRate(shipment.getId(), 0, SmartrateAccuracy.Percentile90);
         });
-
-        SmartRate deprecatedLowestSmartRateFilters = vcr.client.shipment.lowestSmartRate(shipment.getId(), 3,
-                SmartrateAccuracy.Percentile90);
-
-        // Test lowest smartrate with valid filters
-        assertEquals("GroundAdvantage", deprecatedLowestSmartRateFilters.getService());
-        assertEquals(5.93, deprecatedLowestSmartRateFilters.getRate(), 0.01);
-        assertEquals("USPS", deprecatedLowestSmartRateFilters.getCarrier());
-
-        // Test lowest smartrate with invalid filters (should error due to strict
-        // delivery days)
-        assertThrows(EasyPostException.class, () -> {
-            vcr.client.shipment.lowestSmartRate(shipment.getId(), 0, SmartrateAccuracy.Percentile90);
-        });
     }
 
     /**