SVG: Fix handling of the invalid fill-opacity and stroke-opacity values

DEVSIX-2678, DEVSIX-2679

Author: AnhelinaM

svg/src/main/java/com/itextpdf/svg/renderers/impl/AbstractSvgNodeRenderer.java

@@ -568,8 +568,10 @@ private float getOpacityByAttributeName(String attributeName, float generalOpaci
             float opacityValue;
             if (CssTypesValidationUtils.isPercentageValue(opacityStr)) {
                 opacityValue = CssDimensionParsingUtils.parseRelativeValue(opacityStr, 1f);
+            } else if (CssTypesValidationUtils.isNumber(opacityStr)) {
+                opacityValue = (float) CssDimensionParsingUtils.parseFloat(opacityStr);
             } else {
-                opacityValue = Float.valueOf(opacityStr);
+                opacityValue = 1;
             }
             opacity *= opacityValue;
         }

svg/src/test/java/com/itextpdf/svg/renderers/OpacityTest.java

@@ -63,11 +63,8 @@ public void testRGBA() throws IOException, InterruptedException {
   }
 
   @Test
-  //TODO DEVSIX-2678 SVG: Displaying invalid value of fill-opacity incorrectly
-  public void testFillOpacityWithComma() {
-    Assertions.assertThrows(NumberFormatException.class,
-            () -> convertAndCompare(SOURCE_FOLDER, DESTINATION_FOLDER, "testFillOpacityWithComma")
-    );
+  public void testFillOpacityWithComma() throws IOException, InterruptedException {
+    convertAndCompare(SOURCE_FOLDER, DESTINATION_FOLDER, "testFillOpacityWithComma");
   }
 
   @Test
@@ -76,17 +73,13 @@ public void testFillOpacityWithPercents() throws IOException, InterruptedExcepti
   }
 
   @Test
-  //TODO: update after DEVSIX-2678 fix
   public void testFillOpacity() throws IOException, InterruptedException {
     convertAndCompare(SOURCE_FOLDER, DESTINATION_FOLDER, "svg_fill_opacity");
   }
 
   @Test
-  //TODO DEVSIX-2679
   public void testStrokeOpacityWithComma() throws IOException, InterruptedException {
-    Assertions.assertThrows(Exception.class,
-            () -> convertAndCompare(SOURCE_FOLDER, DESTINATION_FOLDER, "testStrokeOpacityWithComma")
-    );
+    convertAndCompare(SOURCE_FOLDER, DESTINATION_FOLDER, "testStrokeOpacityWithComma");
   }
 
   @Test
@@ -95,7 +88,6 @@ public void testStrokeOpacityWithPercents() throws IOException, InterruptedExcep
   }
 
   @Test
-  //TODO: update after DEVSIX-2679 fix
   public void testStrokeOpacity() throws IOException, InterruptedException {
     convertAndCompare(SOURCE_FOLDER, DESTINATION_FOLDER, "svg_stroke_opacity");
   }