drip-ruby 3.4.2 → 3.5.0

data/lib/drip/client/shopper_activity.rb

@@ -11,7 +11,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://developer.drip.com/#cart-activity
       def create_cart_activity_event(data = {})
-        raise ArgumentError, 'email:, person_id:, or :visitor_uuid parameter required' if !data.key?(:email) && !data.key?(:person_id) && !data.key?(:visitor_uuid)
+        raise ArgumentError, 'email:, person_id:, or :visitor_uuid parameter required' if missing_cart_identifier?(data)
 
         %i[provider action cart_id cart_url].each do |key|
           raise ArgumentError, "#{key}: parameter required" unless data.key?(key)
@@ -47,10 +47,12 @@ module Drip
       # Returns a Drip::Response.
       # See https://developer.drip.com/#create-or-update-a-batch-of-orders
       def create_order_activity_events(records = [])
+        required_keys = %i[provider action order_id]
+
         records.each_with_index do |record, i|
           raise ArgumentError, "email: or person_id: parameter required in record #{i}" if !record.key?(:email) && !record.key?(:person_id)
 
-          %i[provider action order_id].each do |key|
+          required_keys.each do |key|
             raise ArgumentError, "#{key}: parameter required in record #{i}" unless record.key?(key)
           end
 
@@ -75,6 +77,28 @@ module Drip
         data[:occurred_at] = Time.now.iso8601 unless data.key?(:occurred_at)
         make_json_request :post, "v3/#{account_id}/shopper_activity/product", data
       end
+
+      # Public: Create a checkout activity event.
+      #
+      # options    - Required. A Hash of additional checkout options. Refer to the
+      #                       Drip API docs for the required schema.
+      #
+      # Returns a Drip::Response.
+      # See https://developer.drip.com/#checkout-activity
+      def create_checkout_activity_event(data = {})
+        %i[provider action checkout_id].each do |key|
+          raise ArgumentError, "#{key}: parameter required" unless data.key?(key)
+        end
+
+        data[:occurred_at] = Time.now.iso8601 unless data.key?(:occurred_at)
+        make_json_request :post, "v3/#{account_id}/shopper_activity/checkout", data
+      end
+    end
+
+  private
+
+    def missing_cart_identifier?(data)
+      !data.key?(:email) && !data.key?(:person_id) && !data.key?(:visitor_uuid)
     end
   end
 end

data/lib/drip/client/subscribers.rb

@@ -57,7 +57,8 @@ module Drip
         data = {}
         data[:email] = args[0] if args[0].is_a? String
         data.merge!(args.last) if args.last.is_a? Hash
-        raise ArgumentError, 'email: or id: or bigcommerce_subscriber_id: parameter required' if missing_subscriber_identifier(data)
+        raise ArgumentError, 'email: or id: or bigcommerce_subscriber_id: parameter required' if missing_subscriber_identifier?(data)
+
         make_json_api_request :post, "v2/#{account_id}/subscribers", private_generate_resource("subscribers", data)
       end
 
@@ -104,7 +105,7 @@ module Drip
       # See https://www.getdrip.com/docs/rest-api#unsubscribe
       def unsubscribe(id_or_email, options = {})
         url = "v2/#{account_id}/subscribers/#{CGI.escape id_or_email}/remove"
-        url += options[:campaign_id] ? "?campaign_id=#{options[:campaign_id]}" : ""
+        url += options[:campaign_id] ? "?campaign_id=#{CGI.escape options[:campaign_id].to_s}" : ""
         make_json_api_request :post, url
       end
 
@@ -133,7 +134,7 @@ module Drip
       # See https://www.getdrip.com/docs/rest-api#subscribe
       def subscribe(email, campaign_id, options = {})
         data = options.merge("email" => email)
-        url = "v2/#{account_id}/campaigns/#{campaign_id}/subscribers"
+        url = "v2/#{account_id}/campaigns/#{CGI.escape campaign_id.to_s}/subscribers"
         make_json_api_request :post, url, private_generate_resource("subscribers", data)
       end
 
@@ -160,7 +161,7 @@ module Drip
 
   private
 
-    def missing_subscriber_identifier(data)
+    def missing_subscriber_identifier?(data)
       external_ids = data[:external_ids] || {}
       !data.key?(:email) && !data.key?(:id) && !external_ids.key?("bigcommerce_subscriber_id")
     end

data/lib/drip/client/webhooks.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "cgi"
+
 module Drip
   class Client
     module Webhooks
