json_api_toolbox 0.17.0 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3ad7345c4b37202ed05ab044fd99d6bfcd8e6e47
-  data.tar.gz: b742d7b754794c1ff35f586851562930cc8e068c
+  metadata.gz: 7449efd8dd09c17235fcb78b359a4bd7050851a6
+  data.tar.gz: c1a744a1eefa42bc00a6bd6cc8172df8a287f008
 SHA512:
-  metadata.gz: 94985f43e75243383ef8e5a4e76f3187383a6a52079c005bba484c31df492b256ede3e6e48efae0df2c36bae76890b6d91fb4a29ef1b1cc9ea5534ec7a1aafc4
-  data.tar.gz: c782dffcd059fa36a99e4f1e49ec9af3182a9f642cd34769e76aa4421c556f37167efcaa860f3c860ad488f60ff93f89e4e588d417589cc59dc28f2c7650588a
+  metadata.gz: 90b05221b72edaef537bbf173313ed6159db759203187b7aa2b3b3211a1403e20896e892cb91bb11aaf7534b4c8c56c7982aea99ff053b112eef3312d49124f6
+  data.tar.gz: b5ba639f2fa06caeda67ecaaa5b87b01aeaba2cadcc7b99e2249b1dd40053c48c0a45a83bbdf6f979e2c62eb29c320b31bb9c1cba0a14b81c682ce3cbd5aaa1d

data/CHANGELOG.md

@@ -5,6 +5,26 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Added
+
+## [Released]
+## [1.3.0] - 2020-02-19
+### Added
+- Remove gem rescue responsibility
+
+## [1.2.0] - 2020-01-16
+### Added
+- Added param `headers` to `#get`, `#post`, `#patch`, `#put`, to send extra headers
+- Added new shared example `a failed attempt to retrieve a resource`
+
+## [1.1.0] - 2019-05-28
+### Added
+- Added log with log_toolbox
+
+## [1.0.0] - 2019-05-13
+### Added
+- Added status internal_server_error on exception renderizable
+- Added status unprocessable_entity on renderizable when object have errors
 
 ## [0.17.0] - 2018-08-14
 ### Added

data/README.md

@@ -120,6 +120,9 @@ it_behaves_like 'a get with jsonapi with default value of', SomeModel
 it_behaves_like 'a json api response with all relations of', SomeModel
   validate if all relations are included on your data
 
+it_behaves_like 'a failed attempt to retrieve a resource'
+  check if the api response will include an error when `find` raise a exception retrieving a resource.
+
 ## Example Tests
 
 
@@ -158,6 +161,12 @@ RSpec.describe ManagersController, type: :controller do
       it_behaves_like 'a json api response with included node'
       it_behaves_like 'a json api response with all relations of', Manager
       it_behaves_like 'a get with jsonapi with default value of', Manager
+
+      context 'When the manager does not exist' do
+        before { get :show, params: { id: 300 } }
+
+        it_behaves_like 'a failed attempt to retrieve a resource'
+      end
     end
   end
 
@@ -202,6 +211,12 @@ RSpec.describe ManagersController, type: :controller do
       it 'validates if values were updated' do
         expect(data['attributes']['name']).to eq(new_name)
       end
+
+      context 'When the manager does not exist' do
+        before { get :show, params: { id: 300 } }
+
+        it_behaves_like 'a failed attempt to retrieve a resource'
+      end
     end
   end
 end

data/json_api_toolbox.gemspec

@@ -6,15 +6,16 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'json_api_toolbox/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = 'json_api_toolbox'
-  spec.version       = JsonApiToolbox::VERSION
-  spec.authors       = ['Adriano Bacha']
-  spec.email         = ['abacha@gmail.com']
+  spec.required_ruby_version  = '>= 2.4.0'
+  spec.name                   = 'json_api_toolbox'
+  spec.version                = JsonApiToolbox::VERSION
+  spec.authors                = ['Adriano Bacha']
+  spec.email                  = ['abacha@gmail.com']
 
