liquid 4.0.0.rc3 → 5.0.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +5 -5
- data/History.md +93 -2
- data/README.md +8 -0
- data/lib/liquid.rb +18 -5
- data/lib/liquid/block.rb +47 -20
- data/lib/liquid/block_body.rb +190 -76
- data/lib/liquid/condition.rb +69 -29
- data/lib/liquid/context.rb +122 -76
- data/lib/liquid/document.rb +47 -9
- data/lib/liquid/drop.rb +4 -2
- data/lib/liquid/errors.rb +20 -25
- data/lib/liquid/expression.rb +30 -31
- data/lib/liquid/extensions.rb +8 -0
- data/lib/liquid/file_system.rb +6 -4
- data/lib/liquid/forloop_drop.rb +11 -4
- data/lib/liquid/i18n.rb +5 -3
- data/lib/liquid/interrupts.rb +3 -1
- data/lib/liquid/lexer.rb +35 -26
- data/lib/liquid/locales/en.yml +4 -2
- data/lib/liquid/parse_context.rb +17 -4
- data/lib/liquid/parse_tree_visitor.rb +42 -0
- data/lib/liquid/parser.rb +30 -18
- data/lib/liquid/parser_switching.rb +17 -3
- data/lib/liquid/partial_cache.rb +24 -0
- data/lib/liquid/profiler.rb +67 -86
- data/lib/liquid/profiler/hooks.rb +26 -14
- data/lib/liquid/range_lookup.rb +5 -3
- data/lib/liquid/register.rb +6 -0
- data/lib/liquid/resource_limits.rb +47 -8
- data/lib/liquid/standardfilters.rb +171 -57
- data/lib/liquid/static_registers.rb +44 -0
- data/lib/liquid/strainer_factory.rb +36 -0
- data/lib/liquid/strainer_template.rb +53 -0
- data/lib/liquid/tablerowloop_drop.rb +6 -4
- data/lib/liquid/tag.rb +28 -6
- data/lib/liquid/tag/disableable.rb +22 -0
- data/lib/liquid/tag/disabler.rb +21 -0
- data/lib/liquid/tags/assign.rb +32 -10
- data/lib/liquid/tags/break.rb +8 -3
- data/lib/liquid/tags/capture.rb +11 -8
- data/lib/liquid/tags/case.rb +41 -27
- data/lib/liquid/tags/comment.rb +5 -3
- data/lib/liquid/tags/continue.rb +8 -3
- data/lib/liquid/tags/cycle.rb +35 -16
- data/lib/liquid/tags/decrement.rb +6 -3
- data/lib/liquid/tags/echo.rb +26 -0
- data/lib/liquid/tags/for.rb +79 -47
- data/lib/liquid/tags/if.rb +53 -30
- data/lib/liquid/tags/ifchanged.rb +11 -10
- data/lib/liquid/tags/include.rb +42 -44
- data/lib/liquid/tags/increment.rb +7 -3
- data/lib/liquid/tags/raw.rb +14 -11
- data/lib/liquid/tags/render.rb +84 -0
- data/lib/liquid/tags/table_row.rb +32 -20
- data/lib/liquid/tags/unless.rb +15 -15
- data/lib/liquid/template.rb +60 -71
- data/lib/liquid/template_factory.rb +9 -0
- data/lib/liquid/tokenizer.rb +17 -9
- data/lib/liquid/usage.rb +8 -0
- data/lib/liquid/utils.rb +6 -4
- data/lib/liquid/variable.rb +55 -38
- data/lib/liquid/variable_lookup.rb +14 -6
- data/lib/liquid/version.rb +3 -1
- data/test/integration/assign_test.rb +74 -5
- data/test/integration/blank_test.rb +11 -8
- data/test/integration/block_test.rb +58 -0
- data/test/integration/capture_test.rb +18 -10
- data/test/integration/context_test.rb +608 -5
- data/test/integration/document_test.rb +4 -2
- data/test/integration/drop_test.rb +67 -83
- data/test/integration/error_handling_test.rb +90 -60
- data/test/integration/expression_test.rb +46 -0
- data/test/integration/filter_test.rb +53 -42
- data/test/integration/hash_ordering_test.rb +5 -3
- data/test/integration/output_test.rb +26 -24
- data/test/integration/parsing_quirks_test.rb +24 -8
- data/test/integration/{render_profiling_test.rb → profiler_test.rb} +84 -25
- data/test/integration/security_test.rb +41 -18
- data/test/integration/standard_filter_test.rb +523 -205
- data/test/integration/tag/disableable_test.rb +59 -0
- data/test/integration/tag_test.rb +45 -0
- data/test/integration/tags/break_tag_test.rb +4 -2
- data/test/integration/tags/continue_tag_test.rb +4 -2
- data/test/integration/tags/echo_test.rb +13 -0
- data/test/integration/tags/for_tag_test.rb +109 -53
- data/test/integration/tags/if_else_tag_test.rb +5 -3
- data/test/integration/tags/include_tag_test.rb +83 -52
- data/test/integration/tags/increment_tag_test.rb +4 -2
- data/test/integration/tags/liquid_tag_test.rb +116 -0
- data/test/integration/tags/raw_tag_test.rb +14 -11
- data/test/integration/tags/render_tag_test.rb +213 -0
- data/test/integration/tags/standard_tag_test.rb +38 -31
- data/test/integration/tags/statements_test.rb +23 -21
- data/test/integration/tags/table_row_test.rb +2 -0
- data/test/integration/tags/unless_else_tag_test.rb +4 -2
- data/test/integration/template_test.rb +128 -121
- data/test/integration/trim_mode_test.rb +82 -44
- data/test/integration/variable_test.rb +46 -31
- data/test/test_helper.rb +75 -23
- data/test/unit/block_unit_test.rb +19 -24
- data/test/unit/condition_unit_test.rb +82 -72
- data/test/unit/file_system_unit_test.rb +6 -4
- data/test/unit/i18n_unit_test.rb +7 -5
- data/test/unit/lexer_unit_test.rb +12 -10
- data/test/unit/parse_tree_visitor_test.rb +247 -0
- data/test/unit/parser_unit_test.rb +37 -35
- data/test/unit/partial_cache_unit_test.rb +128 -0
- data/test/unit/regexp_unit_test.rb +17 -15
- data/test/unit/static_registers_unit_test.rb +156 -0
- data/test/unit/strainer_factory_unit_test.rb +100 -0
- data/test/unit/strainer_template_unit_test.rb +82 -0
- data/test/unit/tag_unit_test.rb +5 -3
- data/test/unit/tags/case_tag_unit_test.rb +3 -1
- data/test/unit/tags/for_tag_unit_test.rb +4 -2
- data/test/unit/tags/if_tag_unit_test.rb +3 -1
- data/test/unit/template_factory_unit_test.rb +12 -0
- data/test/unit/template_unit_test.rb +19 -10
- data/test/unit/tokenizer_unit_test.rb +19 -17
- data/test/unit/variable_unit_test.rb +51 -49
- metadata +83 -50
- data/lib/liquid/strainer.rb +0 -65
- data/test/unit/context_unit_test.rb +0 -483
- data/test/unit/strainer_unit_test.rb +0 -136
@@ -0,0 +1,128 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
require 'test_helper'
|
4
|
+
|
5
|
+
class PartialCacheUnitTest < Minitest::Test
|
6
|
+
def test_uses_the_file_system_register_if_present
|
7
|
+
context = Liquid::Context.build(
|
8
|
+
registers: {
|
9
|
+
file_system: StubFileSystem.new('my_partial' => 'my partial body'),
|
10
|
+
}
|
11
|
+
)
|
12
|
+
|
13
|
+
partial = Liquid::PartialCache.load(
|
14
|
+
'my_partial',
|
15
|
+
context: context,
|
16
|
+
parse_context: Liquid::ParseContext.new
|
17
|
+
)
|
18
|
+
|
19
|
+
assert_equal('my partial body', partial.render)
|
20
|
+
end
|
21
|
+
|
22
|
+
def test_reads_from_the_file_system_only_once_per_file
|
23
|
+
file_system = StubFileSystem.new('my_partial' => 'some partial body')
|
24
|
+
context = Liquid::Context.build(
|
25
|
+
registers: { file_system: file_system }
|
26
|
+
)
|
27
|
+
|
28
|
+
2.times do
|
29
|
+
Liquid::PartialCache.load(
|
30
|
+
'my_partial',
|
31
|
+
context: context,
|
32
|
+
parse_context: Liquid::ParseContext.new
|
33
|
+
)
|
34
|
+
end
|
35
|
+
|
36
|
+
assert_equal(1, file_system.file_read_count)
|
37
|
+
end
|
38
|
+
|
39
|
+
def test_cache_state_is_stored_per_context
|
40
|
+
parse_context = Liquid::ParseContext.new
|
41
|
+
shared_file_system = StubFileSystem.new(
|
42
|
+
'my_partial' => 'my shared value'
|
43
|
+
)
|
44
|
+
context_one = Liquid::Context.build(
|
45
|
+
registers: {
|
46
|
+
file_system: shared_file_system,
|
47
|
+
}
|
48
|
+
)
|
49
|
+
context_two = Liquid::Context.build(
|
50
|
+
registers: {
|
51
|
+
file_system: shared_file_system,
|
52
|
+
}
|
53
|
+
)
|
54
|
+
|
55
|
+
2.times do
|
56
|
+
Liquid::PartialCache.load(
|
57
|
+
'my_partial',
|
58
|
+
context: context_one,
|
59
|
+
parse_context: parse_context
|
60
|
+
)
|
61
|
+
end
|
62
|
+
|
63
|
+
Liquid::PartialCache.load(
|
64
|
+
'my_partial',
|
65
|
+
context: context_two,
|
66
|
+
parse_context: parse_context
|
67
|
+
)
|
68
|
+
|
69
|
+
assert_equal(2, shared_file_system.file_read_count)
|
70
|
+
end
|
71
|
+
|
72
|
+
def test_cache_is_not_broken_when_a_different_parse_context_is_used
|
73
|
+
file_system = StubFileSystem.new('my_partial' => 'some partial body')
|
74
|
+
context = Liquid::Context.build(
|
75
|
+
registers: { file_system: file_system }
|
76
|
+
)
|
77
|
+
|
78
|
+
Liquid::PartialCache.load(
|
79
|
+
'my_partial',
|
80
|
+
context: context,
|
81
|
+
parse_context: Liquid::ParseContext.new(my_key: 'value one')
|
82
|
+
)
|
83
|
+
Liquid::PartialCache.load(
|
84
|
+
'my_partial',
|
85
|
+
context: context,
|
86
|
+
parse_context: Liquid::ParseContext.new(my_key: 'value two')
|
87
|
+
)
|
88
|
+
|
89
|
+
# Technically what we care about is that the file was parsed twice,
|
90
|
+
# but measuring file reads is an OK proxy for this.
|
91
|
+
assert_equal(1, file_system.file_read_count)
|
92
|
+
end
|
93
|
+
|
94
|
+
def test_uses_default_template_factory_when_no_template_factory_found_in_register
|
95
|
+
context = Liquid::Context.build(
|
96
|
+
registers: {
|
97
|
+
file_system: StubFileSystem.new('my_partial' => 'my partial body'),
|
98
|
+
}
|
99
|
+
)
|
100
|
+
|
101
|
+
partial = Liquid::PartialCache.load(
|
102
|
+
'my_partial',
|
103
|
+
context: context,
|
104
|
+
parse_context: Liquid::ParseContext.new
|
105
|
+
)
|
106
|
+
|
107
|
+
assert_equal('my partial body', partial.render)
|
108
|
+
end
|
109
|
+
|
110
|
+
def test_uses_template_factory_register_if_present
|
111
|
+
template_factory = StubTemplateFactory.new
|
112
|
+
context = Liquid::Context.build(
|
113
|
+
registers: {
|
114
|
+
file_system: StubFileSystem.new('my_partial' => 'my partial body'),
|
115
|
+
template_factory: template_factory,
|
116
|
+
}
|
117
|
+
)
|
118
|
+
|
119
|
+
partial = Liquid::PartialCache.load(
|
120
|
+
'my_partial',
|
121
|
+
context: context,
|
122
|
+
parse_context: Liquid::ParseContext.new
|
123
|
+
)
|
124
|
+
|
125
|
+
assert_equal('my partial body', partial.render)
|
126
|
+
assert_equal(1, template_factory.count)
|
127
|
+
end
|
128
|
+
end
|
@@ -1,44 +1,46 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
1
3
|
require 'test_helper'
|
2
4
|
|
3
5
|
class RegexpUnitTest < Minitest::Test
|
4
6
|
include Liquid
|
5
7
|
|
6
8
|
def test_empty
|
7
|
-
assert_equal
|
9
|
+
assert_equal([], ''.scan(QuotedFragment))
|
8
10
|
end
|
9
11
|
|
10
12
|
def test_quote
|
11
|
-
assert_equal
|
13
|
+
assert_equal(['"arg 1"'], '"arg 1"'.scan(QuotedFragment))
|
12
14
|
end
|
13
15
|
|
14
16
|
def test_words
|
15
|
-
assert_equal
|
17
|
+
assert_equal(['arg1', 'arg2'], 'arg1 arg2'.scan(QuotedFragment))
|
16
18
|
end
|
17
19
|
|
18
20
|
def test_tags
|
19
|
-
assert_equal
|
20
|
-
assert_equal
|
21
|
-
assert_equal
|
21
|
+
assert_equal(['<tr>', '</tr>'], '<tr> </tr>'.scan(QuotedFragment))
|
22
|
+
assert_equal(['<tr></tr>'], '<tr></tr>'.scan(QuotedFragment))
|
23
|
+
assert_equal(['<style', 'class="hello">', '</style>'], %(<style class="hello">' </style>).scan(QuotedFragment))
|
22
24
|
end
|
23
25
|
|
24
26
|
def test_double_quoted_words
|
25
|
-
assert_equal
|
27
|
+
assert_equal(['arg1', 'arg2', '"arg 3"'], 'arg1 arg2 "arg 3"'.scan(QuotedFragment))
|
26
28
|
end
|
27
29
|
|
28
30
|
def test_single_quoted_words
|
29
|
-
assert_equal
|
31
|
+
assert_equal(['arg1', 'arg2', "'arg 3'"], 'arg1 arg2 \'arg 3\''.scan(QuotedFragment))
|
30
32
|
end
|
31
33
|
|
32
34
|
def test_quoted_words_in_the_middle
|
33
|
-
assert_equal
|
35
|
+
assert_equal(['arg1', 'arg2', '"arg 3"', 'arg4'], 'arg1 arg2 "arg 3" arg4 '.scan(QuotedFragment))
|
34
36
|
end
|
35
37
|
|
36
38
|
def test_variable_parser
|
37
|
-
assert_equal
|
38
|
-
assert_equal
|
39
|
-
assert_equal
|
40
|
-
assert_equal
|
41
|
-
assert_equal
|
42
|
-
assert_equal
|
39
|
+
assert_equal(['var'], 'var'.scan(VariableParser))
|
40
|
+
assert_equal(['var', 'method'], 'var.method'.scan(VariableParser))
|
41
|
+
assert_equal(['var', '[method]'], 'var[method]'.scan(VariableParser))
|
42
|
+
assert_equal(['var', '[method]', '[0]'], 'var[method][0]'.scan(VariableParser))
|
43
|
+
assert_equal(['var', '["method"]', '[0]'], 'var["method"][0]'.scan(VariableParser))
|
44
|
+
assert_equal(['var', '[method]', '[0]', 'method'], 'var[method][0].method'.scan(VariableParser))
|
43
45
|
end
|
44
46
|
end # RegexpTest
|
@@ -0,0 +1,156 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
require 'test_helper'
|
4
|
+
|
5
|
+
class StaticRegistersUnitTest < Minitest::Test
|
6
|
+
include Liquid
|
7
|
+
|
8
|
+
def test_set
|
9
|
+
static_register = StaticRegisters.new(a: 1, b: 2)
|
10
|
+
static_register[:b] = 22
|
11
|
+
static_register[:c] = 33
|
12
|
+
|
13
|
+
assert_equal(1, static_register[:a])
|
14
|
+
assert_equal(22, static_register[:b])
|
15
|
+
assert_equal(33, static_register[:c])
|
16
|
+
end
|
17
|
+
|
18
|
+
def test_get_missing_key
|
19
|
+
static_register = StaticRegisters.new
|
20
|
+
|
21
|
+
assert_nil(static_register[:missing])
|
22
|
+
end
|
23
|
+
|
24
|
+
def test_delete
|
25
|
+
static_register = StaticRegisters.new(a: 1, b: 2)
|
26
|
+
static_register[:b] = 22
|
27
|
+
static_register[:c] = 33
|
28
|
+
|
29
|
+
assert_nil(static_register.delete(:a))
|
30
|
+
|
31
|
+
assert_equal(22, static_register.delete(:b))
|
32
|
+
|
33
|
+
assert_equal(33, static_register.delete(:c))
|
34
|
+
assert_nil(static_register[:c])
|
35
|
+
|
36
|
+
assert_nil(static_register.delete(:d))
|
37
|
+
end
|
38
|
+
|
39
|
+
def test_fetch
|
40
|
+
static_register = StaticRegisters.new(a: 1, b: 2)
|
41
|
+
static_register[:b] = 22
|
42
|
+
static_register[:c] = 33
|
43
|
+
|
44
|
+
assert_equal(1, static_register.fetch(:a))
|
45
|
+
assert_equal(1, static_register.fetch(:a, "default"))
|
46
|
+
assert_equal(22, static_register.fetch(:b))
|
47
|
+
assert_equal(22, static_register.fetch(:b, "default"))
|
48
|
+
assert_equal(33, static_register.fetch(:c))
|
49
|
+
assert_equal(33, static_register.fetch(:c, "default"))
|
50
|
+
|
51
|
+
assert_raises(KeyError) do
|
52
|
+
static_register.fetch(:d)
|
53
|
+
end
|
54
|
+
assert_equal("default", static_register.fetch(:d, "default"))
|
55
|
+
|
56
|
+
result = static_register.fetch(:d) { "default" }
|
57
|
+
assert_equal("default", result)
|
58
|
+
|
59
|
+
result = static_register.fetch(:d, "default 1") { "default 2" }
|
60
|
+
assert_equal("default 2", result)
|
61
|
+
end
|
62
|
+
|
63
|
+
def test_key
|
64
|
+
static_register = StaticRegisters.new(a: 1, b: 2)
|
65
|
+
static_register[:b] = 22
|
66
|
+
static_register[:c] = 33
|
67
|
+
|
68
|
+
assert_equal(true, static_register.key?(:a))
|
69
|
+
assert_equal(true, static_register.key?(:b))
|
70
|
+
assert_equal(true, static_register.key?(:c))
|
71
|
+
assert_equal(false, static_register.key?(:d))
|
72
|
+
end
|
73
|
+
|
74
|
+
def test_static_register_can_be_frozen
|
75
|
+
static_register = StaticRegisters.new(a: 1)
|
76
|
+
|
77
|
+
static_register.static.freeze
|
78
|
+
|
79
|
+
assert_raises(RuntimeError) do
|
80
|
+
static_register.static[:a] = "foo"
|
81
|
+
end
|
82
|
+
|
83
|
+
assert_raises(RuntimeError) do
|
84
|
+
static_register.static[:b] = "foo"
|
85
|
+
end
|
86
|
+
|
87
|
+
assert_raises(RuntimeError) do
|
88
|
+
static_register.static.delete(:a)
|
89
|
+
end
|
90
|
+
|
91
|
+
assert_raises(RuntimeError) do
|
92
|
+
static_register.static.delete(:c)
|
93
|
+
end
|
94
|
+
end
|
95
|
+
|
96
|
+
def test_new_static_retains_static
|
97
|
+
static_register = StaticRegisters.new(a: 1, b: 2)
|
98
|
+
static_register[:b] = 22
|
99
|
+
static_register[:c] = 33
|
100
|
+
|
101
|
+
new_static_register = StaticRegisters.new(static_register)
|
102
|
+
new_static_register[:b] = 222
|
103
|
+
|
104
|
+
newest_static_register = StaticRegisters.new(new_static_register)
|
105
|
+
newest_static_register[:c] = 333
|
106
|
+
|
107
|
+
assert_equal(1, static_register[:a])
|
108
|
+
assert_equal(22, static_register[:b])
|
109
|
+
assert_equal(33, static_register[:c])
|
110
|
+
|
111
|
+
assert_equal(1, new_static_register[:a])
|
112
|
+
assert_equal(222, new_static_register[:b])
|
113
|
+
assert_nil(new_static_register[:c])
|
114
|
+
|
115
|
+
assert_equal(1, newest_static_register[:a])
|
116
|
+
assert_equal(2, newest_static_register[:b])
|
117
|
+
assert_equal(333, newest_static_register[:c])
|
118
|
+
end
|
119
|
+
|
120
|
+
def test_multiple_instances_are_unique
|
121
|
+
static_register_1 = StaticRegisters.new(a: 1, b: 2)
|
122
|
+
static_register_1[:b] = 22
|
123
|
+
static_register_1[:c] = 33
|
124
|
+
|
125
|
+
static_register_2 = StaticRegisters.new(a: 10, b: 20)
|
126
|
+
static_register_2[:b] = 220
|
127
|
+
static_register_2[:c] = 330
|
128
|
+
|
129
|
+
assert_equal({ a: 1, b: 2 }, static_register_1.static)
|
130
|
+
assert_equal(1, static_register_1[:a])
|
131
|
+
assert_equal(22, static_register_1[:b])
|
132
|
+
assert_equal(33, static_register_1[:c])
|
133
|
+
|
134
|
+
assert_equal({ a: 10, b: 20 }, static_register_2.static)
|
135
|
+
assert_equal(10, static_register_2[:a])
|
136
|
+
assert_equal(220, static_register_2[:b])
|
137
|
+
assert_equal(330, static_register_2[:c])
|
138
|
+
end
|
139
|
+
|
140
|
+
def test_initialization_reused_static_same_memory_object
|
141
|
+
static_register_1 = StaticRegisters.new(a: 1, b: 2)
|
142
|
+
static_register_1[:b] = 22
|
143
|
+
static_register_1[:c] = 33
|
144
|
+
|
145
|
+
static_register_2 = StaticRegisters.new(static_register_1)
|
146
|
+
|
147
|
+
assert_equal(1, static_register_2[:a])
|
148
|
+
assert_equal(2, static_register_2[:b])
|
149
|
+
assert_nil(static_register_2[:c])
|
150
|
+
|
151
|
+
static_register_1.static[:b] = 222
|
152
|
+
static_register_1.static[:c] = 333
|
153
|
+
|
154
|
+
assert_same(static_register_1.static, static_register_2.static)
|
155
|
+
end
|
156
|
+
end
|
@@ -0,0 +1,100 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
require 'test_helper'
|
4
|
+
|
5
|
+
class StrainerFactoryUnitTest < Minitest::Test
|
6
|
+
include Liquid
|
7
|
+
|
8
|
+
module AccessScopeFilters
|
9
|
+
def public_filter
|
10
|
+
"public"
|
11
|
+
end
|
12
|
+
|
13
|
+
def private_filter
|
14
|
+
"private"
|
15
|
+
end
|
16
|
+
private :private_filter
|
17
|
+
end
|
18
|
+
|
19
|
+
StrainerFactory.add_global_filter(AccessScopeFilters)
|
20
|
+
|
21
|
+
module LateAddedFilter
|
22
|
+
def late_added_filter(_input)
|
23
|
+
"filtered"
|
24
|
+
end
|
25
|
+
end
|
26
|
+
|
27
|
+
def setup
|
28
|
+
@context = Context.build
|
29
|
+
end
|
30
|
+
|
31
|
+
def test_strainer
|
32
|
+
strainer = StrainerFactory.create(@context)
|
33
|
+
assert_equal(5, strainer.invoke('size', 'input'))
|
34
|
+
assert_equal("public", strainer.invoke("public_filter"))
|
35
|
+
end
|
36
|
+
|
37
|
+
def test_stainer_raises_argument_error
|
38
|
+
strainer = StrainerFactory.create(@context)
|
39
|
+
assert_raises(Liquid::ArgumentError) do
|
40
|
+
strainer.invoke("public_filter", 1)
|
41
|
+
end
|
42
|
+
end
|
43
|
+
|
44
|
+
def test_stainer_argument_error_contains_backtrace
|
45
|
+
strainer = StrainerFactory.create(@context)
|
46
|
+
|
47
|
+
exception = assert_raises(Liquid::ArgumentError) do
|
48
|
+
strainer.invoke("public_filter", 1)
|
49
|
+
end
|
50
|
+
|
51
|
+
assert_match(
|
52
|
+
/\ALiquid error: wrong number of arguments \((1 for 0|given 1, expected 0)\)\z/,
|
53
|
+
exception.message
|
54
|
+
)
|
55
|
+
assert_equal(exception.backtrace[0].split(':')[0], __FILE__)
|
56
|
+
end
|
57
|
+
|
58
|
+
def test_strainer_only_invokes_public_filter_methods
|
59
|
+
strainer = StrainerFactory.create(@context)
|
60
|
+
assert_equal(false, strainer.class.invokable?('__test__'))
|
61
|
+
assert_equal(false, strainer.class.invokable?('test'))
|
62
|
+
assert_equal(false, strainer.class.invokable?('instance_eval'))
|
63
|
+
assert_equal(false, strainer.class.invokable?('__send__'))
|
64
|
+
assert_equal(true, strainer.class.invokable?('size')) # from the standard lib
|
65
|
+
end
|
66
|
+
|
67
|
+
def test_strainer_returns_nil_if_no_filter_method_found
|
68
|
+
strainer = StrainerFactory.create(@context)
|
69
|
+
assert_nil(strainer.invoke("private_filter"))
|
70
|
+
assert_nil(strainer.invoke("undef_the_filter"))
|
71
|
+
end
|
72
|
+
|
73
|
+
def test_strainer_returns_first_argument_if_no_method_and_arguments_given
|
74
|
+
strainer = StrainerFactory.create(@context)
|
75
|
+
assert_equal("password", strainer.invoke("undef_the_method", "password"))
|
76
|
+
end
|
77
|
+
|
78
|
+
def test_strainer_only_allows_methods_defined_in_filters
|
79
|
+
strainer = StrainerFactory.create(@context)
|
80
|
+
assert_equal("1 + 1", strainer.invoke("instance_eval", "1 + 1"))
|
81
|
+
assert_equal("puts", strainer.invoke("__send__", "puts", "Hi Mom"))
|
82
|
+
assert_equal("has_method?", strainer.invoke("invoke", "has_method?", "invoke"))
|
83
|
+
end
|
84
|
+
|
85
|
+
def test_strainer_uses_a_class_cache_to_avoid_method_cache_invalidation
|
86
|
+
a = Module.new
|
87
|
+
b = Module.new
|
88
|
+
strainer = StrainerFactory.create(@context, [a, b])
|
89
|
+
assert_kind_of(StrainerTemplate, strainer)
|
90
|
+
assert_kind_of(a, strainer)
|
91
|
+
assert_kind_of(b, strainer)
|
92
|
+
assert_kind_of(Liquid::StandardFilters, strainer)
|
93
|
+
end
|
94
|
+
|
95
|
+
def test_add_global_filter_clears_cache
|
96
|
+
assert_equal('input', StrainerFactory.create(@context).invoke('late_added_filter', 'input'))
|
97
|
+
StrainerFactory.add_global_filter(LateAddedFilter)
|
98
|
+
assert_equal('filtered', StrainerFactory.create(nil).invoke('late_added_filter', 'input'))
|
99
|
+
end
|
100
|
+
end
|
@@ -0,0 +1,82 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
require 'test_helper'
|
4
|
+
|
5
|
+
class StrainerTemplateUnitTest < Minitest::Test
|
6
|
+
include Liquid
|
7
|
+
|
8
|
+
def test_add_filter_when_wrong_filter_class
|
9
|
+
c = Context.new
|
10
|
+
s = c.strainer
|
11
|
+
wrong_filter = ->(v) { v.reverse }
|
12
|
+
|
13
|
+
exception = assert_raises(TypeError) do
|
14
|
+
s.class.add_filter(wrong_filter)
|
15
|
+
end
|
16
|
+
assert_equal(exception.message, "wrong argument type Proc (expected Module)")
|
17
|
+
end
|
18
|
+
|
19
|
+
module PrivateMethodOverrideFilter
|
20
|
+
private
|
21
|
+
|
22
|
+
def public_filter
|
23
|
+
"overriden as private"
|
24
|
+
end
|
25
|
+
end
|
26
|
+
|
27
|
+
def test_add_filter_raises_when_module_privately_overrides_registered_public_methods
|
28
|
+
strainer = Context.new.strainer
|
29
|
+
|
30
|
+
error = assert_raises(Liquid::MethodOverrideError) do
|
31
|
+
strainer.class.add_filter(PrivateMethodOverrideFilter)
|
32
|
+
end
|
33
|
+
assert_equal('Liquid error: Filter overrides registered public methods as non public: public_filter', error.message)
|
34
|
+
end
|
35
|
+
|
36
|
+
module ProtectedMethodOverrideFilter
|
37
|
+
protected
|
38
|
+
|
39
|
+
def public_filter
|
40
|
+
"overriden as protected"
|
41
|
+
end
|
42
|
+
end
|
43
|
+
|
44
|
+
def test_add_filter_raises_when_module_overrides_registered_public_method_as_protected
|
45
|
+
strainer = Context.new.strainer
|
46
|
+
|
47
|
+
error = assert_raises(Liquid::MethodOverrideError) do
|
48
|
+
strainer.class.add_filter(ProtectedMethodOverrideFilter)
|
49
|
+
end
|
50
|
+
assert_equal('Liquid error: Filter overrides registered public methods as non public: public_filter', error.message)
|
51
|
+
end
|
52
|
+
|
53
|
+
module PublicMethodOverrideFilter
|
54
|
+
def public_filter
|
55
|
+
"public"
|
56
|
+
end
|
57
|
+
end
|
58
|
+
|
59
|
+
def test_add_filter_does_not_raise_when_module_overrides_previously_registered_method
|
60
|
+
strainer = Context.new.strainer
|
61
|
+
with_global_filter do
|
62
|
+
strainer.class.add_filter(PublicMethodOverrideFilter)
|
63
|
+
assert(strainer.class.send(:filter_methods).include?('public_filter'))
|
64
|
+
end
|
65
|
+
end
|
66
|
+
|
67
|
+
def test_add_filter_does_not_include_already_included_module
|
68
|
+
mod = Module.new do
|
69
|
+
class << self
|
70
|
+
attr_accessor :include_count
|
71
|
+
def included(_mod)
|
72
|
+
self.include_count += 1
|
73
|
+
end
|
74
|
+
end
|
75
|
+
self.include_count = 0
|
76
|
+
end
|
77
|
+
strainer = Context.new.strainer
|
78
|
+
strainer.class.add_filter(mod)
|
79
|
+
strainer.class.add_filter(mod)
|
80
|
+
assert_equal(1, mod.include_count)
|
81
|
+
end
|
82
|
+
end
|