@@ -17,7 +19,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#webhooks
       def webhook(id)
-        make_json_api_request :get, "v2/#{account_id}/webhooks/#{id}"
+        make_json_api_request :get, "v2/#{account_id}/webhooks/#{CGI.escape id.to_s}"
       end
 
       # Public: Create a webhook.
@@ -54,7 +56,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#webhooks
       def delete_webhook(id)
-        make_json_api_request :delete, "v2/#{account_id}/webhooks/#{id}"
+        make_json_api_request :delete, "v2/#{account_id}/webhooks/#{CGI.escape id.to_s}"
       end
     end
   end

data/lib/drip/client/workflow_triggers.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "cgi"
+
 module Drip
   class Client
     module WorkflowTriggers
@@ -9,7 +11,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#workflow_triggers
       def workflow_triggers(id)
-        make_json_api_request :get, "v2/#{account_id}/workflows/#{id}/triggers"
+        make_json_api_request :get, "v2/#{account_id}/workflows/#{CGI.escape id.to_s}/triggers"
       end
 
       # Public: Create a workflow trigger.
@@ -24,7 +26,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#workflows
       def create_workflow_trigger(id, options = {})
-        make_json_api_request :post, "v2/#{account_id}/workflows/#{id}/triggers", private_generate_resource("triggers", options)
+        make_json_api_request :post, "v2/#{account_id}/workflows/#{CGI.escape id.to_s}/triggers", private_generate_resource("triggers", options)
       end
 
       # Public: Update a workflow trigger.
@@ -39,7 +41,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#workflows
       def update_workflow_trigger(id, options = {})
-        make_json_api_request :put, "v2/#{account_id}/workflows/#{id}/triggers", private_generate_resource("triggers", options)
+        make_json_api_request :put, "v2/#{account_id}/workflows/#{CGI.escape id.to_s}/triggers", private_generate_resource("triggers", options)
       end
     end
   end

data/lib/drip/client/workflows.rb

@@ -23,7 +23,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#workflows
       def workflow(id)
-        make_json_api_request :get, "v2/#{account_id}/workflows/#{id}"
+        make_json_api_request :get, "v2/#{account_id}/workflows/#{CGI.escape id.to_s}"
       end
 
       # Public: Activate a workflow.
@@ -32,7 +32,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#workflows
       def activate_workflow(id)
-        make_json_api_request :post, "v2/#{account_id}/workflows/#{id}/activate"
+        make_json_api_request :post, "v2/#{account_id}/workflows/#{CGI.escape id.to_s}/activate"
       end
 
       # Public: Pause a workflow.
@@ -41,7 +41,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#workflows
       def pause_workflow(id)
-        make_json_api_request :post, "v2/#{account_id}/workflows/#{id}/pause"
+        make_json_api_request :post, "v2/#{account_id}/workflows/#{CGI.escape id.to_s}/pause"
       end
 
       # Public: Start someone on a workflow.
@@ -65,7 +65,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#workflows
       def start_subscriber_workflow(id, options = {})
-        make_json_api_request :post, "v2/#{account_id}/workflows/#{id}/subscribers", private_generate_resource("subscribers", options)
+        make_json_api_request :post, "v2/#{account_id}/workflows/#{CGI.escape id.to_s}/subscribers", private_generate_resource("subscribers", options)
       end
 
       # Public: Remove someone from a workflow.
@@ -75,7 +75,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#workflows
       def remove_subscriber_workflow(workflow_id, id_or_email)
-        make_json_api_request :delete, "v2/#{account_id}/workflows/#{workflow_id}/subscribers/#{CGI.escape id_or_email}"
+        make_json_api_request :delete, "v2/#{account_id}/workflows/#{CGI.escape workflow_id.to_s}/subscribers/#{CGI.escape id_or_email}"
       end
     end
   end

data/lib/drip/client.rb

@@ -49,7 +49,7 @@ module Drip
         @config.public_send(config_key)
       end
 
-      setter_name = "#{config_key}=".to_sym
+      setter_name = :"#{config_key}="
       define_method(setter_name) do |val|
         warn "[DEPRECATED] Setting configuration on Drip::Client after initialization will be removed in a future version"
         @config.public_send(setter_name, val)
@@ -77,7 +77,7 @@ module Drip
       JSON_API_CONTENT_TYPE
     end
 
