Skip to content

Commit cf057c9

Browse files
committed
Merge pull request #1 from rmosolgo/dont-crash-on-boot-because-of-file-copying
fix(Railtie) add asset paths another way, add test for precompiling assets
2 parents 260cb28 + a3d0dad commit cf057c9

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

lib/react/rails/railtie.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,8 @@ class Railtie < ::Rails::Railtie
3636
directory = app.config.react.variant == :production ? 'production' : 'development'
3737
directory += '-with-addons' if app.config.react.addons
3838

39-
app.assets.append_path(root_path.join('lib/assets/react-source/').join(directory).to_s)
40-
app.assets.append_path(root_path.join('lib/assets/javascripts/').to_s)
39+
app.config.assets.paths << root_path.join('lib/assets/react-source/').join(directory).to_s
40+
app.config.assets.paths << root_path.join('lib/assets/javascripts/').to_s
4141
end
4242

4343
config.after_initialize do |app|

test/react_test.rb

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,4 +16,10 @@ class ReactTest < ActionDispatch::IntegrationTest
1616
assert_response :success
1717
assert_equal react_file_token, @response.body
1818
end
19+
20+
test 'precompiling assets works' do
21+
Dummy::Application.load_tasks
22+
Rake::Task['assets:precompile'].invoke
23+
FileUtils.rm_r(File.expand_path("../dummy/public/assets", __FILE__))
24+
end
1925
end

0 commit comments

Comments
 (0)