Adds proper handling of empty properties in CssFontFaceRule

glenn.volckaert · glenn.volckaert · commit 4d9f46dfe528 · 2022-02-15T15:33:50.000+01:00
DEVSIX-6339
diff --git a/src/test/java/com/itextpdf/html2pdf/css/FontFaceTest.java b/src/test/java/com/itextpdf/html2pdf/css/FontFaceTest.java
@@ -77,6 +77,15 @@ public static void beforeClass() {
         createOrClearDestinationFolder(destinationFolder);
     }
 
+
+    @LogMessages(messages = {
+            @LogMessage(messageTemplate = Html2PdfLogMessageConstant.UNABLE_TO_RETRIEVE_FONT) })
+    @Test
+    public void emptyFontDefinitionTest() throws IOException, InterruptedException {
+        runTest("emptyWebFontCssTest");
+    }
+
+
     @Test
     public void droidSerifWebFontTest() throws IOException, InterruptedException {
         runTest("droidSerifWebFontTest");
diff --git a/src/test/resources/com/itextpdf/html2pdf/css/FontFaceTest/cmp_emptyWebFontCssTest.pdf b/src/test/resources/com/itextpdf/html2pdf/css/FontFaceTest/cmp_emptyWebFontCssTest.pdf
diff --git a/src/test/resources/com/itextpdf/html2pdf/css/FontFaceTest/empty-font.css b/src/test/resources/com/itextpdf/html2pdf/css/FontFaceTest/empty-font.css
@@ -0,0 +1,3 @@
+@font-face {
+
+}
diff --git a/src/test/resources/com/itextpdf/html2pdf/css/FontFaceTest/emptyWebFontCssTest.html b/src/test/resources/com/itextpdf/html2pdf/css/FontFaceTest/emptyWebFontCssTest.html
@@ -0,0 +1,10 @@
+<!DOCTYPE html>
+<html>
+   <head>
+       <link href='empty-font.css' rel='stylesheet' type='text/css'>
+   </head>
+   <body>
+		<div>You can't reference an empty font definition</div>
+
+   </body>
+</html>
