help_scout-sdk 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: db597946131f89598f48083d3352273b5adde0f23dfe608a9f42f4a0d836be83
-  data.tar.gz: d7e837bf6257fda9b10f21e74948068df19f50033885e70b30120004e98b6717
+  metadata.gz: fde4d41133056de55106a9908c925633544fe8a0f888794ed88dd3eff5502985
+  data.tar.gz: 25e7ad5dcdb45e4d24744bcb72a06a46588cc0712d4eed9e2990246ad3bb625a
 SHA512:
-  metadata.gz: d9b1bec06d8ab4ee6715054ad7a96b3013532c2e62fd76eb5a68e0b6425b8a0f62f7b395dc3c60ab607a76926b3ae1ebeb928fbd9757977064b9c20a643ddc7c
-  data.tar.gz: ea66c98c374e78172b7cfa5daa7b236357bd4a74f624a86803725bddfb2b6503d4da95b88f1220d4d5aef1a48dfb7e481c63897e35fff7bba26d71df387a0ca3
+  metadata.gz: f25c0927c1bdde7e99f7b9be29734978a99d39dba10cc3143a2ef6968fe4456713670709805888c69cb44a2b961deb8e3f8c5d5218f0afbb27d4357cacee80a8
+  data.tar.gz: b94698d0faf4d31672d5434b26ed4e2a3025a55b846db3512d458ab1176bcf45b0f5adbfcc68030e6428472bc5ef9cbf864a5923350d160d81b42c1c9d28d0be

data/.rubocop.yml

@@ -29,5 +29,8 @@ Naming/FileName:
     - 'lib/help_scout-sdk.rb'
     - 'spec/**/*_spec.rb'
 
+Naming/MemoizedInstanceVariableName:
+    EnforcedStyleForLeadingUnderscores: required
+
 Style/Documentation:
   Enabled: False

data/README.md

