fix(ControllerRenderer) update to use helper ivar

Author: rmosolgo

lib/react/rails/controller_lifecycle.rb

@@ -7,6 +7,7 @@ module ControllerLifecycle
         # use old names to support Rails 3
         before_filter :setup_react_component_helper
         after_filter :teardown_react_component_helper
+        attr_reader :__react_component_helper
       end
 
       def setup_react_component_helper

lib/react/rails/controller_renderer.rb

@@ -5,9 +5,9 @@ class React::Rails::ControllerRenderer
 
   attr_accessor :output_buffer
 
-  attr_reader :request
   def initialize(options={})
-    @request = options[:request]
+    controller = options[:controller]
+    @__react_component_helper = controller.__react_component_helper
   end
 
   def call(name, options, &block)

lib/react/rails/railtie.rb

@@ -43,7 +43,7 @@ class Railtie < ::Rails::Railtie
 
       initializer "react_rails.add_component_renderer", group: :all do |app|
         ActionController::Renderers.add :component do |component_name, options|
-          renderer = ::React::Rails::ControllerRenderer.new(request: request)
+          renderer = ::React::Rails::ControllerRenderer.new(controller: self)
           html = renderer.call(component_name, options)
           render_options = options.merge(inline: html)
           render(render_options)

test/react/rails/controller_lifecycle_test.rb

@@ -34,22 +34,22 @@ def teardown
 
   test "it creates a helper object and puts it in the request env" do
     get '/pages/1'
-    helper_obj = controller.instance_variable_get(:@__react_component_helper)
+    helper_obj = controller.__react_component_helper
     assert(helper_obj.is_a?(DummyHelperImplementation), "It uses the view helper implementation class")
   end
 
   test "it calls setup and teardown methods" do
     get '/pages/1?param_test=123'
-    helper_obj = controller.instance_variable_get(:@__react_component_helper)
+    helper_obj = controller.__react_component_helper
     lifecycle_steps = ["123", :react_component, :teardown]
     assert_equal(lifecycle_steps, helper_obj.events)
   end
 
   test "there's a new helper object for every request" do
     get '/pages/1'
-    first_helper = controller.instance_variable_get(:@__react_component_helper)
+    first_helper = controller.__react_component_helper
     get '/pages/1'
-    second_helper = controller.instance_variable_get(:@__react_component_helper)
+    second_helper = controller.__react_component_helper
     assert(first_helper != second_helper, "The helper for the second request is brand new")
   end
 end