Implements border parsing

This doesn't interpret "border" properties as "border-top",
"border-bottom", etc. This parses them directly to the bottom level of
"border-top-width", "border-top-style", etc.

That might come back to bite me, but going to give it a shot and see how
far it goes

Author: zmoazeni

lib/csscss.rb

@@ -25,4 +25,5 @@
 require "csscss/parser/border_color"
 require "csscss/parser/border_style"
 require "csscss/parser/border_side"
+require "csscss/parser/border"
 

lib/csscss/parser/border.rb

@@ -0,0 +1,45 @@
+module Csscss
+  module Parser
+    module Border
+      extend Parser::Base
+
+      class Parser < Parslet::Parser
+        include Color
+
+        rule(:border_side) { BorderSide::Parser.new(:top).border_side_anonymous }
+
+        rule(:border) {
+          (
+           symbol("inherit") >> eof |
+           border_side.maybe.as(:side)
+          ).as(:border)
+        }
+
+        root(:border)
+      end
+
+      class Transformer < Parslet::Transform
+        extend Color::Transformer
+        extend Color::PlainColorValue
+        extend BorderSide::Transformer::Helpers
+
+        rule(border: simple(:inherit)) {[]}
+        rule(border: {
+          side: {
+            width:simple(:width),
+            style:simple(:style),
+            color:simple(:color)
+          }
+        }) {|context|
+          [].tap do |declarations|
+            [:top, :right, :bottom, :left].each do |side|
+              declarations << transform_side(side, context)
+            end
+
+            declarations.flatten!
+          end
+        }
+      end
+    end
+  end
+end

lib/csscss/parser/border_color.rb

@@ -28,15 +28,12 @@ class Transformer < Parslet::Transform
         @property = :border_color
         extend MultiSideTransformer
         extend Color::Transformer
+        extend Color::PlainColorValue
 
         class << self
           def side_declaration(side, value)
             Declaration.from_parser("border-#{side}-color", value)
           end
-
-          def transform_color(context)
-            context[:value]
-          end
         end
       end
     end

lib/csscss/parser/border_side.rb

@@ -15,13 +15,16 @@ def initialize(side)
         rule(:border_style) { BorderStyle::Parser.new.border_style_side }
         rule(:border_color) { BorderColor::Parser.new.border_color_side }
 
+        rule(:border_side_anonymous) {
+          border_width.maybe.as(:width) >>
+          border_style.maybe.as(:style) >>
+          border_color.maybe.as(:color)
+        }
+
         rule(:border_side) {
           (
-           symbol("inherit") >> eof  | (
-             border_width.maybe.as(:width) >>
-             border_style.maybe.as(:style) >>
-             border_color.maybe.as(:color)
-            ).as(side)
+           symbol("inherit") >> eof |
+           border_side_anonymous.as(side)
           ).as(:border_side)
         }
 
@@ -30,8 +33,9 @@ def initialize(side)
 
       class Transformer < Parslet::Transform
         extend Color::Transformer
+        extend Color::PlainColorValue
 
-        class << self
+        module Helpers
           def transform_top(context);    transform_side("top", context);    end
           def transform_right(context);  transform_side("right", context);  end
           def transform_bottom(context); transform_side("bottom", context); end
@@ -45,10 +49,11 @@ def transform_side(side, context)
             [].tap do |declarations|
               declarations << Declaration.from_parser("border-#{side}-width", width) if width
               declarations << Declaration.from_parser("border-#{side}-style", style) if style
-              declarations << Declaration.from_parser("border-#{side}-color", color.value) if color
+              declarations << Declaration.from_parser("border-#{side}-color", color) if color
             end
           end
         end
+        extend Helpers
 
         rule(border_side: simple(:inherit)) {[]}
 
@@ -61,7 +66,6 @@ def transform_side(side, context)
             }
           }, &method("transform_#{side}"))
         end
-
       end
     end
   end

lib/csscss/parser/color.rb

@@ -42,6 +42,12 @@ def transform_color(context)
           end
         end
       end
+
+      module PlainColorValue
+        def transform_color(context)
+          context[:value]
+        end
+      end
     end
   end
 end

test/csscss/parser/border_test.rb

@@ -0,0 +1,31 @@
+require "test_helper"
+
+module Csscss::Parser
+  module Border
+    describe Border do
+      include CommonParserTests
+
+      before do
+        @parser = Parser.new
+        @trans = Transformer.new
+      end
+
+      it "converts shorthand rules to longhand" do
+        trans("1px solid #fff").must_equal([
+          dec("border-top-width", "1px"),
+          dec("border-top-style", "solid"),
+          dec("border-top-color", "#fff"),
+          dec("border-right-width", "1px"),
+          dec("border-right-style", "solid"),
+          dec("border-right-color", "#fff"),
+          dec("border-bottom-width", "1px"),
+          dec("border-bottom-style", "solid"),
+          dec("border-bottom-color", "#fff"),
+          dec("border-left-width", "1px"),
+          dec("border-left-style", "solid"),
+          dec("border-left-color", "#fff")
+        ])
+      end
+    end
+  end
+end