drip-ruby 3.1.1 → 3.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 34ab6a491e2d7b0bdf33b525a11c7d5499bd0e2f
-  data.tar.gz: d5b703ea0a8f113be93b5c19bbf13849f7fbb6b7
+SHA256:
+  metadata.gz: 187036a01190c508e9f62cdb1f0b3feed1f1f48f57b401abbb94f4c76e47f72b
+  data.tar.gz: 5f346605646162e11440de3094602e279d615595d3e3cd6ede2b3739a3893d35
 SHA512:
-  metadata.gz: ffb4cacb8da55651f34b33c9377de84104fa5913eb520f7437a2b0cf045eba8da1743043538c10ee00be64b562697128d22e35225307d5f4dee1c318c5022bdc
-  data.tar.gz: 3bce36166a411416f7258939b2283c403ec35903b882144538f95b893f56a61bd7bb3f90503ae61f7c318a834356ca073b594555fbc62f6185f4d8d6fa29908c
+  metadata.gz: 84d6861c2a98a80e42c385a84b8a09cca7f556029778acd9bd1e8bcc468b2fb85a09ebb3f4c6192ee9069617e3cb1c57ab26996cc5b04fc843db68144c7b583e
+  data.tar.gz: 454de61b3f91e50a7f2948b6698c8e824f8e8a702ff96fb482c2542c0d1437fea5fe051da856f19f9f1340676a5b48c33da3815f989ab7bdbac52829c9f8034c

data/.rubocop.yml

@@ -1,7 +1,7 @@
 inherit_from: .rubocop_todo.yml
 
 AllCops:
-  TargetRubyVersion: '2.1'
+  TargetRubyVersion: '2.3'
   Exclude:
     - 'test/fixtures/**/*'
 
@@ -28,3 +28,11 @@ Layout/MultilineOperationIndentation:
 
 Layout/MultilineMethodCallIndentation:
   EnforcedStyle: 'indented'
+
+Metrics/ClassLength:
+  Exclude:
+    - 'test/**/*'
+
+Metrics/BlockLength:
+  Exclude:
+    - 'test/**/*'

data/.rubocop_todo.yml

@@ -1,11 +1,18 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2018-05-24 10:21:33 -0500 using RuboCop version 0.56.0.
+# on 2019-05-02 16:45:21 -0500 using RuboCop version 0.67.2.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
+# Offense count: 2
+# Cop supports --auto-correct.
+Layout/EmptyLineAfterGuardClause:
+  Exclude:
+    - 'lib/drip/client/subscribers.rb'
+    - 'lib/drip/response.rb'
+
 # Offense count: 2
 # Cop supports --auto-correct.
 # Configuration parameters: AllowUnusedKeywordArguments, IgnoreEmptyMethods.
@@ -14,22 +21,18 @@ Lint/UnusedMethodArgument:
     - 'lib/drip/client.rb'
     - 'lib/drip/collection.rb'
 
-# Offense count: 1
+# Offense count: 2
 Metrics/AbcSize:
-  Max: 26
+  Max: 25
 
-# Offense count: 5
+# Offense count: 8
 # Configuration parameters: CountComments, ExcludedMethods.
+# ExcludedMethods: refine
 Metrics/BlockLength:
   Max: 46
 
-# Offense count: 3
-# Configuration parameters: CountComments.
-Metrics/ClassLength:
-  Max: 173
-
 # Offense count: 4
-# Configuration parameters: CountComments.
+# Configuration parameters: CountComments, ExcludedMethods.
 Metrics/MethodLength:
   Max: 26
 
@@ -44,14 +47,14 @@ Performance/InefficientHashSearch:
     - 'lib/drip/resource.rb'
     - 'lib/drip/response.rb'
 
-# Offense count: 23
+# Offense count: 24
 # Cop supports --auto-correct.
 # Configuration parameters: AutoCorrect, EnforcedStyle.
 # SupportedStyles: nested, compact
 Style/ClassAndModuleChildren:
   Enabled: false
 
-# Offense count: 45
+# Offense count: 44
 Style/Documentation:
   Enabled: false
 
