Merge pull request #272 from EasyPost/fix_custom_info_create_custom_items

fix: customs_info wrapping for customs_items

Author: Justintime50

CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## v5.0.1 (2023-06-20)
+
+- Fixes a bug where the params of a `customs_info` on create weren't wrapped properly which led to an empty set of `customs_items`
+
 ## v5.0.0 (2023-06-06)
 
 See our [Upgrade Guide](UPGRADE_GUIDE.md#upgrading-from-4x-to-50) for more details.

VERSION

@@ -1 +1 @@
-5.0.0
+5.0.1

lib/easypost/http_client.rb

@@ -43,7 +43,10 @@ def default_request_execute(method, uri, headers, open_timeout, read_timeout, bo
       # Attempt to make the request and return the response.
       Net::HTTP.start(
         uri.host,
-        uri.port, use_ssl: true, read_timeout: read_timeout, open_timeout: open_timeout,
+        uri.port,
+        use_ssl: true,
+        read_timeout: read_timeout,
+        open_timeout: open_timeout,
       ) do |http|
         http.request(request)
       end

lib/easypost/services/customs_info.rb

@@ -5,7 +5,9 @@ class EasyPost::Services::CustomsInfo < EasyPost::Services::Service
 
   # Create a CustomsInfo object
   def create(params)
-    @client.make_request(:post, 'customs_infos', MODEL_CLASS, params)
+    wrapped_params = { customs_info: params }
+
+    @client.make_request(:post, 'customs_infos', MODEL_CLASS, wrapped_params)
   end
 
   # Retrieve a CustomsInfo object

spec/cassettes/customs_info/EasyPost_Services_CustomsInfo_create_creates_a_customs_info.yml

@@ -10,6 +10,7 @@
       customs_info = client.customs_info.create(Fixture.basic_customs_info)
 
       expect(customs_info).to be_an_instance_of(EasyPost::Models::CustomsInfo)
+      expect(customs_info.customs_items).to all(be_an_instance_of(EasyPost::Models::CustomsItem))
       expect(customs_info.id).to match('cstinfo_')
       expect(customs_info.eel_pfc).to eq('NOEEI 30.37(a)')
     end