-    Drip::Request::VERB_CLASS_MAPPING.keys.each do |verb|
+    Drip::Request::VERB_CLASS_MAPPING.each_key do |verb|
       define_method(verb) do |path, options = {}|
         warn "[DEPRECATED] Drip::Client##{verb} please use the API endpoint specific methods"
         make_json_api_request(verb, "v2/#{path}", options)
@@ -109,7 +109,7 @@ module Drip
       end
     end
 
-    def build_response(&block)
+    def build_response(&)
       response = yield
       Drip::Response.new(response.code.to_i, response.body || response.body == "" ? JSON.parse(response.body) : nil)
     rescue JSON::ParserError

data/lib/drip/collection.rb

@@ -34,8 +34,8 @@ module Drip
       items.length < 2
     end
 
-    def each(&block)
-      items.each { |item| yield(item) }
+    def each(&)
+      items.each(&)
     end
   end
 end

data/lib/drip/resource.rb

@@ -19,26 +19,26 @@ module Drip
       true
     end
 
-    def respond_to?(method_name, include_private = false)
-      attributes.keys.include?(method_name.to_s) || super
+    def respond_to_missing?(method_name, include_private = false)
+      attributes.key?(method_name.to_s) || super
     end
 
-    def method_missing(method_name, *args, &block)
-      attributes.keys.include?(method_name.to_s) ? attributes[method_name.to_s] : super
+    def method_missing(method_name, *args, &)
+      attributes.key?(method_name.to_s) ? attributes[method_name.to_s] : super
     end
 
   private
 
     def process(attributes)
       {}.tap do |attrs|
-        attributes.keys.each do |key|
+        attributes.each_key do |key|
           attrs[key] = process_attribute(key, attributes[key])
         end
       end
     end
 
     def process_attribute(key, raw_value)
-      if key.to_s =~ /_at$/ # auto-coerce times
+      if key.to_s.match?(/_at$/) # auto-coerce times
         raw_value ? Time.parse(raw_value) : nil
       else
         raw_value

data/lib/drip/resources/tag.rb

@@ -7,10 +7,5 @@ module Drip
     def self.resource_name
       "tag"
     end
-
-    def initialize(raw_data = {})
-      @raw_attributes = raw_data.dup.freeze
-      @attributes = @raw_attributes
-    end
   end
 end

data/lib/drip/response.rb

@@ -26,12 +26,13 @@ module Drip
       (200..299).cover?(status)
     end
 
-    def respond_to?(method_name, include_private = false)
-      member_map.keys.include?(method_name) || super
+    def respond_to_missing?(method_name, include_private = false)
+      member_map.key?(method_name) || super
     end
 
-    def method_missing(method_name, *args, &block)
-      return super unless member_map.keys.include?(method_name)
+    def method_missing(method_name, *args, &)
+      return super unless member_map.key?(method_name)
+
       members[member_map[method_name]]
     end
 
@@ -47,27 +48,29 @@ module Drip
 
     def parse_members
       return body unless success?
+      return {} unless body.is_a?(Hash)
+
       {}.tap do |members|
-        if body.is_a?(Hash)
-          body.each do |key, value|
-            klass = case value
-                    when Array
-                      Drip::Collections.find_class(key)
-                    when String
-                      String
-                    else
-                      Drip::Resources.find_class(key)
-                    end
-
-            members[key] = klass.new(value)
-          end
+        body.each do |key, value|
+          members[key] = member_class(key, value).new(value)
         end
       end
     end
 
+    def member_class(key, value)
+      case value
+      when Array
+        Drip::Collections.find_class(key)
+      when String
+        String
+      else
+        Drip::Resources.find_class(key)
+      end
+    end
+
     def member_map
       @member_map ||= {}.tap do |map|
-        members.each { |key, _value| map[key.to_sym] = key }
+        members.each_key { |key| map[key.to_sym] = key }
       end
     end
   end

data/lib/drip/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Drip
-  VERSION = "3.4.2"
+  VERSION = "3.5.0"
 end

data/test/drip/client/accounts_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 
 class Drip::Client::AccountsTest < Drip::TestCase
   def setup
@@ -26,7 +26,7 @@ class Drip::Client::AccountsTest < Drip::TestCase
     setup do
       @response_status = 200
       @response_body = "{}"
-      @id = 9999999
+      @id = 9_999_999
 
       stub_request(:get, "https://api.getdrip.com/v2/accounts/#{@id}").
         to_return(status: @response_status, body: @response_body, headers: {})
@@ -37,4 +37,20 @@ class Drip::Client::AccountsTest < Drip::TestCase
       assert_equal expected, @client.account(@id)
     end
   end