-  spec.summary       = 'Json API usefull tools'
-  spec.homepage      = 'http://bitbucket.org/guideinvestimentos/json_api_toolbox'
+  spec.summary                = 'Json API usefull tools'
+  spec.homepage               = 'http://bitbucket.org/guideinvestimentos/json_api_toolbox'
 
-  spec.files         = `git ls-files -z`.split("\x0").reject do |f|
+  spec.files                  = `git ls-files -z`.split("\x0").reject do |f|
     f.match(%r{^(test|spec|features)/})
   end
   spec.require_paths = ['lib']
@@ -22,8 +23,9 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'actionpack',           '>= 4.0.0'
   spec.add_dependency 'activerecord',         '>= 4.0.0'
   spec.add_dependency 'activesupport',        '>= 4.0.0'
-  spec.add_dependency 'json-api-vanilla',     '~> 1.0.1'
   spec.add_dependency 'jsonapi-serializable', '>= 0.3.0'
+  spec.add_dependency 'json-api-vanilla',     '~> 1.0.1'
+  spec.add_dependency 'log_toolbox',          '>= 1.0.0'
   spec.add_dependency 'request_store',        '~> 1.3.2'
   spec.add_dependency 'rest-client',          '>= 2.0.1'
   spec.add_dependency 'rspec',                '>= 3.0.0'

data/lib/json_api_toolbox/paginable.rb

@@ -2,6 +2,9 @@
 
 module JsonApiToolbox
   module Paginable
+    DEFAULT_PER_PAGE = 20
+    MAX_PER_PAGE = 100
+
     def pagination_meta(collection_resource)
       {
         'current-page': collection_resource.current_page,
@@ -30,9 +33,18 @@ module JsonApiToolbox
       end
     end
 
+    def per_page
+      return DEFAULT_PER_PAGE if params[:per_page].nil?
+
+      per_page_limit = (ENV['MAX_PER_PAGE'] || MAX_PER_PAGE).to_i
+
+      [per_page_limit, params[:per_page].to_i].min
+    end
+
+    private
+
     def create_page_link(link, page)
-      link_path = collection_resource_path(page: page,
-                                           per_page: params[:per_page])
+      link_path = collection_resource_path(page: page, per_page: per_page)
       { link => link_path }
     end
 

data/lib/json_api_toolbox/spec_support/shared_examples_for_controllers.rb

@@ -37,10 +37,11 @@ module JsonApiToolbox
         let(:model_keys) do
           enums = []
           model.new.attributes.keys.map do |attr|
-            if attr.match?(/_cd$/)
+            case attr
+            when /_cd$/
               enums << attr
               attr.gsub(/_cd/, '')
-            elsif attr.match?(/_id$|^id$/)
+            when /_id$|^id$/
               nil
             else
               attr
@@ -72,6 +73,14 @@ module JsonApiToolbox
         it { expect(data).to have_key('id') }
         it { expect(data).to have_key('attributes') }
       end
+
+      RSpec.shared_examples 'a failed attempt to retrieve a resource' do
+        let(:errors) { JSON.parse(response.body)['errors'] }
+
+        it 'returns response with error\'s title including RecordNotFound' do
+          expect(errors.first['title']).to eq('ActiveRecord::RecordNotFound')
+        end
+      end
     end
   end
 end

data/lib/json_api_toolbox/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module JsonApiToolbox
-  VERSION = '0.17.0'
+  VERSION = '1.4.0'
 end

data/lib/renderizable.rb

@@ -4,7 +4,7 @@ module JsonApiToolbox
   module Renderizable
     def render_object(object, options = {})
       if object.respond_to?(:errors) && object.errors.any?
-        render({ jsonapi_errors: object.errors }.merge(options))
+        render(build_error_options(object, options))
       else
         renderization = { jsonapi: object,
                           include: includes_params,
@@ -20,9 +20,9 @@ module JsonApiToolbox
     def fields
       return {} unless fields?
 
-      params[:fields].permit!.to_h.map do |field, attributes|
-        [field, attributes.split(',')]
-      end.to_h
+      params[:fields].permit!.to_h.transform_values do |attributes|
+        attributes.split(',')
+      end
     end
 
     def permitted_params
@@ -80,5 +80,11 @@ module JsonApiToolbox
         {}
       end
     end
+
+    def build_error_options(object, options)
+      options = options.merge(status: :unprocessable_entity) if options[:status].blank?
+
+      { jsonapi_errors: object.errors }.merge(options)
+    end
   end
 end

data/lib/renderizable_exceptions.rb

@@ -11,7 +11,11 @@ module JsonApiToolbox
     end
 
     def render_any_exception(exception)
-      puts_build_log_error(exception)
+      if Gem.loaded_specs.key?('log_toolbox')
+        log_critical(exception)
+      else
+        puts_build_log_error(exception)
+      end
 
       render build_jsonapi(exception)
     end
@@ -25,7 +29,7 @@ module JsonApiToolbox
           source: {} },
         jsonapi_class: {
           class: { Hash: JsonApiToolbox::Serializables::SerializableException }
-        } }
+        }, status: :internal_server_error }
     end
 
     def puts_build_log_error(error)

