diff options
| author | Dmitry Vorotilin <d.vorotilin@gmail.com> | 2012-09-11 00:58:48 +0400 |
|---|---|---|
| committer | Dmitry Vorotilin <d.vorotilin@gmail.com> | 2012-09-11 01:00:18 +0400 |
| commit | 551066633d83939b1077eef38c3e2d449b230849 (patch) | |
| tree | cc0ad141f788ed188749272d7fce3770c713fee0 /railties | |
| parent | be8685d9227faf5d3691346e2cc0c4f14d833c53 (diff) | |
| download | rails-551066633d83939b1077eef38c3e2d449b230849.tar.gz rails-551066633d83939b1077eef38c3e2d449b230849.tar.bz2 rails-551066633d83939b1077eef38c3e2d449b230849.zip | |
Rake assets:precompile shouldn't fail quietly.
We should show to user original issue if rake task will be aborted.
Diffstat (limited to 'railties')
| -rw-r--r-- | railties/test/application/assets_test.rb | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/railties/test/application/assets_test.rb b/railties/test/application/assets_test.rb index 9c402fab02..841828e634 100644 --- a/railties/test/application/assets_test.rb +++ b/railties/test/application/assets_test.rb @@ -19,13 +19,10 @@ module ApplicationTests def precompile!(env = nil) quietly do - precompile_task = 'bundle exec rake assets:precompile' - precompile_task += ' ' + env if env - out = Dir.chdir(app_path){ %x[ #{precompile_task} ] } - assert $?.exitstatus == 0, - "#{precompile_task} has failed: #{out}.\ - Probably you didn't install JavaScript runtime." - return out + precompile_task = "bundle exec rake assets:precompile #{env} 2>&1" + output = Dir.chdir(app_path) { %x[ #{precompile_task} ] } + assert $?.success?, output + output end end |