Permalink
Browse files

Merge pull request #2985 from kennyj/master

fix #2582 (with testcase)
  • Loading branch information...
2 parents 8397a56 + 3ce8302 commit d762e22c048ac67979fec7ecfb71bd999a1004d9 @spastorino spastorino committed Sep 12, 2011
Showing with 17 additions and 1 deletion.
  1. +1 −1 actionpack/lib/sprockets/assets.rake
  2. +16 −0 railties/test/application/rake_test.rb
@@ -44,7 +44,7 @@ namespace :assets do
end
end
- File.open("#{manifest_path}/manifest.yml", 'w') do |f|
+ File.open("#{manifest_path}/manifest.yml", 'wb') do |f|
YAML.dump(manifest, f)
end
end
@@ -1,3 +1,4 @@
+# coding:utf-8
require "isolation/abstract_unit"
module ApplicationTests
@@ -200,5 +201,20 @@ def test_scaffold_tests_pass_by_default
assert_match(/7 tests, 10 assertions, 0 failures, 0 errors/, content)
end
+
+ def test_assets_precompile_with_utf8_filename
+ add_to_config <<-RUBY
+ config.assets.precompile = [ /\.png$$/, /application.(css|js)$/ ]
+ RUBY
+
+ Dir.chdir(app_path) do
+ `cp app/assets/images/rails.png app/assets/images/レイルズ.png`
+ `rake assets:precompile`
+ open("public/assets/manifest.yml") do |f|
+ assert_match(/レイルズ.png/, f.read)
+ end
+ end
+ end
+
end
end

0 comments on commit d762e22

Please sign in to comment.