Merge pull request #347 from dv/reposition-asset-build-to-fix-initializers

Reposition asset path build

Author: Robert Mosolgo

lib/react/rails/railtie.rb

@@ -51,7 +51,7 @@ class Railtie < ::Rails::Railtie
 
       end
 
-      config.before_initialize do |app|
+      initializer "react_rails.set_variant", after: :engines_blank_point, group: :all do |app|
         asset_variant = React::Rails::AssetVariant.new({
           variant: app.config.react.variant,
           addons: app.config.react.addons,

test/dummy/config/application.rb

@@ -26,6 +26,7 @@ class Application < Rails::Application
     # config.i18n.load_path += Dir[Rails.root.join('my', 'locales', '*.{rb,yml}').to_s]
     # config.i18n.default_locale = :de
     config.react.variant = :production
+    config.react.addons = false
 
     config.assets.enabled = true
   end

test/dummy/config/initializers/react.rb

@@ -0,0 +1,2 @@
+# Override setting set in application.rb
+Rails.application.config.react.addons = true

test/react_test.rb

@@ -35,8 +35,8 @@ class ReactTest < ActionDispatch::IntegrationTest
     end
   end
 
-  test "the development version is loaded" do
+  test "the development version with addons is loaded" do
     asset = Rails.application.assets.find_asset('react')
-    assert asset.pathname.to_s.end_with?('development/react.js')
+    assert asset.pathname.to_s.end_with?('development-with-addons/react.js')
   end
 end