data/lib/service.rb

@@ -6,19 +6,17 @@ require 'json-api-vanilla'
 
 module JsonApiToolbox
   class Service
-    attr_reader :http_method, :url, :body
+    attr_reader :http_method, :url, :body, :extra_headers
 
-    def initialize(http_method, url, body)
+    def initialize(http_method, url, body, extra_headers = {})
       @http_method = http_method
       @url = url
       @body = body
+      @extra_headers = extra_headers
     end
 
     def execute
-      response = request
-      return if response.code == 404
-
-      Service.parse_response(response)
+      Service.parse_response(request)
     end
 
     def request
@@ -28,8 +26,6 @@ module JsonApiToolbox
         payload: build_body,
         headers: build_header
       )
-    rescue RestClient::ExceptionWithResponse => e
-      e.response
     end
 
     def build_header
@@ -38,7 +34,7 @@ module JsonApiToolbox
         'Content-Type' => 'application/json'
       }
       headers[:params] = body if http_method == :get
-      headers
+      headers.merge(extra_headers)
     end
 
     def build_body
@@ -48,21 +44,21 @@ module JsonApiToolbox
     end
 
     class << self
-      def get(url: nil, includes: nil, query_string: nil)
+      def get(url: nil, includes: nil, query_string: nil, headers: {})
         body = build_query_string(includes, query_string)
-        new(:get, url, body).execute
+        new(:get, url, body, headers).execute
       end
 
-      def post(url: nil, body: nil)
-        new(:post, url, body).execute
+      def post(url: nil, body: nil, headers: {})
+        new(:post, url, body, headers).execute
       end
 
-      def patch(url: nil, body: nil)
-        new(:patch, url, body).execute
+      def patch(url: nil, body: nil, headers: {})
+        new(:patch, url, body, headers).execute
       end
 
-      def put(url: nil, body: nil)
-        new(:put, url, body).execute
+      def put(url: nil, body: nil, headers: {})
+        new(:put, url, body, headers).execute
       end
 
       def parse_response(response)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: json_api_toolbox
 version: !ruby/object:Gem::Version
-  version: 0.17.0
+  version: 1.4.0
 platform: ruby
 authors:
 - Adriano Bacha
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-11-29 00:00:00.000000000 Z
+date: 2020-11-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -52,6 +52,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 4.0.0
+- !ruby/object:Gem::Dependency
+  name: jsonapi-serializable
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.3.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.3.0
 - !ruby/object:Gem::Dependency
   name: json-api-vanilla
   requirement: !ruby/object:Gem::Requirement
@@ -67,19 +81,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 1.0.1
 - !ruby/object:Gem::Dependency
-  name: jsonapi-serializable
+  name: log_toolbox
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.3.0
+        version: 1.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.3.0
+        version: 1.0.0
 - !ruby/object:Gem::Dependency
   name: request_store
   requirement: !ruby/object:Gem::Requirement
@@ -275,7 +289,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.4.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="