drip-ruby 3.2.0 → 3.3.0

data/lib/drip/request.rb

@@ -0,0 +1,31 @@
+require "net/http"
+
+module Drip
+  class Request
+    attr_reader :http_verb, :url, :options, :content_type
+
+    VERB_CLASS_MAPPING = {
+      get: Net::HTTP::Get,
+      post: Net::HTTP::Post,
+      put: Net::HTTP::Put,
+      delete: Net::HTTP::Delete
+    }.freeze
+
+    def initialize(http_verb, url, options = {}, content_type = nil)
+      @http_verb = http_verb
+      @url = url
+      @options = options
+      @content_type = content_type
+    end
+
+    def verb_klass
+      VERB_CLASS_MAPPING[http_verb]
+    end
+
+    def body
+      return if http_verb == :get
+
+      options.to_json
+    end
+  end
+end

data/lib/drip/response.rb

@@ -44,20 +44,20 @@ module Drip
     end
 
     def parse_members
-      if status == 429
-        body
-      else
-        {}.tap do |members|
-          if body.is_a?(Hash)
-            body.each do |key, value|
-              klass = if value.is_a?(Array)
-                        Drip::Collections.find_class(key)
-                      else
-                        Drip::Resources.find_class(key)
-                      end
+      return body unless success?
+      {}.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
+            members[key] = klass.new(value)
           end
         end
       end

data/lib/drip/version.rb

@@ -1,3 +1,3 @@
 module Drip
-  VERSION = "3.2.0".freeze
+  VERSION = "3.3.0".freeze
 end

data/test/drip/client/configuration_test.rb

@@ -0,0 +1,122 @@
+require File.dirname(__FILE__) + '/../../test_helper.rb'
+require "drip/client/configuration"
+
+class Drip::Client::ConfigurationTest < Drip::TestCase
+  context "#initializer" do
+    context "with reasonable parameters" do
+      should "accept parameters" do
+        config = Drip::Client::Configuration.new(access_token: "123")
+        assert_equal "123", config.access_token
+      end
+    end
+
+    context "with one additional parameter" do
+      should "raise singular error" do
+        err = assert_raises(ArgumentError) { Drip::Client::Configuration.new(blahdeblah: "123") }
+        assert_equal "unknown keyword: blahdeblah", err.message
+      end
+
+      should "match core ruby behavior" do
+        def test_method(hello: "test"); end
+        err = assert_raises(ArgumentError) { test_method(blahdeblah: "123") }
+        assert_equal "unknown keyword: blahdeblah", err.message
+      end
+    end
+
+    context "with multiple additional parameters" do
+      should "raise plural error" do
+        err = assert_raises(ArgumentError) { Drip::Client::Configuration.new(blahdeblah: "123", blahdeblah1: "123") }
+        assert_equal "unknown keywords: blahdeblah, blahdeblah1", err.message
+      end
+
+      should "match core ruby behavior" do
+        def test_method(hello: "test"); end
+        err = assert_raises(ArgumentError) { test_method(blahdeblah: "123", blahdeblah1: "123") }
+        assert_equal "unknown keywords: blahdeblah, blahdeblah1", err.message
+      end
+    end
+  end
+
+  context "#url_prefix" do
+    should "have default url prefix" do
+      config = Drip::Client::Configuration.new
+      assert_equal "https://api.getdrip.com/", config.url_prefix
+    end
+
+    should "accept passed parameter" do
+      config = Drip::Client::Configuration.new(url_prefix: "https://www.example.com/")
+      assert_equal "https://www.example.com/", config.url_prefix
+    end
+
+    should "allow setter" do
+      config = Drip::Client::Configuration.new
+      config.url_prefix = "https://www.example.com/"
+      assert_equal "https://www.example.com/", config.url_prefix
+    end
+  end
+
+  context "#access_token" do
+    should "accept passed parameter" do
+      config = Drip::Client::Configuration.new(access_token: "blah")
+      assert_equal "blah", config.access_token
+    end
+
+    should "allow setter" do
+      config = Drip::Client::Configuration.new
+      config.access_token = "blah"
+      assert_equal "blah", config.access_token
+    end
+  end
+
+  context "#api_key" do
+    should "accept passed parameter" do
+      config = Drip::Client::Configuration.new(api_key: "blah")
+      assert_equal "blah", config.api_key
+    end
+
+    should "allow setter" do
+      config = Drip::Client::Configuration.new
+      config.api_key = "blah"
+      assert_equal "blah", config.api_key
+    end
+  end
+
+  context "#account_id" do
+    should "accept passed parameter" do
+      config = Drip::Client::Configuration.new(account_id: "1234567")
+      assert_equal "1234567", config.account_id
+    end
+
+    should "allow setter" do
+      config = Drip::Client::Configuration.new
+      config.account_id = "1234567"
+      assert_equal "1234567", config.account_id
+    end
+  end
+
+  context "#http_open_timeout" do
+    should "accept passed parameter" do
+      config = Drip::Client::Configuration.new(http_open_timeout: 12)
+      assert_equal 12, config.http_open_timeout
+    end
+
+    should "allow setter" do
+      config = Drip::Client::Configuration.new
+      config.http_open_timeout = 12
+      assert_equal 12, config.http_open_timeout
+    end
+  end
+
+  context "#http_timeout" do
+    should "accept passed parameter" do
+      config = Drip::Client::Configuration.new(http_timeout: 42)
+      assert_equal 42, config.http_timeout
+    end
+
+    should "allow setter" do
+      config = Drip::Client::Configuration.new
+      config.http_timeout = 42
+      assert_equal 42, config.http_timeout
+    end
+  end
+end

