Fix table with empty rows creation

DEVSIX-6094

Author: vitali-pr

src/main/java/com/itextpdf/html2pdf/attach/wrapelement/TableWrapper.java

@@ -269,15 +269,15 @@ public Table toTable(WaitingColgroupsHelper colgroupsHelper) {
         }
         if (rows != null) {
             for (int i = 0; i < rows.size(); i++) {
+                table.startNewRow();
+
                 if (isRtl) {
                     Collections.reverse(rows.get(i));
                 }
+
                 for (int j = 0; j < rows.get(i).size(); j++) {
                     table.addCell(rows.get(i).get(j).cell);
                 }
-                if (i != rows.size() - 1) {
-                    table.startNewRow();
-                }
             }
         }
         if (caption != null) {

src/test/java/com/itextpdf/html2pdf/css/PageBreakTest.java

@@ -226,7 +226,10 @@ public void pageBreakInsideAvoidInTdWithParaTest() throws IOException, Interrupt
     }
 
     @Test
-    @LogMessages(messages = {@LogMessage(messageTemplate = LayoutLogMessageConstant.ELEMENT_DOES_NOT_FIT_AREA)})
+    @LogMessages(messages = {
+            @LogMessage(messageTemplate =  IoLogMessageConstant.LAST_ROW_IS_NOT_COMPLETE),
+            @LogMessage(messageTemplate = LayoutLogMessageConstant.ELEMENT_DOES_NOT_FIT_AREA)
+    })
     // TODO: DEVSIX-5263 inconsistent behavior when page-break-inside: avoid set in td and td contains inline elements
     public void pageBreakInsideAvoidInTdWithTableTest() throws IOException, InterruptedException {
         convertToElements("pageBreakInsideAvoidInTdWithTable");

src/test/java/com/itextpdf/html2pdf/element/TableTest.java

@@ -35,11 +35,13 @@ This file is part of the iText (R) project.
 import com.itextpdf.layout.Document;
 import com.itextpdf.layout.element.IBlockElement;
 import com.itextpdf.layout.element.IElement;
+import com.itextpdf.layout.element.Table;
 import com.itextpdf.layout.logs.LayoutLogMessageConstant;
 import com.itextpdf.test.ExtendedITextTest;
 import com.itextpdf.test.annotations.LogMessage;
 import com.itextpdf.test.annotations.LogMessages;
 import com.itextpdf.test.annotations.type.IntegrationTest;
+
 import org.junit.Assert;
 import org.junit.BeforeClass;
 import org.junit.Ignore;
@@ -603,6 +605,16 @@ public void thTagConvertToElementTest() throws IOException, InterruptedException
         runConvertToElements("thTagConvertToElement", false);
     }
 
+    @Test
+    public void emptyRowsConvertToElementTest() throws IOException {
+        FileInputStream source = new FileInputStream(sourceFolder + "emptyRowsConvertToElement.html");
+
+        for (IElement element : HtmlConverter.convertToElements(source)) {
+            Assert.assertTrue(element instanceof Table);
+            Assert.assertEquals(4, ((Table) element).getNumberOfRows());
+        }
+    }
+
     @Test
     public void thTagConvertToPdfTest() throws IOException, InterruptedException {
         runTest("thTagConvertToPdf");

src/test/resources/com/itextpdf/html2pdf/element/TableTest/emptyRowsConvertToElement.html

@@ -0,0 +1,9 @@
+<table>
+        <tr></tr>
+        <tr></tr>
+        <tr></tr>
+        <tr>
+            <td>test #1</td>
+            <td>test #2</td>
+        </tr>
+</table>