+
+  context "#account with an id that needs escaping" do
+    setup do
+      @response_status = 200
+      @response_body = "{}"
+      @id = "abc/123 xyz"
+
+      stub_request(:get, "https://api.getdrip.com/v2/accounts/#{CGI.escape @id}").
+        to_return(status: @response_status, body: @response_body, headers: {})
+    end
+
+    should "escape the id in the request path" do
+      expected = Drip::Response.new(@response_status, JSON.parse(@response_body))
+      assert_equal expected, @client.account(@id)
+    end
+  end
 end

data/test/drip/client/broadcasts_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 
 class Drip::Client::BroadcastsTest < Drip::TestCase
   def setup
@@ -26,7 +26,7 @@ class Drip::Client::BroadcastsTest < Drip::TestCase
     setup do
       @response_status = 200
       @response_body = "{}"
-      @id = 99999
+      @id = 99_999
 
       stub_request(:get, "https://api.getdrip.com/v2/12345/broadcasts/#{@id}").
         to_return(status: @response_status, body: @response_body, headers: {})

data/test/drip/client/campaign_subscriptions_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 
 class Drip::Client::CampaignSubscriptionsTest < Drip::TestCase
   def setup

data/test/drip/client/campaigns_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 
 class Drip::Client::CampaignsTest < Drip::TestCase
   def setup
@@ -26,7 +26,7 @@ class Drip::Client::CampaignsTest < Drip::TestCase
     setup do
       @response_status = 200
       @response_body = "{}"
-      @id = 9999999
+      @id = 9_999_999
 
       stub_request(:get, "https://api.getdrip.com/v2/12345/campaigns/#{@id}").
         to_return(status: @response_status, body: @response_body, headers: {})
@@ -42,7 +42,7 @@ class Drip::Client::CampaignsTest < Drip::TestCase
     setup do
       @response_status = 204
       @response_body = nil
-      @id = 9999999
+      @id = 9_999_999
 
       stub_request(:post, "https://api.getdrip.com/v2/12345/campaigns/#{@id}/activate").
         to_return(status: @response_status, body: @response_body, headers: {})
@@ -58,7 +58,7 @@ class Drip::Client::CampaignsTest < Drip::TestCase
     setup do
       @response_status = 204
       @response_body = nil
-      @id = 9999999
+      @id = 9_999_999
 
       stub_request(:post, "https://api.getdrip.com/v2/12345/campaigns/#{@id}/pause").
         to_return(status: @response_status, body: @response_body, headers: {})
@@ -74,7 +74,7 @@ class Drip::Client::CampaignsTest < Drip::TestCase
     setup do
       @response_status = 200
       @response_body = "{}"
-      @id = 9999999
+      @id = 9_999_999
 
       stub_request(:get, "https://api.getdrip.com/v2/12345/campaigns/#{@id}/subscribers").
         to_return(status: @response_status, body: @response_body, headers: {})

data/test/drip/client/configuration_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 require "drip/client/configuration"
 
 class Drip::Client::ConfigurationTest < Drip::TestCase
@@ -109,4 +109,17 @@ class Drip::Client::ConfigurationTest < Drip::TestCase
       assert_equal 42, config.http_timeout
     end
   end
+
+  context "#skip_analytics" do
+    should "accept passed parameter" do
+      config = Drip::Client::Configuration.new(skip_analytics: true)
+      assert_equal true, config.skip_analytics
+    end
+
+    should "allow setter" do
+      config = Drip::Client::Configuration.new
+      config.skip_analytics = true
+      assert_equal true, config.skip_analytics
+    end
+  end
 end

data/test/drip/client/conversions_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 
 class Drip::Client::ConversionsTest < Drip::TestCase
   def setup
@@ -26,7 +26,7 @@ class Drip::Client::ConversionsTest < Drip::TestCase
     setup do
       @response_status = 200
       @response_body = "{}"
-      @id = 9999999
+      @id = 9_999_999
 
       stub_request(:get, "https://api.getdrip.com/v2/12345/goals/#{@id}").
         to_return(status: @response_status, body: @response_body, headers: {})

data/test/drip/client/custom_fields_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 
 class Drip::Client::CustomFieldsTest < Drip::TestCase
   def setup

data/test/drip/client/events_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 
 class Drip::Client::EventsTest < Drip::TestCase
   def setup

data/test/drip/client/forms_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 
 class Drip::Client::FormsTest < Drip::TestCase
   def setup
