Use FaradayMiddleware for Hashie parsing, OAuth2. Straighten up requires to be more explicit, use requires properly to distinguish between Faraday::Middleware and FaradayMiddleware, namespace custom Faraday Middleware, remove unnecessary :url_encoded option, update dependencies, write new (currently failing) test for properly encoded special characters in URI fragments

Author: toastercup

.travis.yml

@@ -1,6 +1,6 @@
 language: ruby
 rvm:
-  - 2.3.0
+  - 2.4.0
 notifications:
   recipients:
-    - ContentEnablementProductTeam@careerbuilder.com
+    - EmployerSiteContentProducts@cb.com

cortex-client.gemspec

@@ -6,21 +6,20 @@ Gem::Specification.new do |s|
   s.version       = Cortex::VERSION
   s.summary       = 'Cortex API Client'
   s.homepage      = 'https://github.com/cortex-cms/cortex-client-ruby'
-  s.authors       = ['CB Content Enablement']
-  s.email         = 'ContentEnablementProductTeam@careerbuilder.com'
+  s.authors       = ['CareerBuilder Employer Site & Content Products']
+  s.email         = ['EmployerSiteContentProducts@cb.com']
   s.license       = 'Apache-2.0'
 
   s.files         = `git ls-files`.split($/).reject { |f| f == '.gitignore' }
   s.test_files    = s.files.grep(%r{^(test|spec|features)/})
   s.require_paths = ['lib']
 
-  s.add_development_dependency 'rake', '~> 11.1'
-  s.add_development_dependency 'rspec', '~> 3.4'
-  s.add_development_dependency 'mocha', '~> 1.1'
+  s.add_development_dependency 'rake', '~> 12.0'
+  s.add_development_dependency 'rspec', '~> 3.5'
+  s.add_development_dependency 'mocha', '~> 1.2'
 
   s.add_dependency 'faraday', '~> 0.9'
   s.add_dependency 'faraday_middleware', '~> 0.10'
-  s.add_dependency 'oauth2', '~> 1.1.0'
   s.add_dependency 'cortex-exceptions', '~> 0.0.4'
   s.add_dependency 'hashie', '~> 3.4'
 end

lib/cortex/client.rb

@@ -5,8 +5,6 @@
 require 'cortex/users'
 require 'cortex/webpages'
 require 'cortex/result'
-require 'oauth2'
-require 'cortex/exceptions'
 
 module Cortex
   class Client
@@ -20,27 +18,17 @@ class Client
     include Cortex::Request
 
     def initialize(hasharg)
-      @base_url = hasharg[:base_url] || 'https://cbcortex.com/api/v1/'
+      @base_url = hasharg[:base_url] || 'https://cbcortex.com/api/v1'
       if hasharg.has_key? :access_token
         @access_token = hasharg[:access_token]
       else
         @key = hasharg[:key]
         @secret = hasharg[:secret]
         @scopes ||= hasharg[:scopes]
-        @access_token = get_cc_token
       end
       @posts = Cortex::Posts.new(self)
       @users = Cortex::Users.new(self)
       @webpages = Cortex::Webpages.new(self)
     end
-
-    def get_cc_token
-      begin
-        client = OAuth2::Client.new(@key, @secret, site: @base_url)
-        client.client_credentials.get_token({scope: @scopes})
-      rescue Faraday::ConnectionFailed
-        raise Cortex::Exceptions::ConnectionFailed.new(base_url: @base_url)
-      end
-    end
   end
 end

lib/cortex/connection.rb

@@ -1,6 +1,6 @@
 require 'faraday'
 require 'faraday_middleware'
-require 'cortex/faraday_middleware'
+require 'cortex/faraday_middleware/response_failures'
 
 module Cortex
   module Connection
@@ -17,11 +17,19 @@ def connection
       end
 
       Faraday.new options do |conn|
-        conn.use Cortex::FaradayMiddleware
-        conn.request :oauth2, access_token.is_a?(OAuth2::AccessToken) ? access_token.token : access_token
+        # Hello, temporal coupling. Order matters here.
+
+        # Request middleware first:
+        conn.use ::FaradayMiddleware::OAuth2, access_token.is_a?(OAuth2::AccessToken) ? access_token.token : access_token
+
+        # Response middleware second:
+        conn.use ::FaradayMiddleware::Mashify
+        conn.use Cortex::FaradayMiddleware::ResponseFailures
+
         conn.request :json
-        conn.request :url_encoded
         conn.response :json, :content_type => /\bjson$/
+
+        # Adapter always last:
         conn.adapter Faraday.default_adapter
       end
     end

lib/cortex/faraday_middleware.rb

@@ -0,0 +1,16 @@
+require 'faraday'
+require 'cortex/exceptions'
+
+module Cortex
+  module FaradayMiddleware
+    class ResponseFailures < Faraday::Middleware
+      def call(env)
+        begin
+          @app.call(env)
+        rescue Faraday::ConnectionFailed
+          raise Cortex::Exceptions::ConnectionFailed.new(base_url: env[:url])
+        end
+      end
+    end
+  end
+end

lib/cortex/faraday_middleware/response_failures.rb

@@ -1,5 +1,3 @@
-require 'ostruct'
-
 module Cortex
   module Request
     def get(path, params = {})

lib/cortex/request.rb

@@ -7,4 +7,4 @@ def initialize(client)
     protected
     attr_accessor :client
   end
-end
+end

lib/cortex/resource.rb

@@ -1,10 +1,9 @@
-require 'hashie'
 module Cortex
   class Result
     attr_reader :raw_headers, :contents, :total_items, :page, :per_page, :errors, :range_start, :range_end, :range, :status, :total_pages, :next_page, :prev_page
 
     def initialize(body, headers, status)
-      @contents = parse(body)
+      @contents = body
       @raw_headers = headers
       @status = status
       parse_headers(headers)
@@ -32,17 +31,6 @@ def parse_headers(headers)
       end
     end
 
-    def parse(body)
-      case body
-        when Hash
-          ::Hashie::Mash.new(body)
-        when Array
-          body.map { |item| parse(item) }
-        else
-          body
-      end
-    end
-
     def find_errors
       if is_error?
         if @contents.is_a?(Hash)

lib/cortex/result.rb

@@ -2,6 +2,7 @@
 
 RSpec.describe Cortex::Posts do
 
+  # TODO: Stub out Faraday somewhere. See: https://github.com/lostisland/faraday#using-faraday-for-testing
   let(:client) { Cortex::Client.new(access_token: '123') }
 
   describe :get do
@@ -16,6 +17,10 @@
       client.expects(:get).with('/posts/feed/1').returns('response')
       expect(client.posts.get_published(1)).to eq('response')
     end
+
+    it 'should work with special characters' do
+      expect { client.posts.get_published('1 post') }.to_not raise_error(URI::InvalidURIError)
+    end
   end
 
   describe :feed do