data/test/drip/client/http_client_test.rb

@@ -0,0 +1,96 @@
+require File.dirname(__FILE__) + '/../../test_helper.rb'
+require "drip/client/configuration"
+require "drip/client/http_client"
+require "drip/request"
+
+class Drip::Client::HTTPClientTest < Drip::TestCase
+  context "given a personal api key" do
+    setup do
+      @key = "aaaa"
+      @config = Drip::Client::Configuration.new(api_key: @key)
+      @client = Drip::Client::HTTPClient.new(@config)
+    end
+
+    should "use Basic authentication" do
+      stub_request(:get, "https://api.getdrip.com/v2/testpath").
+        to_return(status: 200, body: "", headers: {})
+
+      @client.make_request(Drip::Request.new(:get, URI("https://api.getdrip.com/v2/testpath")))
+
+      header = "Basic #{Base64.encode64(@key + ':')}".strip
+      assert_requested :get, "https://api.getdrip.com/v2/testpath", headers: { 'Authorization' => header }
+    end
+  end
+
+  context "given a OAuth access token" do
+    setup do
+      @key = "aaaa"
+      @config = Drip::Client::Configuration.new(access_token: @key)
+      @client = Drip::Client::HTTPClient.new(@config)
+    end
+
+    should "use Bearer token authentication" do
+      stub_request(:get, "https://api.getdrip.com/v2/testpath").
+        to_return(status: 200, body: "", headers: {})
+      @client.make_request(Drip::Request.new(:get, URI("https://api.getdrip.com/v2/testpath")))
+      header = "Bearer #{@key}"
+      assert_requested :get, "https://api.getdrip.com/v2/testpath", headers: { 'Authorization' => header }
+    end
+  end
+
+  context "given a redirecting url" do
+    setup do
+      @config = Drip::Client::Configuration.new
+      @client = Drip::Client::HTTPClient.new(@config)
+    end
+
+    should "follow redirect" do
+      stub_request(:get, "https://api.getdrip.com/v2/testpath").
+        to_return(status: 301, body: "", headers: { "Location" => "https://api.example.com/mytestpath" })
+      stub_request(:get, "https://api.example.com/mytestpath").
+        to_return(status: 200, body: "{}")
+      response = @client.make_request(Drip::Request.new(:get, URI("https://api.getdrip.com/v2/testpath")))
+      assert_requested :get, "https://api.getdrip.com/v2/testpath"
+      assert_requested :get, "https://api.example.com/mytestpath"
+      assert_equal("{}", response.body)
+    end
+
+    should "not follow too many redirects" do
+      stub_request(:get, "https://api.getdrip.com/v2/accounts").
+        to_return(status: 301, body: "", headers: { "Location" => "https://api.example.com/mytestpath" })
+      stub_request(:get, "https://api.example.com/mytestpath").
+        to_return(status: 302, body: "", headers: { "Location" => "https://api.getdrip.com/v2/accounts" })
+      assert_raises(Drip::TooManyRedirectsError) { @client.make_request(Drip::Request.new(:get, URI("https://api.getdrip.com/v2/accounts"))) }
+      assert_requested :get, "https://api.getdrip.com/v2/accounts", times: 5
+      assert_requested :get, "https://api.example.com/mytestpath", times: 5
+    end
+  end
+
+  context "given a get request" do
+    setup do
+      @config = Drip::Client::Configuration.new(http_open_timeout: 20)
+      @client = Drip::Client::HTTPClient.new(@config)
+      @response = mock
+      @response.stubs(:code).returns('200')
+      @response.stubs(:body).returns('{}')
+
+      @http = mock
+      @http.expects(:request).returns(@response)
+
+      @request = mock
+      @request.stubs(:[]=)
+      @request.stubs(:basic_auth)
+    end
+
+    should "encode query and not set body" do
+      Net::HTTP::Get.expects(:new).returns(@request)
+      Net::HTTP.expects(:start).yields(@http).returns(@response)
+
+      @request.expects(:body=).with(nil)
+      URI.expects(:encode_www_form).once
+
+      response = @client.make_request(Drip::Request.new(:get, URI("https://api.getdrip.com/v2/testpath")))
+      assert_equal("{}", response.body)
+    end
+  end
+end