@@ -26,7 +26,7 @@ class Drip::Client::FormsTest < Drip::TestCase
     setup do
       @response_status = 200
       @response_body = "{}"
-      @id = 9999999
+      @id = 9_999_999
 
       stub_request(:get, "https://api.getdrip.com/v2/12345/forms/#{@id}").
         to_return(status: @response_status, body: @response_body, headers: {})

data/test/drip/client/http_client_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 require "drip/client/configuration"
 require "drip/client/http_client"
 require "drip/request"
@@ -19,7 +19,7 @@ class Drip::Client::HTTPClientTest < Drip::TestCase
 
       @client.make_request(Drip::Request.new(:get, URI("https://api.getdrip.com/v2/testpath")))
 
-      header = "Basic #{Base64.encode64(@key + ':')}".strip
+      header = "Basic #{Base64.encode64("#{@key}:")}".strip
       assert_requested :get, "https://api.getdrip.com/v2/testpath", headers: { 'Authorization' => header }
     end
   end
@@ -68,6 +68,36 @@ class Drip::Client::HTTPClientTest < Drip::TestCase
     end
   end
 
+  context "given a redirect to a different host" do
+    setup do
+      @config = Drip::Client::Configuration.new(api_key: "secret-key")
+      @client = Drip::Client::HTTPClient.new(@config)
+    end
+
+    should "not forward credentials to the redirect target" do
+      stub_request(:get, "https://api.getdrip.com/v2/testpath").
+        to_return(status: 301, body: "", headers: { "Location" => "https://evil.example.com/steal" })
+      stub_request(:get, "https://evil.example.com/steal").
+        to_return(status: 200, body: "{}")
+
+      @client.make_request(Drip::Request.new(:get, URI("https://api.getdrip.com/v2/testpath")))
+
+      assert_requested(:get, "https://api.getdrip.com/v2/testpath") { |req| req.headers.key?("Authorization") }
+      assert_requested(:get, "https://evil.example.com/steal") { |req| !req.headers.key?("Authorization") }
+    end
+
+    should "still forward credentials when the redirect stays on the same host" do
+      stub_request(:get, "https://api.getdrip.com/v2/testpath").
+        to_return(status: 301, body: "", headers: { "Location" => "https://api.getdrip.com/v2/othertestpath" })
+      stub_request(:get, "https://api.getdrip.com/v2/othertestpath").
+        to_return(status: 200, body: "{}")
+
+      @client.make_request(Drip::Request.new(:get, URI("https://api.getdrip.com/v2/testpath")))
+
+      assert_requested(:get, "https://api.getdrip.com/v2/othertestpath") { |req| req.headers.key?("Authorization") }
+    end
+  end
+
   context "given a get request" do
     setup do
       @config = Drip::Client::Configuration.new(http_open_timeout: 20)

data/test/drip/client/orders_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 
 class Drip::Client::OrdersTest < Drip::TestCase
   def setup

data/test/drip/client/shopper_activity_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.dirname(__FILE__) + '/../../test_helper.rb'
+require_relative '../../test_helper'
 
 class Drip::Client::ShopperActivityTest < Drip::TestCase
   def setup
@@ -186,4 +186,38 @@ class Drip::Client::ShopperActivityTest < Drip::TestCase
       assert_raises(ArgumentError) { @client.create_product_activity_event(@options) }
     end
   end
+
+  context "#create_checkout_activity_event" do
+    setup do
+      @email = "drippy@drip.com"
+      @options = {
+        email: @email,
+        provider: "shopify",
+        action: "created",
+        occurred_at: "2019-01-28T12:15:23Z",
+        checkout_id: "B01J4",
+        total_price: 11.16
+      }
+      @response_status = 202
+      @response_body = "{}"
+
+      stub_request(:post, "https://api.getdrip.com/v3/12345/shopper_activity/checkout").
+        with(headers: { "Content-Type" => "application/json" }).
+        to_return(status: @response_status, body: @response_body, headers: {})
+    end
+
+    should "send the right request" do
+      expected = Drip::Response.new(@response_status, JSON.parse(@response_body))
+      assert_equal expected, @client.create_checkout_activity_event(@options)
+    end
+
+    should "return error with missing fields" do
+      %i[provider action checkout_id].each do |key|
+        params = @options.dup
+        params.delete(key)
+
+        assert_raises(ArgumentError) { @client.create_checkout_activity_event(params) }
+      end
+    end
+  end
 end