Move tests for VariableLookup

lib/liquid/variable_lookup.rb

@@ -110,10 +110,11 @@ module Liquid
     def to_s
       str = name.dup
       lookups.each do |lookup|
+        str +=
           if lookup.instance_of?(String)
-          str += '.' + lookup
+            '.' + lookup
           else
-          str += '[' + lookup.to_s + ']'
+            '[' + lookup.to_s + ']'
           end
       end
       str

test/unit/variable_lookup_unit_test.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class VariableLookupUnitTest < Minitest::Test
+  include Liquid
+
+  def test_variable_lookup_parsing
+    lookup = parse_variable_lookup('a.b.c')
+    assert_equal('a', lookup.name)
+    assert_equal(['b', 'c'], lookup.lookups)
+
+    lookup = parse_variable_lookup('a[b]')
+    assert_equal('a', lookup.name)
+    assert_equal([parse_variable_lookup('b')], lookup.lookups)
+  end
+
+  def test_to_s
+    lookup = parse_variable_lookup('a.b.c')
+    assert_equal('a.b.c', lookup.to_s)
+
+    lookup = parse_variable_lookup('a[b.c].d')
+    assert_equal('a[b.c].d', lookup.to_s)
+  end
+
+  private
+
+  def parse_variable_lookup(markup)
+    if Liquid::Template.error_mode == :strict
+      p = Liquid::Parser.new(markup)
+      VariableLookup.strict_parse(p)
+    else
+      VariableLookup.lax_parse(markup)
+    end
+  end
+end

test/unit/variable_unit_test.rb

@@ -152,12 +152,6 @@ class VariableUnitTest < Minitest::Test
     assert_equal(" name_of_variable | upcase ", var.raw)
   end
 
-  def test_variable_lookup_interface
-    lookup = parse_variable_lookup('a.b.c')
-    assert_equal('a', lookup.name)
-    assert_equal(['b', 'c'], lookup.lookups)
-  end
-
   private
 
   def parse_variable_lookup(markup)