Add ticket number and tests for the corresponding TODOs

DEVSIX-6453

Author: Evgeniy Prudnikov

layout/src/main/java/com/itextpdf/layout/renderer/BlockRenderer.java

@@ -284,9 +284,7 @@ public LayoutResult layout(LayoutContext layoutContext) {
                 fixOccupiedAreaIfOverflowedX(overflowX, layoutBox);
 
                 if (result.getSplitRenderer() != null) {
-                    // Use occupied area's bbox width so that for absolutely positioned renderers we do not align using full width
-                    // in case when parent box should wrap around child boxes.
-                    // TODO in the latter case, all elements should be layouted first so that we know maximum width needed to place all children and then apply horizontal alignment
+                    // TODO DEVSIX-6488 all elements should be layouted first in case when parent box should wrap around child boxes
                     alignChildHorizontally(result.getSplitRenderer(), occupiedArea.getBBox());
                 }
 
@@ -340,9 +338,7 @@ public LayoutResult layout(LayoutContext layoutContext) {
             if (result.getStatus() == LayoutResult.FULL) {
                 decreaseLayoutBoxAfterChildPlacement(layoutBox, result, childRenderer);
                 if (childRenderer.getOccupiedArea() != null) {
-                    // Use occupied area's bbox width so that for absolutely positioned renderers we do not align using full width
-                    // in case when parent box should wrap around child boxes.
-                    // TODO in the latter case, all elements should be layouted first so that we know maximum width needed to place all children and then apply horizontal alignment
+                    // TODO DEVSIX-6488 all elements should be layouted first in case when parent box should wrap around child boxes
                     alignChildHorizontally(childRenderer, occupiedArea.getBBox());
                 }
             }

layout/src/test/java/com/itextpdf/layout/renderer/BlockRendererTest.java

@@ -22,23 +22,45 @@ This file is part of the iText (R) project.
  */
 package com.itextpdf.layout.renderer;
 
+import com.itextpdf.io.logs.IoLogMessageConstant;
+import com.itextpdf.kernel.colors.ColorConstants;
 import com.itextpdf.kernel.geom.Rectangle;
+import com.itextpdf.kernel.pdf.PdfDocument;
+import com.itextpdf.kernel.pdf.PdfWriter;
+import com.itextpdf.kernel.utils.CompareTool;
+import com.itextpdf.layout.Document;
 import com.itextpdf.layout.element.Div;
+import com.itextpdf.layout.element.Paragraph;
 import com.itextpdf.layout.layout.LayoutArea;
+import com.itextpdf.layout.layout.LayoutPosition;
 import com.itextpdf.layout.properties.OverflowPropertyValue;
+import com.itextpdf.layout.properties.Property;
 import com.itextpdf.layout.properties.UnitValue;
 import com.itextpdf.test.ExtendedITextTest;
+import com.itextpdf.test.LogLevelConstants;
+import com.itextpdf.test.annotations.LogMessage;
+import com.itextpdf.test.annotations.LogMessages;
 import com.itextpdf.test.annotations.type.IntegrationTest;
 
+import java.io.IOException;
 import org.junit.Assert;
+import org.junit.BeforeClass;
 import org.junit.Test;
 import org.junit.experimental.categories.Category;
 
 @Category(IntegrationTest.class)
 public class BlockRendererTest extends ExtendedITextTest {
 
+    public static final String SOURCE_FOLDER = "./src/test/resources/com/itextpdf/layout/BlockRendererTest/";
+    public static final String DESTINATION_FOLDER = "./target/test/com/itextpdf/layout/BlockRendererTest/";
+
+    @BeforeClass
+    public static void beforeClass() {
+        createOrClearDestinationFolder(DESTINATION_FOLDER);
+    }
+
     @Test
-    public void applyMinHeightForSpecificDimensionsCausingFloatPrecisionError () {
+    public void applyMinHeightForSpecificDimensionsCausingFloatPrecisionErrorTest() {
         float divHeight = 42.55f;
 
         Div div = new Div();
@@ -55,4 +77,40 @@ public void applyMinHeightForSpecificDimensionsCausingFloatPrecisionError () {
                 new Rectangle(0, 243.40012f, 0, leftHeight));
         Assert.assertNull(renderer);
     }
+
+    @Test
+    @LogMessages(messages = {
+            @LogMessage(messageTemplate = IoLogMessageConstant.OCCUPIED_AREA_HAS_NOT_BEEN_INITIALIZED, count = 2,
+                    logLevel = LogLevelConstants.ERROR)
+    })
+    // TODO DEVSIX-6488 all elements should be layouted first in case when parent box should wrap around child boxes
+    public void parentBoxWrapAroundChildBoxesTest() throws IOException, InterruptedException {
+        String cmpFileName = SOURCE_FOLDER + "cmp_parentBoxWrapAroundChildBoxes.pdf";
+        String outFile = DESTINATION_FOLDER + "parentBoxWrapAroundChildBoxes.pdf";
+        int enoughDivsToOccupyWholePage = 30;
+        Document document = new Document(new PdfDocument(new PdfWriter(outFile)));
+
+        Div div = new Div();
+        div.setBackgroundColor(ColorConstants.CYAN);
+        div.setProperty(Property.POSITION, LayoutPosition.ABSOLUTE);
+
+        Div childDiv = new Div();
+        childDiv.add(new Paragraph("ChildDiv"));
+        childDiv.setBackgroundColor(ColorConstants.YELLOW);
+        childDiv.setWidth(100);
+
+        for (int i = 0; enoughDivsToOccupyWholePage > i; i++) {
+            div.add(childDiv);
+        }
+        Div divThatDoesntFitButItsWidthShouldBeConsidered = new Div();
+        divThatDoesntFitButItsWidthShouldBeConsidered.add(new Paragraph("ChildDiv1"));
+        divThatDoesntFitButItsWidthShouldBeConsidered.setBackgroundColor(ColorConstants.GREEN);
+        divThatDoesntFitButItsWidthShouldBeConsidered.setWidth(200);
+
+        div.add(divThatDoesntFitButItsWidthShouldBeConsidered);
+        document.add(div);
+
+        document.close();
+        Assert.assertNull(new CompareTool().compareByContent(outFile, cmpFileName, DESTINATION_FOLDER));
+    }
 }