cortex-client 0.9.0 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0ae9160a1f77ff75bbdd94a1dd24c2158dd24dd2
-  data.tar.gz: 32696a70d36bfb41f7b3769ef6d8c86f71ad7c1f
+  metadata.gz: b344d2092431f0f6f392a2f1ca342ce0e70f6705
+  data.tar.gz: c501c028bb02bee309120a3624a345114e5b6865
 SHA512:
-  metadata.gz: a6bd08c5ae793a6aaac1e5bb8c320164cd2c776a1515a76f7860b305c7daacf59fc30a175f8d2383f2fa3bd99dcb90777fe4aee13408114adfae60aebcffc6fd
-  data.tar.gz: 4342fc1a3198dda90259324c55b0f7bc821300eb9f7a1f98e8d461b909acc6e8e8949ba9ca13c0545afba720ae01ad617832c16c230600a488bc8629061a0595
+  metadata.gz: 27a6bf2e5747a8a412bdd9a337b0209e4fdeac13ec1e0c0a5510567a904d354c4877963fdfcdd12d6797e252b0aa0a40d47f35101ea137285e9ccef730e3c781
+  data.tar.gz: 98ab2b349ef4391bc14b87d31a5ca07e18e9e6d67fedb31d409e267bb19ddf634948ed2bceca13aafbac95d277331e681e41d1a5925abd9ce87e738fbefd7d2b

data/.travis.yml

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

data/CHANGELOG.md

@@ -1,8 +1,13 @@
 Version History
 ====
-    * All Version bumps are required to update this file as well!!
+    * All Version bumps are required to update this file as well!
 ----
 
+* 0.10.0:
+  * Utilize `Addressable::URI` as Faraday's default URI parser so that resourceful URI fragments are escaped and parsed properly
+  * Utilize `Faraday::Middleware` for `Hashie::Mash`ing & OAuth2 Token insertion
+  * Update dependencies
+  * No major breaking changes anticipated, but please *test* your use cases!
 * 0.9.0 - Add total pages, next page, and prev page to result object
 * 0.8.0 - Revert ETag caching flow implementation
 * 0.7.0 - Implement ability to specify OAuth scopes

data/cortex-client.gemspec

@@ -6,21 +6,22 @@ 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 'oauth2', '~> 1.1.0'
   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 'addressable', '~> 2.5'
   s.add_dependency 'hashie', '~> 3.4'
+  s.add_dependency 'cortex-exceptions', '~> 0.0.4'
 end

data/lib/cortex/client.rb

@@ -1,3 +1,5 @@
+require 'oauth2'
+
 require 'cortex/connection'
 require 'cortex/request'
 require 'cortex/resource'
@@ -5,8 +7,6 @@ require 'cortex/posts'
 require 'cortex/users'
 require 'cortex/webpages'
 require 'cortex/result'
-require 'oauth2'
-require 'cortex/exceptions'
 
 module Cortex
   class Client
@@ -20,7 +20,7 @@ module Cortex
     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

data/lib/cortex/connection.rb

@@ -1,6 +1,9 @@
 require 'faraday'
 require 'faraday_middleware'
-require 'cortex/faraday_middleware'
+require 'addressable/uri'
+require 'hashie/mash'
+
+require 'cortex/faraday_middleware/response_failures'
 
 module Cortex
   module Connection
@@ -16,11 +19,19 @@ module Cortex
         @access_token = get_cc_token
       end
 
+      Faraday::Utils.default_uri_parser = Addressable::URI
       Faraday.new options do |conn|
-        conn.use Cortex::FaradayMiddleware
-        conn.request :oauth2, access_token.is_a?(OAuth2::AccessToken) ? access_token.token : access_token
+        ## 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.response :json, :content_type => /\bjson$/
+
+        ## Adapter always last:
         conn.adapter Faraday.default_adapter
       end
     end

data/lib/cortex/faraday_middleware/response_failures.rb

@@ -0,0 +1,17 @@
+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

data/lib/cortex/request.rb

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

data/lib/cortex/resource.rb

@@ -7,4 +7,4 @@ module Cortex
     protected
     attr_accessor :client
   end
-end
+end

data/lib/cortex/result.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)
@@ -12,7 +11,7 @@ module Cortex
     end
 
     def is_error?
-      @status >= 400 || (@contents.is_a?(Hash) && @contents.has_key?('errors'))
+      @status >= 400 || (@contents.is_a?(Hash) && @contents.errors?)
     end
 
     private
@@ -32,31 +31,21 @@ module Cortex
       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
+      errors = nil
       if is_error?
         if @contents.is_a?(Hash)
-          if @contents.has_key?('errors')
-            Array(@contents['errors'])
+          if @contents.errors?
+            errors = @contents.errors
           else
-            Array(@contents['message'])
+            errors = @contents.message
           end
         else
-          Array(@contents)
+          errors = @contents
         end
-      else
-        Array(nil)
       end
+
+      Array(errors)
     end
   end
 end

data/lib/cortex/version.rb

@@ -1,3 +1,3 @@
 module Cortex
-  VERSION = '0.9.0'
+  VERSION = '0.10.0'
 end

data/spec/posts_spec.rb

@@ -2,6 +2,7 @@ require 'spec_helper'
 
 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 @@ RSpec.describe Cortex::Posts do
       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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cortex-client
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.10.0
 platform: ruby
 authors:
-- CB Content Enablement
+- CareerBuilder Employer Site & Content Products
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-09 00:00:00.000000000 Z
+date: 2017-01-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -16,42 +16,56 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.1'
+        version: '12.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.1'
+        version: '12.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '3.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '3.5'
 - !ruby/object:Gem::Dependency
   name: mocha
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.2'
+- !ruby/object:Gem::Dependency
+  name: oauth2
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.1.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.1.0
 - !ruby/object:Gem::Dependency
   name: faraday
   requirement: !ruby/object:Gem::Requirement
@@ -81,49 +95,50 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0.10'
 - !ruby/object:Gem::Dependency
-  name: oauth2
+  name: addressable
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: '2.5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: '2.5'
 - !ruby/object:Gem::Dependency
-  name: cortex-exceptions
+  name: hashie
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.0.4
+        version: '3.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.0.4
+        version: '3.4'
 - !ruby/object:Gem::Dependency
-  name: hashie
+  name: cortex-exceptions
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: 0.0.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: 0.0.4
 description: 
-email: ContentEnablementProductTeam@careerbuilder.com
+email:
+- EmployerSiteContentProducts@cb.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -139,7 +154,7 @@ files:
 - lib/cortex/client.rb
 - lib/cortex/client_exceptions.rb
 - lib/cortex/connection.rb
-- lib/cortex/faraday_middleware.rb
+- lib/cortex/faraday_middleware/response_failures.rb
 - lib/cortex/posts.rb
 - lib/cortex/request.rb
 - lib/cortex/resource.rb
@@ -173,7 +188,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.8
 signing_key: 
 specification_version: 4
 summary: Cortex API Client

data/lib/cortex/faraday_middleware.rb

@@ -1,14 +0,0 @@
-require 'faraday'
-require 'faraday_middleware'
-
-module Cortex
-  class FaradayMiddleware < 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