@@ -86,13 +89,13 @@ Style/MissingRespondToMissing:
     - 'lib/drip/resource.rb'
     - 'lib/drip/response.rb'
 
-# Offense count: 10
+# Offense count: 12
 # Cop supports --auto-correct.
 # Configuration parameters: Strict.
 Style/NumericLiterals:
   MinDigits: 8
 
-# Offense count: 605
+# Offense count: 700
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, ConsistentQuotesInMultiline.
 # SupportedStyles: single_quotes, double_quotes

data/.travis.yml

@@ -1,9 +1,7 @@
 language: ruby
 sudo: false
 rvm:
+  - 2.7
+  - 2.6
   - 2.5
-  - 2.4
-  - 2.3
-  - 2.2
-  - 2.1
   - jruby-19mode

data/CHANGELOG.md

@@ -6,10 +6,60 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
-[master]: https://github.com/DripEmail/drip-ruby/compare/v3.0.0...HEAD
+[master]: https://github.com/DripEmail/drip-ruby/compare/v3.4.1...HEAD
 
 - Your contribution here!
 
+## [3.4.1] - 2020-04-21
+
+[3.4.1]: https://github.com/DripEmail/drip-ruby/compare/v3.4.0...v3.4.1
+
+### Fixed
+- Support for Ruby 2.7 hash parameters.
+
+## [3.4.0] - 2020-04-21
+
+[3.4.0]: https://github.com/DripEmail/drip-ruby/compare/v3.3.1...v3.4.0
+
+### Added
+- Support for the user endpoint in the REST API
+
+### Changed
+- `Drip::Client#create_cart_activity_event` can be used with visitor UUID
+
+## [3.3.1] - 2019-05-28
+
+[3.3.1]: https://github.com/DripEmail/drip-ruby/compare/v3.3.0...v3.3.1
+
+### Fixed
+- Correct shopper activity batch orders API to wrap things in an `orders` key.
+
+## [3.3.0] - 2019-05-24
+
+[3.3.0]: https://github.com/DripEmail/drip-ruby/compare/v3.2.0...v3.3.0
+
+### Added
+- Support for the cart, order, and product endpoints in the shopper activity API.
+
+### Changed
+- `Drip::Client#url_prefix` parameter no longer includes `/vN` part of the URL in order to prepare for Shopper Activity API. This breaks backwards compatibility for this option, but there is no expected production usage of this parameter.
+- `Drip::Client#get`, `Drip::Client#post`, `Drip::Client#put`, and `Drip::Client#delete` are deprecated. If you are using these to hit a Drip API endpoint, please file a ticket or PR to fix the use case.
+- `Drip::Client#generate_resource` is deprecated and will be removed in a future version.
+- `Drip::Client#content_type` is deprecated and will be removed in a future version. It is no longer used internally, effective immediately.
+- When using the block form of parameter initialization, a configuration object is provided instead of the client itself.
+- Calling configuration setters on `Drip::Client` is deprecated.
+- `Drip::Client::REDIRECT_LIMIT` constant is now private. This is supposed to be an implementation detail and shouldn't leak.
+
+### Removed
+- Drop support for Ruby 2.1.
+
+## [3.2.0] - 2018-08-15
+
+[3.2.0]: https://github.com/DripEmail/drip-ruby/compare/v3.1.1...v3.2.0
+
+- Allow `#create_or_update_subscriber` to work with Drip id. Fixes [#50](https://github.com/DripEmail/drip-ruby/issues/50)
+- [#52](https://github.com/DripEmail/drip-ruby/pull/52): Fix `#custom_fields` to accept the API response without error. Fixes [#30](https://github.com/DripEmail/drip-ruby/issues/30)
+
 ## [3.1.1] - 2018-06-06
 
 [3.1.1]: https://github.com/DripEmail/drip-ruby/compare/v3.1.0...v3.1.1

data/Gemfile

@@ -1,4 +1,16 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in drip-ruby.gemspec
 gemspec
+
+gem "bundler", ">= 1.6", "< 3"
+gem "minitest", "~> 5.0"
+gem "mocha", "~> 1.1"
+gem "rake", "~> 12.0"
+gem "rubocop", "~> 0.67.2"
+gem "rubocop-performance", "~> 1.1.0"
+gem "shoulda-context", "~> 1.0"
+gem "simplecov", require: false
+gem "webmock", "~> 3.4"

data/README.md

@@ -150,6 +150,12 @@ as methods on the client object. The following methods are currently available:
 | Apply a tag                | `#apply_tag(email, tag)`                             |
 | Remove a tag               | `#remove_tag(email, tag)`                            |
 
+
+#### Users
+| Actions                    | Methods                                              |
+| :------------------------- | :--------------------------------------------------- |
+| Fetch user                 | `#user`                                              |
+
 #### Webhooks
 
 | Actions                    | Methods                                              |

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "bundler/gem_tasks"
 require "rake/testtask"
 

data/drip-ruby.gemspec

@@ -1,4 +1,5 @@
 # coding: utf-8
+# frozen_string_literal: true
 
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
@@ -7,8 +8,8 @@ require 'drip/version'
 Gem::Specification.new do |spec|
   spec.name          = "drip-ruby"
   spec.version       = Drip::VERSION
-  spec.authors       = ["Derrick Reimer"]
-  spec.email         = ["derrickreimer@gmail.com"]
+  spec.authors       = ["Drip"]
+  spec.email         = ["support@drip.com"]
   spec.summary       = 'A Ruby gem for interacting with the Drip API'
   spec.description   = 'A simple wrapper for the Drip API'
   spec.homepage      = "http://github.com/DripEmail/drip-ruby"
@@ -19,13 +20,5 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.required_ruby_version = '>= 2.1'
-
-  spec.add_development_dependency "bundler", "~> 1.6"
-  spec.add_development_dependency "minitest", "~> 5.0"
-  spec.add_development_dependency "mocha", "~> 1.1"
-  spec.add_development_dependency "rake", "~> 10.0"
-  spec.add_development_dependency "rubocop", "~> 0.56.0"
-  spec.add_development_dependency "shoulda-context", "~> 1.0"
-  spec.add_development_dependency "webmock", "~> 3.4"
+  spec.required_ruby_version = '>= 2.3'
 end

data/lib/drip.rb

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 require "drip/version"
 require "drip/client"

data/lib/drip/client.rb

@@ -1,16 +1,23 @@
+# frozen_string_literal: true
+
 require "drip/errors"
+require "drip/request"
 require "drip/response"
 require "drip/client/accounts"
 require "drip/client/broadcasts"
 require "drip/client/campaigns"
 require "drip/client/campaign_subscriptions"
+require "drip/client/configuration"
 require "drip/client/conversions"
 require "drip/client/custom_fields"
 require "drip/client/events"
 require "drip/client/forms"
+require "drip/client/http_client"
 require "drip/client/orders"
+require "drip/client/shopper_activity"
 require "drip/client/subscribers"
 require "drip/client/tags"
+require "drip/client/users"
 require "drip/client/webhooks"
 require "drip/client/workflows"
 require "drip/client/workflow_triggers"
@@ -29,88 +36,76 @@ module Drip
     include Events
     include Forms
     include Orders
+    include ShopperActivity
     include Subscribers
     include Tags
+    include Users
     include Webhooks
     include Workflows
     include WorkflowTriggers
 
-    REDIRECT_LIMIT = 10
+    Drip::Client::Configuration::CONFIGURATION_FIELDS.each do |config_key|
+      define_method(config_key) do
+        @config.public_send(config_key)
+      end
+
+      setter_name = "#{config_key}=".to_sym
+      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)
+      end
+    end
+
+    JSON_API_CONTENT_TYPE = "application/vnd.api+json"
+    private_constant :JSON_API_CONTENT_TYPE
 
-    attr_accessor :access_token, :api_key, :account_id, :url_prefix, :http_open_timeout, :http_timeout
+    JSON_CONTENT_TYPE = "application/json"
+    private_constant :JSON_CONTENT_TYPE
 
     def initialize(options = {})
-      @account_id = options[:account_id]
-      @access_token = options[:access_token]
-      @api_key = options[:api_key]
-      @url_prefix = options[:url_prefix] || "https://api.getdrip.com/v2/"
-      @http_open_timeout = options[:http_open_timeout]
-      @http_timeout = options[:http_timeout]
-      yield(self) if block_given?
+      @config = Drip::Client::Configuration.new(**options)
+      yield(@config) if block_given?
     end
 
     def generate_resource(key, *args)
-      { key => args }
+      warn "[DEPRECATED] Drip::Client#generate_resource is deprecated and will be removed in a future version"
+      private_generate_resource(key, *args)
     end
 
     def content_type
-      'application/vnd.api+json'
+      warn "[DEPRECATED] Drip::Client#content_type is deprecated and will be removed in a future version"
+      JSON_API_CONTENT_TYPE
     end
 
-    def get(url, options = {})
-      make_request(Net::HTTP::Get, make_uri(url), options)
+    Drip::Request::VERB_CLASS_MAPPING.keys.each 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)
+      end
     end
 
