test: custom_headers webhooks

Author: Justintime50

CHANGELOG.md

@@ -3,7 +3,6 @@
 ## Next Release (minor)
 
 - Adds Ruby 3.4 support
-- Fixes the update webhook function to wrap parameters in the request correctly
 - Corrects the HTTP verb for updating a brand from `GET` to `PATCH`
 - Removes the deprecated `create_list` tracker endpoint function as it is no longer available via API
 

examples

@@ -27,8 +27,7 @@ def all(params = {})
 
   # Update a Webhook.
   def update(id, params = {})
-    wrapped_params = { webhook: params }
-    response = @client.make_request(:patch, "webhooks/#{id}", wrapped_params)
+    response = @client.make_request(:patch, "webhooks/#{id}", params)
 
     EasyPost::InternalUtilities::Json.convert_json_to_object(response, MODEL_CLASS)
   end

lib/easypost/services/webhook.rb

@@ -129,15 +129,19 @@ def self.event_bytes
   end
 
   def self.webhook_hmac_signature
-    read_fixture_data['webhook_hmac_signature']
+    read_fixture_data['webhooks']['hmac_signature']
   end
 
   def self.webhook_secret
-    read_fixture_data['webhook_secret']
+    read_fixture_data['webhooks']['secret']
   end
 
   def self.webhook_url
-    read_fixture_data['webhook_url']
+    read_fixture_data['webhooks']['url']
+  end
+
+  def self.webhook_custom_headers
+    read_fixture_data['webhooks']['custom_headers']
   end
 
   # The credit card details below are for a valid proxy card usable

spec/cassettes/webhook/EasyPost_Services_Webhook_create_creates_a_webhook.yml

@@ -10,11 +10,14 @@
       webhook = client.webhook.create(
         url: Fixture.webhook_url,
         webhook_secret: Fixture.webhook_secret,
+        custom_headers: Fixture.webhook_custom_headers,
       )
 
       expect(webhook).to be_an_instance_of(EasyPost::Models::Webhook)
       expect(webhook.id).to match('hook_')
       expect(webhook.url).to eq(Fixture.webhook_url)
+      expect(webhook.custom_headers[0].name).to eq('test')
+      expect(webhook.custom_headers[0].value).to eq('header')
 
       # Remove the webhook once we have tested it so we don't pollute the account with test webhooks
       client.webhook.delete(webhook.id)
@@ -56,9 +59,14 @@
         url: Fixture.webhook_url,
       )
 
-      updated_webhook = client.webhook.update(webhook.id, { webhook_secret: Fixture.webhook_secret })
+      updated_webhook = client.webhook.update(webhook.id, { 
+        webhook_secret: Fixture.webhook_secret, 
+        custom_headers: Fixture.webhook_custom_headers, 
+      })
 
       expect(updated_webhook).to be_an_instance_of(EasyPost::Models::Webhook)
+      expect(updated_webhook.custom_headers[0].name).to eq('test')
+      expect(updated_webhook.custom_headers[0].value).to eq('header')
 
       client.webhook.delete(webhook.id) # Remove the webhook once we have tested it so we don't pollute the account with test webhooks
     end