Refactor rendering + improve inference. (#2)

* Refactor rendering + improve inference.

* Rename rendering.rb => renderer.rb.

* Factor-out Renderer class into JSONAPI::Serializable::Renderer.

Author: beauby

jsonapi-rails.gemspec

@@ -3,23 +3,24 @@ version = File.read(File.expand_path('../VERSION', __FILE__)).strip
 Gem::Specification.new do |spec|
   spec.name          = 'jsonapi-rails'
   spec.version       = version
-  spec.author        = ['L. Preston Sego III', 'Lucas Hosseini']
-  spec.email         = ['LPSego3+dev@gmail.com', 'lucas.hosseini@gmail.com']
-  spec.summary       = 'Rails plugin for (de)serialization of JSONAPI resources'
-  spec.description   = 'Efficiently build and consume JSONAPI documents.'
+  spec.author        = ['Lucas Hosseini']
+  spec.email         = ['lucas.hosseini@gmail.com']
+  spec.summary       = 'jsonapi-rb integrations for Rails.'
+  spec.description   = 'Efficient, convenient, non-intrusive JSONAPI ' \
+                       'framework for Rails.'
   spec.homepage      = 'https://github.com/jsonapi-rb/rails'
   spec.license       = 'MIT'
 
   spec.files         = Dir['README.md', 'lib/**/*']
   spec.require_path  = 'lib'
 
-  spec.add_dependency 'jsonapi-renderer', '0.1.1.beta2'
-  spec.add_dependency 'jsonapi-parser', '0.1.1.beta3'
-  spec.add_dependency 'jsonapi-serializable', '0.1.1.beta2'
+  spec.add_dependency 'jsonapi-renderer',       '0.1.1.beta3'
+  spec.add_dependency 'jsonapi-parser',         '0.1.1.beta3'
+  spec.add_dependency 'jsonapi-serializable',   '0.1.1.beta3'
   spec.add_dependency 'jsonapi-deserializable', '0.1.1.beta3'
 
   spec.add_development_dependency 'activerecord', '>=5'
-  spec.add_development_dependency 'sqlite3', '>= 1.3.12'
-  spec.add_development_dependency 'rake', '>=0.9'
-  spec.add_development_dependency 'rspec', '~>3.4'
+  spec.add_development_dependency 'sqlite3',      '>= 1.3.12'
+  spec.add_development_dependency 'rake',         '>=0.9'
+  spec.add_development_dependency 'rspec',        '~>3.5'
 end

lib/jsonapi/rails.rb

@@ -4,7 +4,6 @@
 require 'jsonapi/deserializable'
 require 'jsonapi/serializable'
 
-require 'jsonapi/rails/serializable/model_extensions'
 require 'jsonapi/rails/deserializable/resource_extensions'
 
 require 'jsonapi/rails/railtie'

lib/jsonapi/rails/railtie.rb

@@ -4,16 +4,12 @@
 require 'active_support'
 
 require 'jsonapi/rails/parser'
-require 'jsonapi/rails/render'
+require 'jsonapi/rails/renderer'
 
 module JSONAPI
   module Rails
     class Railtie < ::Rails::Railtie
       MEDIA_TYPE = 'application/vnd.api+json'.freeze
-      HEADERS = {
-        response: { 'CONTENT_TYPE'.freeze => MEDIA_TYPE },
-        request:  { 'ACCEPT'.freeze => MEDIA_TYPE }
-      }.freeze
       PARSER = JSONAPI::Rails.parser
 
       initializer 'JSONAPI::Rails.initialize' do
@@ -23,16 +19,22 @@ class Railtie < ::Rails::Railtie
         else
           ActionDispatch::ParamsParser::DEFAULT_PARSERS[Mime[:jsonapi]] = PARSER
         end
+
         ActionController::Renderers.add :jsonapi do |json, options|
           unless json.is_a?(String)
-            json = JSONAPI::Rails.render(json, options)
-                                 .to_json(options)
+            json = JSONAPI::Rails::Renderer.render(json, options)
           end
           self.content_type ||= Mime[:jsonapi]
           self.response_body = json
         end
 
-        # TODO(beauby): Add renderer for `jsonapi_errors`.
+        ActionController::Renderers.add :jsonapi_errors do |json, options|
+          unless json.is_a?(String)
+            json = JSONAPI::Rails::ErrorRender.render_errors(json, options)
+          end
+          self.content_type ||= Mime[:jsonapi]
+          self.response_body = json
+        end
       end
     end
   end

lib/jsonapi/rails/render.rb

@@ -0,0 +1,27 @@
+require 'jsonapi/serializable/renderer'
+require 'jsonapi/serializable/resource_builder'
+
+module JSONAPI
+  module Rails
+    class Renderer
+      def self.render(resources, options)
+        # TODO(beauby): handle status option.
+        opts = options.dup
+        # TODO(beauby): Move this to a global configuration.
+        default_exposures = {
+          url_helpers: ::Rails.application.routes.url_helpers
+        }
+        opts[:expose] = default_exposures.merge!(opts[:expose] || {})
+
+        JSONAPI::Serializable::Renderer.render(resources, opts)
+      end
+    end
+
+    class ErrorRenderer
+      def self.render(errors, options)
+        # TODO(beauby): SerializableError inference on AR validation errors.
+        JSONAPI::Serializable::ErrorRenderer.render(errors, options)
+      end
+    end
+  end
+end