Skip to content

Commit 45cab24

Browse files
committed
Merge pull request #144 from matthewd/ignore_missing_dependencies
Ignore missing dependencies, instead of failing
2 parents 3970696 + 55184e6 commit 45cab24

File tree

3 files changed

+7
-2
lines changed

3 files changed

+7
-2
lines changed

lib/sprockets/rails/helper.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,7 @@ def stylesheet_link_tag(*sources)
166166
def check_dependencies!(dep)
167167
depend_on(dep)
168168
depend_on_asset(dep)
169+
rescue Sprockets::FileNotFound
169170
end
170171

171172
# Raise errors when source does not exist or is not in the precompiled list

test/fixtures/error/missing.css.erb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
p { background: url(<%= image_path "does_not_exist.png" %>); }

test/test_helper.rb

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -450,8 +450,11 @@ def test_asset_digest
450450
class AutomaticDependenciesFromHelpersTest < HelperTest
451451

452452
def test_dependency_added
453-
@view.asset_path("url.css")
454-
455453
assert_equal ["logo.png", "url.css.erb"], @assets['url.css'].send(:dependency_paths).map {|d| File.basename(d.pathname) }.sort
456454
end
455+
456+
def test_ignores_missing_dependencies
457+
assert_equal ["missing.css.erb"], @assets['error/missing.css'].send(:dependency_paths).map {|d| File.basename(d.pathname) }.sort
458+
end
459+
457460
end

0 commit comments

Comments
 (0)