add missing test cases

Author: rbri

src/main/javacc/CSS3Parser.jj

@@ -1969,7 +1969,7 @@ LexicalUnit rgbColor(LexicalUnit prev) :
             | next = percentage(next, op)
         )
     )
-    { op = ' ';  }
+    { op = ' '; }
 
     ( <S> )*
     (
@@ -1985,7 +1985,7 @@ LexicalUnit rgbColor(LexicalUnit prev) :
             | next = percentage(next, op)
         )
     )
-    { op = ' ';  }
+    { op = ' '; }
 
     ( <S> )*
     (
@@ -2054,7 +2054,7 @@ LexicalUnit hslColor(LexicalUnit prev) :
             | next = percentage(next, op)
         )
     )
-    { op = ' ';  }
+    { op = ' '; }
 
     ( <S> )*
     (
@@ -2069,7 +2069,7 @@ LexicalUnit hslColor(LexicalUnit prev) :
             | next = percentage(next, op)
         )
     )
-    { op = ' ';  }
+    { op = ' '; }
 
     ( <S> )*
     (
@@ -2134,7 +2134,7 @@ LexicalUnit hwbColor(LexicalUnit prev) :
             | next = percentage(next, op)
         )
     )
-    { op = ' ';  }
+    { op = ' '; }
 
     ( <S> )*
 
@@ -2145,7 +2145,7 @@ LexicalUnit hwbColor(LexicalUnit prev) :
             | next = percentage(next, op)
         )
     )
-    { op = ' ';  }
+    { op = ' '; }
 
     ( <S> )*
     (
@@ -2205,7 +2205,7 @@ LexicalUnit labColor(LexicalUnit prev) :
             | next = percentage(next, op)
         )
     )
-    { op = ' ';  }
+    { op = ' '; }
 
     ( <S> )*
 
@@ -2217,7 +2217,7 @@ LexicalUnit labColor(LexicalUnit prev) :
             | next = percentage(next, op)
         )
     )
-    { op = ' ';  }
+    { op = ' '; }
 
     ( <S> )*
     (
@@ -2277,7 +2277,7 @@ LexicalUnit lchColor(LexicalUnit prev) :
             | next = percentage(next, op)
         )
     )
-    { op = ' ';  }
+    { op = ' '; }
 
     ( <S> )*
 
@@ -2291,7 +2291,7 @@ LexicalUnit lchColor(LexicalUnit prev) :
             | t = <ANGLE_TURN>    { next = LexicalUnitImpl.createTurn(next, doubleValue(op, t.image)); }
         )
     )
-    { op = ' ';  }
+    { op = ' '; }
 
     ( <S> )*
     (

src/test/java/org/htmlunit/cssparser/parser/CSS3ParserTest.java

@@ -57,6 +57,7 @@
 import org.htmlunit.cssparser.parser.selector.Selector;
 import org.htmlunit.cssparser.parser.selector.Selector.SelectorType;
 import org.htmlunit.cssparser.parser.selector.SelectorList;
+import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Test;
 
 /**
@@ -1271,6 +1272,15 @@ public void hslVariousErrors() throws Exception {
         color(1, "DOM exception: ''hsl' has to use blank as separator if none is used.'", "foreground: hsl(10, 20%, none)");
         color(1, "DOM exception: ''hsl' has to use blank as separator if none is used.'", "foreground: hsl(10, 20%, 30%, none)");
 
+        color(1, "Error in expression. (Invalid token \"-none\". Was expecting one of: <S>, <NUMBER>, \"none\", \"-\", \"+\", <ANGLE_DEG>, <ANGLE_RAD>, <ANGLE_GRAD>, <ANGLE_TURN>.)",
+                "foreground: hsl(-none 20% 30%)");
+        color(1, "Error in expression. (Invalid token \"-none\". Was expecting one of: <S>, \"none\", \"-\", \"+\", \",\", <PERCENTAGE>.)",
+                "foreground: hsl(10 -none 30%)");
+        color(1, "Error in expression. (Invalid token \"-none\". Was expecting one of: <S>, \"none\", \"-\", \"+\", \",\", <PERCENTAGE>.)",
+                "foreground: hsl(10 20% -none)");
+        color(1, "Error in expression. (Invalid token \"-none\". Was expecting one of: <S>, <NUMBER>, \"none\", \"-\", \"+\", <PERCENTAGE>.)",
+                "foreground: hsl(10 20% 30% / -none)");
+
         // mixing numbers and percentages is supported by current browsers
         //        color(1, "DOM exception: 'hsl mixing numbers and percentages.'", "foreground: hsl(10rad, 20, 30)");
         //        color(1, "DOM exception: 'hsl mixing numbers and percentages.'", "foreground: hsl(10rad, 20%, 30)");
@@ -1349,6 +1359,15 @@ public void hwbVariousErrors() throws Exception {
 
         color(1, "DOM exception: ''hwb' alpha value must be separated by '/'.'", "foreground: hwb(10 20% 30% 40)");
 
+        color(1, "Error in expression. (Invalid token \"-none\". Was expecting one of: <S>, <NUMBER>, \"none\", \"-\", \"+\", <ANGLE_DEG>, <ANGLE_RAD>, <ANGLE_GRAD>, <ANGLE_TURN>.)",
+                "foreground: hwb(-none 20% 30%)");
+        color(1, "Error in expression. (Invalid token \"-none\". Was expecting one of: <S>, \"none\", \"-\", \"+\", <PERCENTAGE>.)",
+                "foreground: hwb(10 -none 30%)");
+        color(1, "Error in expression. (Invalid token \"-none\". Was expecting one of: <S>, \"none\", \"-\", \"+\", <PERCENTAGE>.)",
+                "foreground: hwb(10 20% -none)");
+        color(1, "Error in expression. (Invalid token \"-none\". Was expecting one of: <S>, <NUMBER>, \"none\", \"-\", \"+\", <PERCENTAGE>.)",
+                "foreground: hwb(10 20% 30% / -none)");
+
         color(1, "Error in expression. (Invalid token \")\". Was expecting one of: <S>, <NUMBER>, \"none\", \"-\", \"+\", <ANGLE_DEG>, <ANGLE_RAD>, <ANGLE_GRAD>, <ANGLE_TURN>.)",
                 "foreground: hwb()");
         color(1, "Error in expression. (Invalid token \")\". Was expecting one of: <S>, \"none\", \"-\", \"+\", <PERCENTAGE>.)",
@@ -1693,6 +1712,9 @@ private void color(final int errorCount, final String expected, final String css
 
         final CSSStyleDeclarationImpl style = parser.parseStyleDeclaration(cssText);
 
+        if (errorCount == 0 && errorHandler.getErrorCount() > 0) {
+            Assertions.fail("Found errors; First: " + errorHandler.getErrorMessage());
+        }
         assertEquals(errorCount, errorHandler.getErrorCount());
         if (errorCount > 0) {
             assertEquals(expected, errorHandler.getErrorMessage());