Labyricorn/liquid
1
0
Fork 0
mirror of https://github.com/Shopify/liquid.git synced 2026-09-15 08:50:45 -07:00
Code Issues Packages Projects Releases Wiki Activity

Raise FilterNotFound on use of non-existent filter

Browse Source
This commit is contained in:
Brian Candler
2009-06-06 16:32:20 +01:00
parent f29b9335c5
commit 01c25a11a3
3 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/liquid/context.rb
+1 -1
View File
@@ -56,7 +56,7 @@ module Liquid
if strainer.respond_to?(method) if strainer.respond_to?(method)
strainer.__send__(method, *args) strainer.__send__(method, *args)
else else
args.first raise FilterNotFound, "Filter '#{method}' not found"
end end
end end
test/context_test.rb
+3 -1
View File
@@ -156,7 +156,9 @@ class ContextTest < Test::Unit::TestCase
assert_equal 'hi? hi!', context.invoke(:hi, 'hi?') assert_equal 'hi? hi!', context.invoke(:hi, 'hi?')
context = Context.new(@template) context = Context.new(@template)
assert_equal 'hi?', context.invoke(:hi, 'hi?') assert_raises(FilterNotFound) {
context.invoke(:hi, 'hi?')
}
context.add_filters(filter) context.add_filters(filter)
assert_equal 'hi? hi!', context.invoke(:hi, 'hi?') assert_equal 'hi? hi!', context.invoke(:hi, 'hi?')
test/security_test.rb
+4 -4
View File
@@ -11,14 +11,14 @@ class SecurityTest < Test::Unit::TestCase
def test_no_instance_eval def test_no_instance_eval
text = %( {{ '1+1' | instance_eval }} ) text = %( {{ '1+1' | instance_eval }} )
expected = %| 1+1 | expected = %! Liquid error: Error - filter 'instance_eval' in ''1+1' | instance_eval' could not be found. !
assert_equal expected, Template.parse(text).render(@assigns) assert_equal expected, Template.parse(text).render(@assigns)
end end
def test_no_existing_instance_eval def test_no_existing_instance_eval
text = %( {{ '1+1' | __instance_eval__ }} ) text = %( {{ '1+1' | __instance_eval__ }} )
expected = %| 1+1 | expected = %! Liquid error: Error - filter '__instance_eval__' in ''1+1' | __instance_eval__' could not be found. !
assert_equal expected, Template.parse(text).render(@assigns) assert_equal expected, Template.parse(text).render(@assigns)
end end
@@ -26,7 +26,7 @@ class SecurityTest < Test::Unit::TestCase
def test_no_instance_eval_after_mixing_in_new_filter def test_no_instance_eval_after_mixing_in_new_filter
text = %( {{ '1+1' | instance_eval }} ) text = %( {{ '1+1' | instance_eval }} )
expected = %| 1+1 | expected = %! Liquid error: Error - filter 'instance_eval' in ''1+1' | instance_eval' could not be found. !
assert_equal expected, Template.parse(text).render(@assigns) assert_equal expected, Template.parse(text).render(@assigns)
end end
@@ -34,7 +34,7 @@ class SecurityTest < Test::Unit::TestCase
def test_no_instance_eval_later_in_chain def test_no_instance_eval_later_in_chain
text = %( {{ '1+1' | add_one | instance_eval }} ) text = %( {{ '1+1' | add_one | instance_eval }} )
expected = %| 1+1 + 1 | expected = %! Liquid error: Error - filter 'instance_eval' in ''1+1' | add_one | instance_eval' could not be found. !
assert_equal expected, Template.parse(text).render(@assigns, :filters => SecurityFilter) assert_equal expected, Template.parse(text).render(@assigns, :filters => SecurityFilter)
end end