Merge pull request #551 from bogdan-dumitru/550-fix-initialize-order

Run initializer blocks after :load_config_initializers

Author: Robert Mosolgo

lib/react/rails/railtie.rb

@@ -25,7 +25,7 @@ class Railtie < ::Rails::Railtie
       end
 
       # Include the react-rails view helper lazily
-      initializer "react_rails.setup_view_helpers", group: :all do |app|
+      initializer "react_rails.setup_view_helpers", after: :load_config_initializers, group: :all do |app|
 
         app.config.react.jsx_transformer_class ||= React::JSX::DEFAULT_TRANSFORMER
         React::JSX.transformer_class = app.config.react.jsx_transformer_class
@@ -53,7 +53,7 @@ class Railtie < ::Rails::Railtie
         end
       end
 
-      initializer "react_rails.bust_cache", group: :all do |app|
+      initializer "react_rails.bust_cache", after: :load_config_initializers, group: :all do |app|
         asset_variant = React::Rails::AssetVariant.new({
           variant: app.config.react.variant,
           addons: app.config.react.addons,

test/dummy/config/initializers/react.rb

@@ -1,2 +1,9 @@
 # Override setting set in application.rb
-Rails.application.config.react.addons = true
+class CustomComponentMount < React::Rails::ComponentMount
+end
+
+Dummy::Application.configure do
+  config.react.addons = true
+  config.react.view_helper_implementation = CustomComponentMount
+end
+

test/react/rails/view_helper_test.rb

@@ -17,4 +17,11 @@ class ViewHelperTest < ActionView::TestCase
     render template: "pages/show"
     assert_includes(rendered, "React-Rails")
   end
+
+  test "view helper uses the implementation class set in the initializer" do
+    assert_equal(
+      React::Rails::ViewHelper.helper_implementation_class.to_s,
+      "CustomComponentMount"
+    )
+  end
 end