mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
Merge pull request #20058 from farukaydin/cache-digests-dependency-rake
add error log that notify 'file not found' when using cache_digest dependency rake
This commit is contained in:
commit
fcbd2e821e
2 changed files with 22 additions and 1 deletions
|
@ -70,7 +70,8 @@ module ActionView
|
|||
def dependencies
|
||||
DependencyTracker.find_dependencies(name, template)
|
||||
rescue ActionView::MissingTemplate
|
||||
[] # File doesn't exist, so no dependencies
|
||||
logger.try :error, " '#{name}' file doesn't exist, so no dependencies"
|
||||
[]
|
||||
end
|
||||
|
||||
def nested_dependencies
|
||||
|
|
|
@ -111,6 +111,18 @@ class TemplateDigestorTest < ActionView::TestCase
|
|||
end
|
||||
end
|
||||
|
||||
def test_logging_of_missing_template_for_dependencies
|
||||
assert_logged "'messages/something_missing' file doesn't exist, so no dependencies" do
|
||||
dependencies("messages/something_missing")
|
||||
end
|
||||
end
|
||||
|
||||
def test_logging_of_missing_template_for_nested_dependencies
|
||||
assert_logged "'messages/something_missing' file doesn't exist, so no dependencies" do
|
||||
nested_dependencies("messages/something_missing")
|
||||
end
|
||||
end
|
||||
|
||||
def test_nested_template_directory
|
||||
assert_digest_difference("messages/show") do
|
||||
change_template("messages/actions/_move")
|
||||
|
@ -298,6 +310,14 @@ class TemplateDigestorTest < ActionView::TestCase
|
|||
ActionView::Digestor.digest({ name: template_name, finder: finder }.merge(options))
|
||||
end
|
||||
|
||||
def dependencies(template_name)
|
||||
ActionView::Digestor.new({ name: template_name, finder: finder }).dependencies
|
||||
end
|
||||
|
||||
def nested_dependencies(template_name)
|
||||
ActionView::Digestor.new({ name: template_name, finder: finder }).nested_dependencies
|
||||
end
|
||||
|
||||
def finder
|
||||
@finder ||= FixtureFinder.new
|
||||
end
|
||||
|
|
Loading…
Reference in a new issue