Remove rendering partial outside of Rails.

Implementation was partial and completing it requires to rely on internal of
action_view.

CHANGELOG.md

@@ -5,6 +5,7 @@
   * Drop support for Rails < 4.2
   * Replace `thread_safe` with `concurrent_ruby`
   * Remove custom responder
+  * Remove rendering outside of Rails
 
 ## 0.4.3
   * Fix custom responder compatibility with responders 2.1 (itkin)

lib/rabl-rails.rb

@@ -1,19 +1,25 @@
-require 'rails/railtie'
-
 require 'active_support'
 
 require 'rabl-rails/version'
 require 'rabl-rails/helpers'
+require 'rabl-rails/exceptions'
 require 'rabl-rails/template'
 require 'rabl-rails/nodes'
 require 'rabl-rails/compiler'
 
 require 'rabl-rails/visitors'
-require 'rabl-rails/renderer'
+require 'rabl-rails/renderers/hash'
+require 'rabl-rails/renderers/json'
+require 'rabl-rails/renderers/xml'
+require 'rabl-rails/renderers/plist'
 require 'rabl-rails/library'
 
 require 'rabl-rails/handler'
-require 'rabl-rails/railtie'
+
+if defined?(Rails)
+  require 'rails/railtie'
+  require 'rabl-rails/railtie'
+end
 
 require 'rabl-rails/configuration'
 
@@ -25,8 +31,6 @@
 end
 
 module RablRails
-  extend Renderer
-
   class << self
     def configure
       yield configuration

lib/rabl-rails/exceptions.rb

@@ -0,0 +1,3 @@
+module RablRails
+  class PartialError < StandardError; end
+end

lib/rabl-rails/renderers/hash.rb

@@ -13,8 +13,7 @@ module Hash
       def render(template, context, locals = nil)
         visitor = Visitors::ToHash.new(context)
 
-        collection_or_resource    = locals[:resource] if locals
-        collection_or_resource  ||= if template.data
+        collection_or_resource = if template.data
           if context.respond_to?(template.data)
             context.send(template.data)
           else

lib/rabl-rails/visitors/to_hash.rb

@@ -44,7 +44,7 @@ def visit_Code n
         result = instance_exec _resource, &(n.block)
 
         if n.merge?
-          raise RablRails::Renderer::PartialError, '`merge` block should return a hash' unless result.is_a?(Hash)
+          raise RablRails::PartialError, '`merge` block should return a hash' unless result.is_a?(Hash)
           @_result.merge!(result)
         else
           @_result[n.name] = result
@@ -98,7 +98,7 @@ def locals
     # rendering time).
     #
     def partial(template_path, options = {})
-      raise RablRails::Renderer::PartialError.new("No object was given to partial #{template_path}") unless options[:object]
+      raise RablRails::PartialError.new("No object was given to partial #{template_path}") unless options[:object]
       object = options[:object]
       @_locals = options[:locals].freeze
 

test/renderers/test_hash_renderer.rb

@@ -2,8 +2,8 @@
 
 class TestHashRenderer < Minitest::Test
   describe 'hash renderer' do
-    def render(locals = nil)
-      RablRails::Renderers::Hash.render(@template, @context, locals)
+    def render
+      RablRails::Renderers::Hash.render(@template, @context, {})
     end
 
     def with_cache
@@ -71,12 +71,6 @@ def @resource.cache_key; 'marty_cache' end
       assert_equal({ name: 'Marty' }, render)
     end
 
-    it "uses resource passed in locals if template don't have data" do
-      @template.data = nil
-      resource = User.new(2, 'Biff')
-      assert_equal({ name: 'Biff' }, render(resource: resource))
-    end
-
     it 'uses template root_name option' do
       @template.root_name = :user
       assert_equal({ user: { name: 'Marty' } }, render)
@@ -87,4 +81,4 @@ def @resource.cache_key; 'marty_cache' end
       assert_equal([{ name: 'Marty' }], render)
     end
   end
-end
+end

test/test_hash_visitor.rb

@@ -161,7 +161,7 @@ def @context.context_method; end
     it 'raises an exception when trying to merge a non hash object' do
       proc = ->(c) { c.name }
       @nodes << RablRails::Nodes::Code.new(nil, proc)
-      assert_raises(RablRails::Renderer::PartialError) { visitor_result }
+      assert_raises(RablRails::PartialError) { visitor_result }
     end
 
     it 'renders partial defined in node' do
@@ -216,7 +216,7 @@ def @context.context_method; end
     it 'raises an exception when calling a partial without a target' do
       proc = ->(u) { partial('users/base') }
       @nodes << RablRails::Nodes::Code.new(:user, proc)
-      assert_raises(RablRails::Renderer::PartialError) { visitor_result }
+      assert_raises(RablRails::PartialError) { visitor_result }
     end
 
     describe 'when hash options are set' do