-    def post(url, options = {})
-      make_request(Net::HTTP::Post, make_uri(url), options)
-    end
+  private
 
-    def put(url, options = {})
-      make_request(Net::HTTP::Put, make_uri(url), options)
+    def make_json_api_request(http_verb, path, options = {})
+      make_request Drip::Request.new(http_verb, make_uri(path), options, JSON_API_CONTENT_TYPE)
     end
 
-    def delete(url, options = {})
-      make_request(Net::HTTP::Delete, make_uri(url), options)
+    def make_json_request(http_verb, path, options = {})
+      make_request Drip::Request.new(http_verb, make_uri(path), options, JSON_CONTENT_TYPE)
     end
 
-  private
+    def private_generate_resource(key, *args)
+      # No reason for this to be part of the public API, so making a duplicate method to make it private.
+      { key => args }
+    end
 
     def make_uri(path)
-      URI(url_prefix) + URI(path)
+      URI(@config.url_prefix) + URI(path)
     end
 
-    def make_request(verb_klass, uri, options, step = 0)
-      raise TooManyRedirectsError, 'too many HTTP redirects' if step >= REDIRECT_LIMIT
-
+    def make_request(drip_request)
       build_response do
-        Net::HTTP.start(uri.host, uri.port, connection_options(uri.scheme)) do |http|
-          if verb_klass == Net::HTTP::Get
-            uri.query = URI.encode_www_form(options)
-          end
-
-          request = verb_klass.new uri
-
-          unless verb_klass == Net::HTTP::Get
-            request.body = options.to_json
-          end
-
-          request['User-Agent'] = "Drip Ruby v#{Drip::VERSION}"
-          request['Content-Type'] = content_type
-          request['Accept'] = "*/*"
-
-          if access_token
-            request['Authorization'] = "Bearer #{access_token}"
-          else
-            request.basic_auth api_key, ""
-          end
-
-          response = http.request request
-          if response.is_a?(Net::HTTPRedirection)
-            return make_request(verb_klass, URI(response["Location"]), options, step + 1)
-          else
-            response
-          end
-        end
+        Drip::Client::HTTPClient.new(@config).make_request(drip_request)
       end
     end
 
@@ -120,18 +115,5 @@ module Drip
     rescue JSON::ParserError
       Drip::Response.new(response.code.to_i, nil)
     end
-
-    def connection_options(uri_scheme)
-      options = { use_ssl: uri_scheme == "https" }
-
-      if @http_open_timeout
-        options[:open_timeout] = @http_open_timeout
-        options[:ssl_timeout] = @http_open_timeout
-      end
-
-      options[:read_timeout] = @http_timeout if @http_timeout
-
-      options
-    end
   end
 end

data/lib/drip/client/accounts.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Drip
   class Client
     module Accounts
@@ -6,7 +8,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#accounts
       def accounts
-        get "accounts"
+        make_json_api_request :get, "v2/accounts"
       end
 
       # Public: Fetch an account.
@@ -16,7 +18,7 @@ module Drip
       # Returns a Drip::Response.
       # See https://www.getdrip.com/docs/rest-api#accounts
       def account(id)
-        get "accounts/#{id}"
+        make_json_api_request :get, "v2/accounts/#{id}"
       end
     end
   end