Fix tests after fixing LayoutExceptionMessageConstant.INVALID_CELL_INDEXES exception

DEVSIX-8427

Author: vitali-pr

src/test/java/com/itextpdf/html2pdf/css/grid/GridItemPlacementTest.java

@@ -88,16 +88,20 @@ public void fewCellsPlacement4Test() throws IOException, InterruptedException {
     }
 
     @Test
-    public void fewCellsPlacement5Test() {
-        Exception e = Assertions.assertThrows(IllegalArgumentException.class, () -> runTest("fewCellsPlacement5"));
-        Assertions.assertEquals(LayoutExceptionMessageConstant.INVALID_CELL_INDEXES, e.getMessage());
+    public void fewCellsPlacement5Test() throws IOException, InterruptedException {
+        runTest("fewCellsPlacement5");
     }
 
     @Test
     public void fewCellsPlacement6Test() throws IOException, InterruptedException {
         runTest("fewCellsPlacement6");
     }
 
+    @Test
+    public void fewCellsPlacement7Test() throws IOException, InterruptedException {
+        runTest("fewCellsPlacement7");
+    }
+
     @Test
     public void rowStartEnd1Test() throws IOException, InterruptedException {
         runTest("rowStartEnd1");

src/test/java/com/itextpdf/html2pdf/css/grid/GridTemplatesTest.java

@@ -634,6 +634,7 @@ public void maxHeightFlexRowsTest2() throws IOException, InterruptedException {
         runTest("maxHeightFlexRowsTest2");
     }
 
+    @Test
     @LogMessages(messages = {
             @LogMessage(messageTemplate = Html2PdfLogMessageConstant.INVALID_CSS_PROPERTY_DECLARATION, logLevel =
                     LogLevelConstants.WARN)})

src/test/java/com/itextpdf/html2pdf/css/multicol/ColumnCountTest.java

@@ -132,7 +132,6 @@ public void convertBasicDisplayPropertyTest() throws IOException, InterruptedExc
         runTest("basicDisplayPropertyTest");
     }
 
-    @Disabled("DEVSIX-8427")
     @Test
     public void convertBasicDisplayPropertyWithNestedColumnsTest() throws IOException, InterruptedException {
         runTest("basicDisplayPropertyWithNestedColumnsTest");

src/test/java/com/itextpdf/html2pdf/css/multicol/ColumnWidthTest.java

@@ -108,7 +108,6 @@ public void convertNestedElementsTest() throws IOException, InterruptedException
         runTest("nestedElementsTest");
     }
 
-    @Disabled("DEVSIX-8427")
     @Test
     public void convertBasicDisplayPropertyTest() throws IOException, InterruptedException {
         runTest("basicDisplayPropertyTest");

src/test/java/com/itextpdf/html2pdf/css/w3c/css_grid/grid_model/GridFloat002Test.java

@@ -27,11 +27,7 @@ This file is part of the iText (R) project.
 import com.itextpdf.test.annotations.LogMessage;
 import com.itextpdf.test.annotations.LogMessages;
 
-import org.junit.jupiter.api.Disabled;
-
 //TODO DEVSIX-2449 z-index is not supported
-//TODO DEVSIX-8427: Fix LayoutExceptionMessageConstant.INVALID_CELL_INDEXES exception
-@Disabled("DEVSIX-8427")
 @LogMessages(messages = {
         @LogMessage(messageTemplate = Html2PdfLogMessageConstant.NO_WORKER_FOUND_FOR_TAG, count = 3)})
 public class GridFloat002Test extends W3CCssTest {

src/test/java/com/itextpdf/html2pdf/css/w3c/css_multicol/BalanceGridContainerRefTest.java

@@ -22,62 +22,11 @@ This file is part of the iText (R) project.
  */
 package com.itextpdf.html2pdf.css.w3c.css_multicol;
 
-import com.itextpdf.html2pdf.HtmlConverter;
 import com.itextpdf.html2pdf.css.w3c.W3CCssTest;
-import com.itextpdf.io.util.UrlUtil;
-import com.itextpdf.kernel.utils.CompareTool;
-import org.junit.jupiter.api.Assertions;
-import org.junit.jupiter.api.Disabled;
-import org.junit.jupiter.api.Test;
-
-import java.io.File;
-import java.io.IOException;
 
 public class BalanceGridContainerRefTest extends W3CCssTest {
-    private static final String baseDestinationFolder = "./target/test/com/itextpdf/html2pdf/css/w3c/";
-    private static final String baseSourceFolder = "./src/test/resources/com/itextpdf/html2pdf/css/w3c/";
-    //after  completing DEVSIX-8427, remove everything from this class except for getHtmlFileName() method
     @Override
     protected String getHtmlFileName() {
         return "balance-grid-container-ref.html";
     }
-
-    @Override
-    @Disabled("DEVSIX-8427")
-    @Test
-    public void test() throws IOException, InterruptedException {
-        String sourceFolder = getSourceFolder();
-        String destinationFolder = getDestinationFolder();
-        String htmlFilePath = sourceFolder + getHtmlFileName();
-        String outFilePath = destinationFolder + getOutPdfFileName();
-        String cmpFilePath = sourceFolder + getOutPdfFileName();
-        System.out.println("html: " + UrlUtil.getNormalizedFileUriString(htmlFilePath) + "\n");
-        HtmlConverter.convertToPdf(new File(htmlFilePath), new File(outFilePath), getConverterProperties());
-        Assertions.assertNull(new CompareTool().compareByContent(outFilePath, cmpFilePath, destinationFolder, "diff_"));
-    }
-
-    private String getOutPdfFileName() {
-        String htmlFileName = getHtmlFileName();
-        return htmlFileName.replaceAll("\\.[a-zA-Z]+?$", ".pdf");
-    }
-
-    private String getDestinationFolder() {
-        String localPackage = getLocalPackage();
-        return baseDestinationFolder + localPackage + File.separatorChar + getTestClassName() + File.separatorChar;
-    }
-
-    private String getSourceFolder() {
-        String localPackage = getLocalPackage();
-        return baseSourceFolder + localPackage + File.separatorChar;
-    }
-
-    private String getLocalPackage() {
-        String packageName = getClass().getPackage().getName();
-        String basePackageName = W3CCssTest.class.getPackage().getName();
-        return packageName.substring(basePackageName.length()).replace('.', File.separatorChar);
-    }
-
-    private String getTestClassName() {
-        return getClass().getSimpleName();
-    }
 }

src/test/java/com/itextpdf/html2pdf/css/w3c/css_multicol/BalanceGridContainerTest.java

@@ -22,63 +22,11 @@ This file is part of the iText (R) project.
  */
 package com.itextpdf.html2pdf.css.w3c.css_multicol;
 
-import com.itextpdf.html2pdf.HtmlConverter;
 import com.itextpdf.html2pdf.css.w3c.W3CCssTest;
-import com.itextpdf.io.util.UrlUtil;
-import com.itextpdf.kernel.utils.CompareTool;
-import org.junit.jupiter.api.Assertions;
-import org.junit.jupiter.api.Disabled;
-import org.junit.jupiter.api.Test;
-
-import java.io.File;
-import java.io.IOException;
 
 public class BalanceGridContainerTest extends W3CCssTest {
-    private static final String baseDestinationFolder = "./target/test/com/itextpdf/html2pdf/css/w3c/";
-    private static final String baseSourceFolder = "./src/test/resources/com/itextpdf/html2pdf/css/w3c/";
-
-    //after completing DEVSIX-8427, remove everything from this class except for getHtmlFileName() method
     @Override
     protected String getHtmlFileName() {
         return "balance-grid-container.html";
     }
-
-    @Override
-    @Disabled("DEVSIX-8427")
-    @Test
-    public void test() throws IOException, InterruptedException {
-        String sourceFolder = getSourceFolder();
-        String destinationFolder = getDestinationFolder();
-        String htmlFilePath = sourceFolder + getHtmlFileName();
-        String outFilePath = destinationFolder + getOutPdfFileName();
-        String cmpFilePath = sourceFolder + getOutPdfFileName();
-        System.out.println("html: " + UrlUtil.getNormalizedFileUriString(htmlFilePath) + "\n");
-        HtmlConverter.convertToPdf(new File(htmlFilePath), new File(outFilePath), getConverterProperties());
-        Assertions.assertNull(new CompareTool().compareByContent(outFilePath, cmpFilePath, destinationFolder, "diff_"));
-    }
-
-    private String getOutPdfFileName() {
-        String htmlFileName = getHtmlFileName();
-        return htmlFileName.replaceAll("\\.[a-zA-Z]+?$", ".pdf");
-    }
-
-    private String getDestinationFolder() {
-        String localPackage = getLocalPackage();
-        return baseDestinationFolder + localPackage + File.separatorChar + getTestClassName() + File.separatorChar;
-    }
-
-    private String getSourceFolder() {
-        String localPackage = getLocalPackage();
-        return baseSourceFolder + localPackage + File.separatorChar;
-    }
-
-    private String getLocalPackage() {
-        String packageName = getClass().getPackage().getName();
-        String basePackageName = W3CCssTest.class.getPackage().getName();
-        return packageName.substring(basePackageName.length()).replace('.', File.separatorChar);
-    }
-
-    private String getTestClassName() {
-        return getClass().getSimpleName();
-    }
 }

src/test/resources/com/itextpdf/html2pdf/css/grid/GridItemPlacementTest/cmp_fewCellsPlacement5.pdf

@@ -11,7 +11,6 @@
 
       .wrapper > div {
         border: 2px solid darkblue;
-        background-color: lightblue;
       }
 
       .wrapper > div:nth-of-type(2) {