SVG: don't draw rect with negative or zero height or width

DEVSIX-8760

Author: AnhelinaM

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

@@ -58,6 +58,9 @@ protected void doDraw(SvgDrawContext context) {
         PdfCanvas cv = context.getCurrentCanvas();
         cv.writeLiteral("% rect\n");
         setParameters(context);
+        if (width <= 0 || height <= 0) {
+            return;
+        }
         boolean singleValuePresent = (rxPresent && !ryPresent) || (!rxPresent && ryPresent);
 
         AffineTransform transform = applyNonScalingStrokeTransform(context);

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

@@ -131,7 +131,6 @@ public void invalidUrlFillTest() throws IOException, InterruptedException {
     @LogMessages(
             messages = {@LogMessage(messageTemplate = SvgLogMessageConstant.UNMAPPED_TAG, count = 4)}
     )
-    //TODO update cmp file after DEVSIX-2915 will be fixed
     public void textFillFallbackTest() throws IOException, InterruptedException {
         convertAndCompareSinglePage(SOURCE_FOLDER, DESTINATION_FOLDER, "textFillFallbackTest");
     }

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

@@ -104,9 +104,13 @@ public void strokeAttributesTest() throws IOException, InterruptedException {
     }
 
     @Test
-    //TODO DEVSIX-8760 : change cmp after fix
     public void heightWidthZeroTest() throws IOException, InterruptedException {
-        convertAndCompareSinglePage(SOURCE_FOLDER, DESTINATION_FOLDER,"height-width-zero");
+        convertAndCompareSinglePage(SOURCE_FOLDER, DESTINATION_FOLDER,"heightWidthZero");
+    }
+
+    @Test
+    public void heightWidthNegativeTest() throws IOException, InterruptedException {
+        convertAndCompareSinglePage(SOURCE_FOLDER, DESTINATION_FOLDER,"heightWidthNegative");
     }
 
     @Test

svg/src/test/java/com/itextpdf/svg/renderers/impl/ClipPathSvgNodeRendererLowLevelIntegrationTest.java

@@ -91,7 +91,7 @@ public void testRectClipPathRenderer() {
         clipRenderer.setClippedRenderer(new RectangleSvgNodeRenderer());
         clipRenderer.draw(sdc);
 
-        Assertions.assertEquals("q\n% rect\n0 0 300 300 re\nW\nn\n0 0 0 rg\n% rect\n0 0 0 0 re\nf\nQ\n", new String(cv.getContentStream().getBytes()));
+        Assertions.assertEquals("q\n% rect\n0 0 300 300 re\nW\nn\n0 0 0 rg\n% rect\nf\nQ\n", new String(cv.getContentStream().getBytes()));
     }
 
     @Test
@@ -106,7 +106,7 @@ public void testRectClipPathEoRendererNoChange() {
         clipRenderer.setClippedRenderer(new RectangleSvgNodeRenderer());
         clipRenderer.draw(sdc);
 
-        Assertions.assertEquals("q\n% rect\n0 0 300 300 re\nW\nn\n0 0 0 rg\n% rect\n0 0 0 0 re\nf\nQ\n", new String(cv.getContentStream().getBytes()));
+        Assertions.assertEquals("q\n% rect\n0 0 300 300 re\nW\nn\n0 0 0 rg\n% rect\nf\nQ\n", new String(cv.getContentStream().getBytes()));
     }
 
     @Test
@@ -121,7 +121,7 @@ public void testRectEoClipPathRenderer() {
         clipRenderer.setClippedRenderer(new RectangleSvgNodeRenderer());
         clipRenderer.draw(sdc);
 
-        Assertions.assertEquals("q\n% rect\n0 0 300 300 re\nW*\nn\n0 0 0 rg\n% rect\n0 0 0 0 re\nf\nQ\n", new String(cv.getContentStream().getBytes()));
+        Assertions.assertEquals("q\n% rect\n0 0 300 300 re\nW*\nn\n0 0 0 rg\n% rect\nf\nQ\n", new String(cv.getContentStream().getBytes()));
     }
 
     @Test