data/test/drip/client/shopper_activity_test.rb

@@ -0,0 +1,175 @@
+require File.dirname(__FILE__) + '/../../test_helper.rb'
+
+class Drip::Client::ShopperActivityTest < Drip::TestCase
+  def setup
+    @client = Drip::Client.new { |c| c.account_id = "12345" }
+  end
+
+  context "#create_cart_activity_event" do
+    setup do
+      @email = "drippy@drip.com"
+      @options = {
+        email: @email,
+        action: "created",
+        provider: "shopify",
+        cart_id: "abcdef",
+        cart_url: "https://www.example.com/mythingy",
+        amount: 4900,
+        tax: 100,
+        fees: 0,
+        discount: 0,
+        currency_code: "USD",
+        properties: {
+          "size" => "medium",
+          "color" => "red"
+        }
+      }
+      @response_status = 202
+      @response_body = "{}"
+
+      stub_request(:post, "https://api.getdrip.com/v3/12345/shopper_activity/cart").
+        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_cart_activity_event(@options)
+    end
+
+    should "return error when missing fields" do
+      @options.delete(:cart_id)
+      assert_raises(ArgumentError) { @client.create_cart_activity_event(@options) }
+    end
+  end
+
+  context "#create_order_activity_event" do
+    setup do
+      @email = "drippy@drip.com"
+      @options = {
+        email: @email,
+        action: "created",
+        provider: "shopify",
+        order_id: "abcdef",
+        amount: 4900,
+        tax: 100,
+        fees: 0,
+        discount: 0,
+        currency_code: "USD",
+        properties: {
+          "size" => "medium",
+          "color" => "red"
+        }
+      }
+      @response_status = 202
+      @response_body = "{}"
+
+      stub_request(:post, "https://api.getdrip.com/v3/12345/shopper_activity/order").
+        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_order_activity_event(@options)
+    end
+
+    should "return error when missing fields" do
+      @options.delete(:order_id)
+      assert_raises(ArgumentError) { @client.create_order_activity_event(@options) }
+    end
+  end
+
+  context "#create_order_activity_events" do
+    setup do
+      @records = [
+        {
+          email: "drippy@example.com",
+          action: "created",
+          provider: "shopify",
+          order_id: "abcdef",
+          amount: 4900,
+          tax: 100,
+          fees: 0,
+          discount: 0,
+          currency_code: "USD",
+          properties: {
+            "size" => "medium",
+            "color" => "red"
+          }
+        },
+        {
+          email: "drippy1@example.com",
+          action: "created",
+          provider: "shopify",
+          order_id: "fdsgs",
+          amount: 4900,
+          tax: 100,
+          fees: 0,
+          discount: 0,
+          currency_code: "USD",
+          properties: {
+            "size" => "medium",
+            "color" => "red"
+          }
+        }
+      ]
+      @response_status = 202
+      @response_body = "{}"
+
+      stub_request(:post, "https://api.getdrip.com/v3/12345/shopper_activity/order/batch").
+        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_order_activity_events(@records)
+    end
+
+    should "return error when missing fields" do
+      @records[1].delete(:order_id)
+      err = assert_raises(ArgumentError) { @client.create_order_activity_events(@records) }
+      assert_equal "order_id: parameter required in record 1", err.message
+    end
+  end
+
+  context "#create_product_activity_event" do
+    setup do
+      @email = "drippy@drip.com"
+      @options =  {
+        provider: "my_custom_platform",
+        action: "created",
+        occurred_at: "2019-01-28T12:15:23Z",
+        product_id: "B01J4SWO1G",
+        product_variant_id: "B01J4SWO1G-CW-BOTT",
+        sku: "XHB-1234",
+        name: "The Coolest Water Bottle",
+        brand: "Drip",
+        categories: [
+          "Accessories"
+        ],
+        price: 11.16,
+        inventory: 42,
+        product_url: "https://mysuperstore.example.com/dp/B01J4SWO1G",
+        image_url: "https://www.getdrip.com/images/example_products/water_bottle.png"
+      }
+      @response_status = 202
+      @response_body = "{}"
+
+      stub_request(:post, "https://api.getdrip.com/v3/12345/shopper_activity/product").
+        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_product_activity_event(@options)
+    end
+
+    should "return error when missing fields" do
+      @options.delete(:product_id)
+      assert_raises(ArgumentError) { @client.create_product_activity_event(@options) }
+    end
+  end
+end