fix: parse Selectors::ComplexSelectors properly

- avoid clobbering the combinator method with a local variable
- return a recursive tree of complex selectors

Partial fix for #25

Author: flavorjones

lib/syntax_tree/css/selectors.rb

@@ -339,25 +339,19 @@ def relative_selector_list
       def complex_selector
         left = compound_selector
 
-        loop do
-          if (combinator = maybe { combinator })
-            ComplexSelector.new(left: left, combinator: combinator, right: compound_selector)
-          elsif (right = maybe { compound_selector })
-            ComplexSelector.new(left: left, combinator: nil, right: right)
-          else
-            break
-          end
+        if (c = maybe { combinator })
+          ComplexSelector.new(left: left, combinator: c, right: complex_selector)
+        elsif (right = maybe { complex_selector })
+          ComplexSelector.new(left: left, combinator: nil, right: right)
+        else
+          left
         end
-
-        left
       end
 
       # <relative-selector> = <combinator>? <complex-selector>
       def relative_selector
-        combinator = maybe { combinator }
-
-        if combinator
-          RelativeSelector.new(combinator: combinator, complex_selector: complex_selector)
+        if (c = maybe { combinator })
+          RelativeSelector.new(combinator: c, complex_selector: complex_selector)
         else
           complex_selector
         end

test/selectors_test.rb

@@ -65,6 +65,38 @@ class SelectorsTest < Minitest::Spec
         end
       end
 
+      it "parses a complex selector" do
+        actual = parse_selectors("section>table")
+
+        assert_pattern do
+          actual => [
+            Selectors::ComplexSelector[
+              left: Selectors::TypeSelector[value: { name: { value: "section" } }],
+              combinator: { value: { value: ">" } },
+              right: Selectors::TypeSelector[value: { name: { value: "table" } }]
+            ]
+          ]
+        end
+      end
+
+      it "parses a complex selector with many selectors" do
+        actual = parse_selectors("section>table>tr")
+
+        assert_pattern do
+          actual => [
+            Selectors::ComplexSelector[
+              left: Selectors::TypeSelector[value: { name: { value: "section" } }],
+              combinator: { value: { value: ">" } },
+              right: Selectors::ComplexSelector[
+                left: Selectors::TypeSelector[value: { name: { value: "table" } }],
+                combinator: { value: { value: ">" } },
+                right: Selectors::TypeSelector[value: { name: { value: "tr" } }]
+              ]
+            ]
+          ]
+        end
+      end
+
       private
 
       def parse_selectors(selectors)