Remove closed TODO remarks. Add ticket references for unresolved TODOs

BlackEgoist · iText-CI · commit b9ee085fe50a · 2020-06-30T09:13:28.000Z
DEVSIX-3933
diff --git a/src/test/java/com/itextpdf/html2pdf/css/TextDecorationTest.java b/src/test/java/com/itextpdf/html2pdf/css/TextDecorationTest.java
@@ -92,19 +92,18 @@ public void textDecoration05Test() throws IOException, InterruptedException {
     }
 
     @Test
-    // TODO DEVSIX-3933
+    //TODO update after DEVSIX-4063 is closed
     public void textDecorationShorthandAllValuesTest() throws IOException, InterruptedException {
         convertToPdfAndCompare("textDecorationShorthandAllValues", sourceFolder, destinationFolder);
     }
 
     @Test
-    // TODO DEVSIX-3933
     public void textDecorationShorthandOneValueTest() throws IOException, InterruptedException {
         convertToPdfAndCompare("textDecorationShorthandOneValue", sourceFolder, destinationFolder);
     }
 
     @Test
-    // TODO DEVSIX-3933
+    //TODO update after DEVSIX-4063 is closed
     public void textDecorationShorthandTwoValuesTest() throws IOException, InterruptedException {
         convertToPdfAndCompare("textDecorationShorthandTwoValues", sourceFolder, destinationFolder);
     }
@@ -115,7 +114,7 @@ public void textDecorationWithChildElementTest() throws IOException, Interrupted
     }
 
     @Test
-    //TODO: DEVSIX-3933
+    //TODO: DEVSIX-4201
     @LogMessages(messages = {
             @LogMessage(messageTemplate = LogMessageConstant.HSL_COLOR_NOT_SUPPORTED)})
     public void textDecorationColorTest() throws IOException, InterruptedException {
@@ -128,7 +127,6 @@ public void textDecorationColorWithTransparencyTest() throws IOException, Interr
     }
 
     @Test
-    //TODO: DEVSIX-3933
     public void textDecorationLineTest() throws IOException, InterruptedException {
         convertToPdfAndCompare("textDecorationLine", sourceFolder, destinationFolder);
     }
@@ -139,19 +137,18 @@ public void textDecorationLineNoneAndUnderlineTogetherTest() throws IOException,
     }
 
     @Test
-    //TODO: DEVSIX-3933
+    //TODO update after DEVSIX-4063 is closed
     public void textDecorationStyleTest() throws IOException, InterruptedException {
         convertToPdfAndCompare("textDecorationStyle", sourceFolder, destinationFolder);
     }
 
     @Test
-    //TODO: DEVSIX-3933
+    //TODO update after DEVSIX-4063 is closed
     public void shorthandAndSpecificTextDecorPropsTest() throws IOException, InterruptedException {
         convertToPdfAndCompare("shorthandAndSpecificTextDecorProps", sourceFolder, destinationFolder);
     }
 
     @Test
-    //TODO: DEVSIX-3933
     public void combinationOfLinesInTextDecorationTest() throws IOException, InterruptedException {
         convertToPdfAndCompare("combinationOfLinesInTextDecoration", sourceFolder, destinationFolder);
     }
diff --git a/src/test/resources/com/itextpdf/html2pdf/css/TextDecorationTest/textDecorationShorthandOneValue.html b/src/test/resources/com/itextpdf/html2pdf/css/TextDecorationTest/textDecorationShorthandOneValue.html
@@ -16,7 +16,7 @@
 <body>
 <h1>Text-decoration: underline</h1>
 
-<h2>Text-decoration: dashed</h2>
+<h2>Text-decoration: solid</h2>
 
 <h3>Text-decoration: red</h3>
 

Original file line number	Diff line number	Diff line change
`@@ -92,19 +92,18 @@ public void textDecoration05Test() throws IOException, InterruptedException {`
`92`	`92`	`}`
`93`	`93`
`94`	`94`	`@Test`
`95`		`- // TODO DEVSIX-3933`
	`95`	`+ //TODO update after DEVSIX-4063 is closed`
`96`	`96`	`public void textDecorationShorthandAllValuesTest() throws IOException, InterruptedException {`
`97`	`97`	`convertToPdfAndCompare("textDecorationShorthandAllValues", sourceFolder, destinationFolder);`
`98`	`98`	`}`
`99`	`99`
`100`	`100`	`@Test`
`101`		`- // TODO DEVSIX-3933`
`102`	`101`	`public void textDecorationShorthandOneValueTest() throws IOException, InterruptedException {`
`103`	`102`	`convertToPdfAndCompare("textDecorationShorthandOneValue", sourceFolder, destinationFolder);`
`104`	`103`	`}`
`105`	`104`
`106`	`105`	`@Test`
`107`		`- // TODO DEVSIX-3933`
	`106`	`+ //TODO update after DEVSIX-4063 is closed`
`108`	`107`	`public void textDecorationShorthandTwoValuesTest() throws IOException, InterruptedException {`
`109`	`108`	`convertToPdfAndCompare("textDecorationShorthandTwoValues", sourceFolder, destinationFolder);`
`110`	`109`	`}`
`@@ -115,7 +114,7 @@ public void textDecorationWithChildElementTest() throws IOException, Interrupted`
`115`	`114`	`}`
`116`	`115`
`117`	`116`	`@Test`
`118`		`- //TODO: DEVSIX-3933`
	`117`	`+ //TODO: DEVSIX-4201`
`119`	`118`	`@LogMessages(messages = {`
`120`	`119`	`@LogMessage(messageTemplate = LogMessageConstant.HSL_COLOR_NOT_SUPPORTED)})`
`121`	`120`	`public void textDecorationColorTest() throws IOException, InterruptedException {`
`@@ -128,7 +127,6 @@ public void textDecorationColorWithTransparencyTest() throws IOException, Interr`
`128`	`127`	`}`
`129`	`128`
`130`	`129`	`@Test`
`131`		`- //TODO: DEVSIX-3933`
`132`	`130`	`public void textDecorationLineTest() throws IOException, InterruptedException {`
`133`	`131`	`convertToPdfAndCompare("textDecorationLine", sourceFolder, destinationFolder);`
`134`	`132`	`}`
`@@ -139,19 +137,18 @@ public void textDecorationLineNoneAndUnderlineTogetherTest() throws IOException,`
`139`	`137`	`}`
`140`	`138`
`141`	`139`	`@Test`
`142`		`- //TODO: DEVSIX-3933`
	`140`	`+ //TODO update after DEVSIX-4063 is closed`
`143`	`141`	`public void textDecorationStyleTest() throws IOException, InterruptedException {`
`144`	`142`	`convertToPdfAndCompare("textDecorationStyle", sourceFolder, destinationFolder);`
`145`	`143`	`}`
`146`	`144`
`147`	`145`	`@Test`
`148`		`- //TODO: DEVSIX-3933`
	`146`	`+ //TODO update after DEVSIX-4063 is closed`
`149`	`147`	`public void shorthandAndSpecificTextDecorPropsTest() throws IOException, InterruptedException {`
`150`	`148`	`convertToPdfAndCompare("shorthandAndSpecificTextDecorProps", sourceFolder, destinationFolder);`
`151`	`149`	`}`
`152`	`150`
`153`	`151`	`@Test`
`154`		`- //TODO: DEVSIX-3933`
`155`	`152`	`public void combinationOfLinesInTextDecorationTest() throws IOException, InterruptedException {`
`156`	`153`	`convertToPdfAndCompare("combinationOfLinesInTextDecoration", sourceFolder, destinationFolder);`
`157`	`154`	`}`