add update params

Author: jchen293

CHANGELOG.md

@@ -8,6 +8,7 @@
   - `BetaReferralCustomerService.create_bank_account_client_secret`
   - `ReferralCustomerService.add_credit_card_from_stripe`
   - `ReferralCustomerService.add_bank_account_from_stripe`
+- Routes `AmazonShippingAccount` create requests to the new `/register_oauth` endpoint
 - Fixes the payload wrapping for updating a webhook
 - Removes deprecated `user.all_api_keys` and `user.api_keys`, use `api_key.all` and `api_key.retrieve_api_keys_for_user` respectively
 - Bumps all dev dependencies
@@ -16,7 +17,6 @@
 
 - Adds `tracking_codes` as a parameter of the `all` method on the TrackerService
 - Removes the deprecated `create_list` tracker endpoint function as it is no longer available via API
-- Routes `AmazonShippingAccount` create/update requests to the new `/register_oauth` endpoint
 
 ## v9.4.1 (2024-08-09)
 

easypost/services/base_service.py

@@ -85,6 +85,7 @@ def _update_resource(
         """Update an EasyPost object via the EasyPost API."""
         url = self._instance_url(class_name, id)
         wrapped_params = {self._snakecase_name(class_name): params}
+
         response = Requestor(self._client).request(method=method, url=url, params=wrapped_params, beta=beta)
 
         return convert_to_easypost_object(response=response)

easypost/services/carrier_account_service.py

@@ -61,7 +61,7 @@ def update(self, id: str, **params) -> CarrierAccount:
             response = Requestor(self._client).request(
                 method=RequestMethod.PATCH,
                 url=f"/carrier_accounts/register_oauth/{id}",
-                params={},
+                params={"carrier_account_oauth_registrations": params},
             )
             return convert_to_easypost_object(response=response)
         else:

tests/cassettes/test_carrier_account_update_amazon_shipping.yaml

@@ -157,21 +157,23 @@ def test_carrier_account_create_amazon_shipping(prod_client):
 
 
 @pytest.mark.vcr()
-def test_carrier_account_update_amazon_shipping(prod_client, synchronous_sleep_seconds):
+def test_carrier_account_update_amazon_shipping(prod_client):
     """Test updating an Amazon Shipping Carrier Account which uses a different URL and schema."""
-    function_name = inspect.stack()[0][3]
     params = {
         "type": "AmazonShippingAccount",
     }
 
     amazon_shipping_account = prod_client.carrier_account.create(**params)
-    if not os.path.exists(os.path.join("tests", "cassettes", f"{function_name}.yaml")):
-        time.sleep(synchronous_sleep_seconds)  # Wait enough time for updating the carrier account
-    updated_amazon_shipping_account = prod_client.carrier_account.update(amazon_shipping_account.id)
+
+    # TODO: Re-record this cassettes and add two assertions for description and reference when we made the changes in API level for update endpoint
+    updated_amazon_shipping_account = prod_client.carrier_account.update(
+        amazon_shipping_account.id,
+        description="test description",
+        reference="test reference",
+    )
 
     assert isinstance(updated_amazon_shipping_account, CarrierAccount)
     assert str.startswith(updated_amazon_shipping_account.id, "ca_")
-    assert amazon_shipping_account.updated_at != updated_amazon_shipping_account.updated_at
 
     prod_client.carrier_account.delete(
         amazon_shipping_account.id