A BUG! fixed render :state in cell views.

lib/cells/cell/view.rb

@@ -18,19 +18,18 @@ def try_picking_template_for_path(template_path)
       ### so that we can overwrite the helper and cleanly reuse the internal method? using the same
       ### method both internally and externally sucks ass.
       def render(options = {}, local_assigns = {}, &block)
-        ### TODO: delegate dynamically:
         ### TODO: we have to find out if this is a call to the cells #render method, or to the rails
         ###       method (e.g. when rendering a layout). what a shit.
-        if view = options[:view]
-          return cell.render_view_for(options, view)
+        if (options.keys & [:view, :state]).present?  ### TODO: is there something like has_keys?
+          # that's better: simply delegate render back to the cell/controller.
+          return cell.render(options)
         end
 
         # rails compatibility we should get rid of:
         if partial_path = options[:partial]
           # adds the cell name to the partial name.
           options[:partial] = expand_view_path(partial_path)
         end
-        #throw Exception.new
 
         super(options, local_assigns, &block)
       end

lib/cells/version.rb

@@ -1,5 +1,5 @@
 # encoding: utf-8
 
 module Cells
-  VERSION = '3.3.1'.freeze
+  VERSION = '3.3.2'
 end

test/app/cells/bassist/jam.html.erb

@@ -0,0 +1,3 @@
+<% @chords.each do |chord| %>
+<%= render :state => :play %>
+<% end %>

test/app/cells/bassist/play.html.erb

@@ -0,0 +1 @@
+Doo

test/app/cells/bassist_cell.rb

@@ -0,0 +1,3 @@
+class BassistCell < Cell::Base
+
+end

test/render_test.rb

@@ -283,4 +283,20 @@ def another_state
     end
     assert_equal "Metal:A/another_state/a,b", render_cell(:a, :existing_view)
   end
+
+  context "render :state within a view" do
+    should "return the state content" do
+      assert_equal( "\nDoo\n\nDoo\n\nDoo\n\nDoo\n", 
+        bassist_mock do
+          def jam
+            @chords = %w(d a c g)
+            render
+          end
+          def play
+            render
+          end
+        end.render_state(:jam)
+      )
+    end
+  end
 end

test/support/internal_assertions_helper.rb

@@ -60,5 +60,11 @@ def cell_mock(options={}, &block)
       cell.instance_eval(&block) if block_given?
       cell
     end
+
+    def bassist_mock(options={}, &block)
+      cell = BassistCell.new(@controller, options)
+      cell.instance_eval(&block) if block_given?
+      cell
+    end
   end
 end

test/test_helper.rb

@@ -74,3 +74,5 @@
   include Cells::AssertionsHelper
   include Cells::InternalAssertionsHelper
 end
+
+require File.join(File.dirname(__FILE__), *%w[app cells bassist_cell])