@@ -1,8 +1,7 @@
 # Help Scout API Wrapper
 
 [![Build Status](https://travis-ci.org/taxjar/help_scout-sdk.svg?branch=master)](https://travis-ci.org/taxjar/help_scout-sdk)
-
-🚨 WORK IN PROGRESS 🚨
+[![Gem Version](https://badge.fury.io/rb/help_scout-sdk.svg)](https://badge.fury.io/rb/help_scout-sdk)
 
 This gem is a wrapper around the Help Scout API. The current version is targeting the [Mailbox 2.0 API](https://developer.helpscout.com/mailbox-api/).
 

data/help_scout-sdk.gemspec

@@ -11,7 +11,7 @@ Gem::Specification.new do |spec|
   spec.email         = ['support@taxjar.com']
 
   spec.summary       = 'Ruby Help Scout SDK'
-  spec.description   = 'Ruby Help Scout SDK.'
+  spec.description   = 'Ruby Help Scout (aka HelpScout) SDK.'
   spec.homepage      = 'https://github.com/taxjar/help_scout-sdk'
   spec.license       = 'MIT'
 

data/lib/help_scout/api/access_token.rb

@@ -5,15 +5,14 @@ module HelpScout
     class AccessToken
       class << self
         def create
-          connection = HelpScout::API::Client.new.unauthorized_connection
+          connection = HelpScout::API::Client.new(authorize: false).connection
           response = connection.post('oauth2/token', token_request_params)
 
           case response.status
-          when 429 then raise ThrottleLimitReached, response.body&.dig('error')
-          when 500, 501, 503 then raise InternalError, response.body&.dig('error')
+          when 200 then new HelpScout::Response.new(response).body
+          when 429 then raise HelpScout::API::ThrottleLimitReached, response.body&.dig('error')
+          else raise HelpScout::API::InternalError, "unexpected response (status #{response.status})"
           end
-
-          new HelpScout::Response.new(response).body
         end
 
         def refresh!
@@ -23,7 +22,7 @@ module HelpScout
         private
 
         def token_request_params
-          @token_request_params ||= {
+          @_token_request_params ||= {
             grant_type: 'client_credentials',
             client_id: HelpScout.app_id,
             client_secret: HelpScout.app_secret
@@ -31,21 +30,33 @@ module HelpScout
         end
       end
 
-      attr_reader :expires_at, :expires_in, :value
+      attr_accessor :invalid
+      attr_reader :expires_in, :value
 
       def initialize(params)
         @value = params[:access_token]
-        expires_in = params[:expires_in]
-
-        return unless expires_in
+        @expires_in = params[:expires_in]
+      end
 
-        @expires_in = expires_in
-        @expires_at = Time.now.utc + expires_in
+      def expires_at
+        @_expires_at ||= Time.now.utc + expires_in
       end
 
       def expired?
         Time.now.utc > expires_at
       end
+
+      def invalid?
+        invalid
+      end
+
+      def invalidate!
+        self.invalid = true
+      end
+
+      def stale?
+        invalid? || expired?
+      end
     end
   end
 end

data/lib/help_scout/api/client.rb

@@ -3,29 +3,37 @@
 module HelpScout
   class API
     class Client
-      def authorized_connection
-        @authorized_connection ||= begin
-          HelpScout::API::AccessToken.refresh! if HelpScout.access_token.nil?
-          build_connection
-        end
+      attr_reader :authorize
+
+      def initialize(authorize: true)
+        @authorize = authorize
       end
 
-      def unauthorized_connection
-        @unauthorized_connection ||= begin
-          build_connection(authorize: false)
+      def connection
+        @_connection ||= begin
+          HelpScout::API::AccessToken.refresh! if authorize? && token_needs_refresh?
+          build_connection
         end
       end
 
       private
 
-      def build_connection(authorize: true)
+      def authorize?
+        authorize
+      end
+
+      def build_connection
         Faraday.new(url: BASE_URL) do |conn|
           conn.request :json
-          conn.authorization(:Bearer, HelpScout.access_token.value) if authorize && HelpScout.access_token&.value
+          conn.authorization(:Bearer, HelpScout.access_token.value) if authorize? && HelpScout.access_token&.value
           conn.response(:json, content_type: /\bjson$/)
           conn.adapter(Faraday.default_adapter)
         end
       end
+
+      def token_needs_refresh?
+        HelpScout.access_token.nil? || HelpScout.access_token.stale?
+      end
     end
   end
 end

data/lib/help_scout/api.rb

@@ -30,29 +30,30 @@ module HelpScout
 
     private
 
-    def handle_response(result) # rubocop:disable AbcSize
-      case result.status
-      when 400 then raise BadRequest, result.body&.dig('validationErrors')
-      when 401 then raise NotAuthorized, result.body&.dig('error_description')
-      when 404 then raise NotFound, 'Resource Not Found'
-      when 429 then raise ThrottleLimitReached, result.body&.dig('error')
-      when 500, 501, 503 then raise InternalError, result.body&.dig('error')
+    def handle_response(result) # rubocop:disable AbcSize, Metrics/MethodLength
+      if (200...300).include? result.status
+        HelpScout::Response.new(result)
+      else
+        case result.status
+        when 400 then raise BadRequest, result.body&.dig('validationErrors')
+        when 401 then raise NotAuthorized, result.body&.dig('error_description')
+        when 404 then raise NotFound, 'Resource Not Found'
+        when 429 then raise ThrottleLimitReached, result.body&.dig('error')
+        else raise InternalError, result.body
+        end
       end
-
-      HelpScout::Response.new(result)
     end
 
     def new_connection
-      HelpScout::API::Client.new.authorized_connection
+      HelpScout::API::Client.new.connection
     end
 
     def send_request(action, path, params)
-      connection = new_connection
-      response = connection.send(action, path, params.compact)
+      response = new_connection.send(action, path, params.compact)
 
       if response.status == 401
-        HelpScout::API::AccessToken.refresh!
-        response = connection.send(action, path, params.compact)
+        access_token.invalidate!
+        response = new_connection.send(action, path, params.compact)
       end
 
       handle_response(response)

data/lib/help_scout/configuration.rb

@@ -6,6 +6,8 @@ module HelpScout
     attr_reader :access_token
 
     def access_token=(token_value)
+      return unless token_value
+
       @access_token = HelpScout::API::AccessToken.new(access_token: token_value)
     end
   end

data/lib/help_scout/conversation.rb

@@ -61,7 +61,7 @@ module HelpScout
     end
 
     def populated_threads
-      @populated_threads ||= HelpScout::Thread.list(id)
+      @_populated_threads ||= HelpScout::Thread.list(id)
     end
 
     def update(operation, path, value = nil)

data/lib/help_scout/mailbox.rb

@@ -27,11 +27,11 @@ module HelpScout
     end
 
     def fields
-      @fields ||= HelpScout.api.get(fields_path).embedded[:fields]
+      @_fields ||= HelpScout.api.get(fields_path).embedded[:fields]
     end
 
     def folders
-      @folders ||= HelpScout::Folder.list(id: id)
+      @_folders ||= HelpScout::Folder.list(id: id)
     end
 
     private

data/lib/help_scout/response.rb

@@ -10,7 +10,7 @@ module HelpScout
     end
 
     def body
-      @body ||= response.body.deep_transform_keys { |key| key.to_s.underscore.to_sym }
+      @_body ||= response.body.deep_transform_keys { |key| key.to_s.underscore.to_sym }
     end
 
     def embedded

data/lib/help_scout/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HelpScout
-  VERSION = '1.0.0'
+  VERSION = '1.0.1'
 end

data/lib/help_scout-sdk.rb

@@ -35,7 +35,7 @@ module HelpScout
     end
 
     def api
-      @api ||= HelpScout::API.new
+      @_api ||= HelpScout::API.new
     end
 
     def app_id
@@ -47,7 +47,7 @@ module HelpScout
     end
 
     def configuration
-      @configuration ||= Configuration.new
+      @_configuration ||= Configuration.new
     end
 
     def configure
@@ -59,8 +59,8 @@ module HelpScout
       configuration.default_mailbox
     end
 
-    def refresh!
-      @api = nil
+    def reset!
+      @_api = HelpScout::API.new
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: help_scout-sdk
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - TaxJar
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-06-05 00:00:00.000000000 Z
+date: 2019-06-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -212,7 +212,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.3'
-description: Ruby Help Scout SDK.
+description: Ruby Help Scout (aka HelpScout) SDK.
 email:
 - support@taxjar.com
 executables: []