diff --git a/lib/liquid/expression.rb b/lib/liquid/expression.rb index 1bf7edcf..a0112f66 100644 --- a/lib/liquid/expression.rb +++ b/lib/liquid/expression.rb @@ -48,7 +48,7 @@ module Liquid end def inner_parse(markup, ss, cache) - if (markup.start_with?("(") && markup.end_with?(")")) && markup =~ RANGES_REGEX + if markup.start_with?("(") && markup.end_with?(")") && markup =~ RANGES_REGEX start_markup = Regexp.last_match(1) end_markup = Regexp.last_match(2) start_obj = parse(start_markup, ss, cache) diff --git a/test/unit/parser_unit_test.rb b/test/unit/parser_unit_test.rb index cf0d8c7b..cee971d1 100644 --- a/test/unit/parser_unit_test.rb +++ b/test/unit/parser_unit_test.rb @@ -74,7 +74,7 @@ class ParserUnitTest < Minitest::Test assert_equal('wut', p.expression) assert_equal(true, p.expression) assert_equal(false, p.expression) - assert_equal((0..5), p.expression) + assert_equal(0..5, p.expression) end def test_number @@ -104,11 +104,11 @@ class ParserUnitTest < Minitest::Test def test_range_lookup p = new_parser('(0..5) (a..b)') - assert_equal((0..5), p.expression) + assert_equal(0..5, p.expression) r2 = p.expression assert(r2.is_a?(RangeLookup)) - assert_equal((1..4), r2.evaluate(Context.new({ 'a' => 1, 'b' => 4 }))) + assert_equal(1..4, r2.evaluate(Context.new({ 'a' => 1, 'b' => 4 }))) end def test_ranges