format: implement CSS selector formatting for more nodes

Implement formatting for IdSelector, HashToken, PseudoClassSelector,
and PseudoElementSelector.

Also, add formatting test coverage.

Author: flavorjones

lib/syntax_tree/css/format.rb

@@ -47,6 +47,11 @@ def visit_ident_token(node)
         q.text(node.value)
       end
 
+      # Visit a HashToken node.
+      def visit_hash_token(node)
+        q.text(node.value)
+      end
+
       # Visit a StyleRule node.
       def visit_style_rule(node)
         q.group do
@@ -77,12 +82,30 @@ def visit_type_selector(node)
         end
       end
 
+      # Visit a Selectors::IdSelector node.
+      def visit_id_selector(node)
+        q.text("#")
+        node.value.format(q)
+      end
+
       # Visit a Selectors::ClassSelector node.
       def visit_class_selector(node)
         q.text(".")
         node.value.format(q)
       end
 
+      # Visit a Selectors::PseudoClassSelector node.
+      def visit_pseudo_class_selector(node)
+        q.text(":")
+        node.value.format(q)
+      end
+
+      # Visit a Selectors::PseudoElementSelector node.
+      def visit_pseudo_element_selector(node)
+        q.text(":")
+        node.value.format(q)
+      end
+
       # Visit a Selectors::Combinator node.
       def visit_combinator(node)
         node.value.format(q)
@@ -101,9 +124,8 @@ def visit_complex_selector(node)
       # Visit a Selectors::CompoundSelector node.
       def visit_compound_selector(node)
         q.group do
-          node.type.format(q) if node.type
-          node.subclasses.each do |subclass|
-            subclass.format(q)
+          node.child_nodes.each do |node_|
+            node_.format(q)
           end
           # TODO: pseudo-elements
         end

test/selectors_test.rb

@@ -212,8 +212,43 @@ class SelectorsTest < Minitest::Spec
       end
 
       describe "formatting" do
-        it "formats complex selectors" do
-          assert_selector_format(".outer section.foo>table.bar   tr", ".outer section.foo > table.bar tr")
+        describe Selectors::CompoundSelector do
+          it "with an id selector" do
+            assert_selector_format(
+              "div#foo",
+              "div#foo",
+            )
+          end
+
+          it "with a pseudo-class selector" do
+            assert_selector_format(
+              "div:hover",
+              "div:hover",
+            )
+          end
+
+          it "with class selectors" do
+            assert_selector_format(
+              "div.flex.text-xl",
+              "div.flex.text-xl",
+            )
+          end
+
+          it "with pseudo-elements" do
+            assert_selector_format(
+              "div.flex:hover::first-line:last-child:active::first-letter",
+              "div.flex:hover::first-line:last-child:active::first-letter",
+            )
+          end
+        end
+
+        describe Selectors::ComplexSelector do
+          it "with whitespace" do
+            assert_selector_format(
+              ".outer section.foo>table.bar   tr",
+              ".outer section.foo > table.bar tr",
+            )
+          end
         end
 
         private
@@ -223,7 +258,7 @@ def assert_selector_format(selectors, expected)
 
           io = StringIO.new
           selectors.each do |selector|
-            selector.format(::PrettyPrint.new(io))
+            selector.format(::PP.new(io))
             assert_equal(expected, io.string)
           end
         end