mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
70e684a681
A declarative API for specifying dependencies that affect template cache digest computation. In your controller, specify any of said dependencies: view_cache_dependency { "phone" if using_phone? } When the block is evaluated, the resulting value is included in the cache digest calculation, allowing you to generate different digests for effectively the same template. (Mostly useful if you're mucking with template load paths.)
190 lines
4.9 KiB
Ruby
190 lines
4.9 KiB
Ruby
require 'abstract_unit'
|
|
require 'fileutils'
|
|
|
|
class FixtureTemplate
|
|
attr_reader :source
|
|
|
|
def initialize(template_path)
|
|
@source = File.read(template_path)
|
|
rescue Errno::ENOENT
|
|
raise ActionView::MissingTemplate.new([], "", [], true, [])
|
|
end
|
|
end
|
|
|
|
class FixtureFinder
|
|
FIXTURES_DIR = "#{File.dirname(__FILE__)}/../fixtures/digestor"
|
|
|
|
def find(logical_name, keys, partial, options)
|
|
FixtureTemplate.new("digestor/#{partial ? logical_name.gsub(%r|/([^/]+)$|, '/_\1') : logical_name}.#{options[:formats].first}.erb")
|
|
end
|
|
end
|
|
|
|
class TemplateDigestorTest < ActionView::TestCase
|
|
def setup
|
|
@cwd = Dir.pwd
|
|
@tmp_dir = Dir.mktmpdir
|
|
|
|
FileUtils.cp_r FixtureFinder::FIXTURES_DIR, @tmp_dir
|
|
Dir.chdir @tmp_dir
|
|
end
|
|
|
|
def teardown
|
|
Dir.chdir @cwd
|
|
FileUtils.rm_r @tmp_dir
|
|
ActionView::Digestor.cache.clear
|
|
end
|
|
|
|
def test_top_level_change_reflected
|
|
assert_digest_difference("messages/show") do
|
|
change_template("messages/show")
|
|
end
|
|
end
|
|
|
|
def test_explicit_dependency
|
|
assert_digest_difference("messages/show") do
|
|
change_template("messages/_message")
|
|
end
|
|
end
|
|
|
|
def test_explicit_dependency_in_multiline_erb_tag
|
|
assert_digest_difference("messages/show") do
|
|
change_template("messages/_form")
|
|
end
|
|
end
|
|
|
|
def test_second_level_dependency
|
|
assert_digest_difference("messages/show") do
|
|
change_template("comments/_comments")
|
|
end
|
|
end
|
|
|
|
def test_second_level_dependency_within_same_directory
|
|
assert_digest_difference("messages/show") do
|
|
change_template("messages/_header")
|
|
end
|
|
end
|
|
|
|
def test_third_level_dependency
|
|
assert_digest_difference("messages/show") do
|
|
change_template("comments/_comment")
|
|
end
|
|
end
|
|
|
|
def test_directory_depth_dependency
|
|
assert_digest_difference("level/below/index") do
|
|
change_template("level/below/_header")
|
|
end
|
|
end
|
|
|
|
def test_logging_of_missing_template
|
|
assert_logged "Couldn't find template for digesting: messages/something_missing.html" do
|
|
digest("messages/show")
|
|
end
|
|
end
|
|
|
|
def test_nested_template_directory
|
|
assert_digest_difference("messages/show") do
|
|
change_template("messages/actions/_move")
|
|
end
|
|
end
|
|
|
|
def test_dont_generate_a_digest_for_missing_templates
|
|
assert_equal '', digest("nothing/there")
|
|
end
|
|
|
|
def test_collection_dependency
|
|
assert_digest_difference("messages/index") do
|
|
change_template("messages/_message")
|
|
end
|
|
|
|
assert_digest_difference("messages/index") do
|
|
change_template("events/_event")
|
|
end
|
|
end
|
|
|
|
def test_collection_derived_from_record_dependency
|
|
assert_digest_difference("messages/show") do
|
|
change_template("events/_event")
|
|
end
|
|
end
|
|
|
|
def test_extra_whitespace_in_render_partial
|
|
assert_digest_difference("messages/edit") do
|
|
change_template("messages/_form")
|
|
end
|
|
end
|
|
|
|
def test_extra_whitespace_in_render_named_partial
|
|
assert_digest_difference("messages/edit") do
|
|
change_template("messages/_header")
|
|
end
|
|
end
|
|
|
|
def test_extra_whitespace_in_render_record
|
|
assert_digest_difference("messages/edit") do
|
|
change_template("messages/_message")
|
|
end
|
|
end
|
|
|
|
def test_extra_whitespace_in_render_with_parenthesis
|
|
assert_digest_difference("messages/edit") do
|
|
change_template("events/_event")
|
|
end
|
|
end
|
|
|
|
def test_old_style_hash_in_render_invocation
|
|
assert_digest_difference("messages/edit") do
|
|
change_template("comments/_comment")
|
|
end
|
|
end
|
|
|
|
def test_dependencies_via_options_results_in_different_digest
|
|
digest_plain = digest("comments/_comment")
|
|
digest_fridge = digest("comments/_comment", dependencies: ["fridge"])
|
|
digest_phone = digest("comments/_comment", dependencies: ["phone"])
|
|
digest_fridge_phone = digest("comments/_comment", dependencies: ["fridge", "phone"])
|
|
|
|
assert_not_equal digest_plain, digest_fridge
|
|
assert_not_equal digest_plain, digest_phone
|
|
assert_not_equal digest_plain, digest_fridge_phone
|
|
assert_not_equal digest_fridge, digest_phone
|
|
assert_not_equal digest_fridge, digest_fridge_phone
|
|
assert_not_equal digest_phone, digest_fridge_phone
|
|
end
|
|
|
|
private
|
|
def assert_logged(message)
|
|
old_logger = ActionView::Base.logger
|
|
log = StringIO.new
|
|
ActionView::Base.logger = Logger.new(log)
|
|
|
|
begin
|
|
yield
|
|
|
|
log.rewind
|
|
assert_match message, log.read
|
|
ensure
|
|
ActionView::Base.logger = old_logger
|
|
end
|
|
end
|
|
|
|
def assert_digest_difference(template_name)
|
|
previous_digest = digest(template_name)
|
|
ActionView::Digestor.cache.clear
|
|
|
|
yield
|
|
|
|
assert previous_digest != digest(template_name), "digest didn't change"
|
|
ActionView::Digestor.cache.clear
|
|
end
|
|
|
|
def digest(template_name, options={})
|
|
ActionView::Digestor.digest(template_name, :html, FixtureFinder.new, options)
|
|
end
|
|
|
|
def change_template(template_name)
|
|
File.open("digestor/#{template_name}.html.erb", "w") do |f|
|
|
f.write "\nTHIS WAS CHANGED!"
|
|
end
|
|
end
|
|
end
|