Add test for check resources released after conversion

Veronika Lisovskaya · yulian-gaponenko · commit bc77af24173c · 2018-10-02T14:21:27.000+03:00
DEVSIX-2314

Autoported commit.
Original commit hash: [635d7abc38]
diff --git a/itext.tests/itext.html2pdf.tests/itext/html2pdf/resolver/font/LocalFontResolverReleaseTest.cs b/itext.tests/itext.html2pdf.tests/itext/html2pdf/resolver/font/LocalFontResolverReleaseTest.cs
@@ -37,9 +37,9 @@ public virtual void TestThatLocalFontIsReleasedAfterConversion() {
                 HtmlConverter.ConvertToPdf(new FileInfo(workDirHtmlFile), new FileInfo(targetPdfFile));
             }
             // The resource must be freed after the conversion
-            FileInfo resourceToBeremoved = new FileInfo(workDirFontFile);
-            resourceToBeremoved.Delete();
-            NUnit.Framework.Assert.IsFalse(resourceToBeremoved.Exists);
+            FileInfo resourceToBeRemoved = new FileInfo(workDirFontFile);
+            resourceToBeRemoved.Delete();
+            NUnit.Framework.Assert.IsFalse(resourceToBeRemoved.Exists);
         }
     }
 }
diff --git a/itext.tests/itext.html2pdf.tests/itext/html2pdf/resolver/resource/LocalImageResolverReleaseTest.cs b/itext.tests/itext.html2pdf.tests/itext/html2pdf/resolver/resource/LocalImageResolverReleaseTest.cs
@@ -0,0 +1,51 @@
+using System;
+using System.IO;
+using iText.Html2pdf;
+using iText.Test;
+using iText.Test.Attributes;
+
+namespace iText.Html2pdf.Resolver.Resource {
+    public class LocalImageResolverReleaseTest : ExtendedITextTest {
+        public static readonly String sourceFolder = iText.Test.TestUtil.GetParentProjectDirectory(NUnit.Framework.TestContext
+            .CurrentContext.TestDirectory) + "/resources/itext/html2pdf/resolver/resource/LocalImageResolverReleaseTest/";
+
+        public static readonly String destinationFolder = NUnit.Framework.TestContext.CurrentContext.TestDirectory
+             + "/test/itext/html2pdf/resolver/resource/LocalImageResolverReleaseTest/";
+
+        [NUnit.Framework.OneTimeSetUp]
+        public static void BeforeClass() {
+            CreateOrClearDestinationFolder(destinationFolder);
+        }
+
+        /// <exception cref="System.IO.IOException"/>
+        [NUnit.Framework.Test]
+        [LogMessage(iText.StyledXmlParser.LogMessageConstant.ERROR_RESOLVING_PARENT_STYLES, Count = 60)]
+        public virtual void TestThatSvgIsReleasedAfterConversion() {
+            String htmlFileName = "testWithSvg.html";
+            String svgFileName = "imageWithMultipleShapes.svg";
+            String imageFileName = "image.png";
+            String sourceHtmlFile = sourceFolder + htmlFileName;
+            String sourceSvgFile = sourceFolder + svgFileName;
+            String sourceImageFile = sourceFolder + imageFileName;
+            String workDir = destinationFolder + "work/";
+            CreateDestinationFolder(workDir);
+            String targetPdfFile = workDir + "target.pdf";
+            String workDirHtmlFile = workDir + htmlFileName;
+            String workDirSvgFile = workDir + svgFileName;
+            String workDirImageFile = workDir + imageFileName;
+            File.Copy(Path.Combine(sourceHtmlFile), Path.Combine(workDirHtmlFile));
+            File.Copy(Path.Combine(sourceSvgFile), Path.Combine(workDirSvgFile));
+            File.Copy(Path.Combine(sourceImageFile), Path.Combine(workDirImageFile));
+            for (int i = 0; i < 10; i++) {
+                HtmlConverter.ConvertToPdf(new FileInfo(workDirHtmlFile), new FileInfo(targetPdfFile));
+            }
+            // The resource must be freed after the conversion
+            FileInfo resourceToBeRemoved = new FileInfo(workDirSvgFile);
+            resourceToBeRemoved.Delete();
+            NUnit.Framework.Assert.IsFalse(resourceToBeRemoved.Exists);
+            resourceToBeRemoved = new FileInfo(workDirImageFile);
+            resourceToBeRemoved.Delete();
+            NUnit.Framework.Assert.IsFalse(resourceToBeRemoved.Exists);
+        }
+    }
+}
diff --git a/itext.tests/itext.html2pdf.tests/resources/itext/html2pdf/resolver/resource/LocalImageResolverReleaseTest/image.png b/itext.tests/itext.html2pdf.tests/resources/itext/html2pdf/resolver/resource/LocalImageResolverReleaseTest/image.png
diff --git a/itext.tests/itext.html2pdf.tests/resources/itext/html2pdf/resolver/resource/LocalImageResolverReleaseTest/imageWithMultipleShapes.svg b/itext.tests/itext.html2pdf.tests/resources/itext/html2pdf/resolver/resource/LocalImageResolverReleaseTest/imageWithMultipleShapes.svg
@@ -0,0 +1,5 @@
+<svg width="400" height="400" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink">
+    <rect x='0' y='0' width='400' height='400' stroke='green' fill='cyan'/>
+    <rect x='10' y='10' width='250' height='312' stroke='orange' fill='red'/>
+    <circle cx='350' cy='378' r='150' stroke='red' fill='green'/>
+</svg>
diff --git a/itext.tests/itext.html2pdf.tests/resources/itext/html2pdf/resolver/resource/LocalImageResolverReleaseTest/testWithSvg.html b/itext.tests/itext.html2pdf.tests/resources/itext/html2pdf/resolver/resource/LocalImageResolverReleaseTest/testWithSvg.html
@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<html>
+<head>
+    <title>Example</title>
+    <meta http-equiv="Content-Type" content="text/html;charset=utf-8"/>
+</head>
+
+<body>
+<h1>images within html</h1>
+<p> Example SVG Image</p>
+<img src="imageWithMultipleShapes.svg"/>
+<p>Example Object1</p>
+<object data="imageWithMultipleShapes.svg" type="image/svg+xml"></object>
+<p> Example Image</p>
+<img src="image.png"/>
+</body>
+</html>
diff --git a/port-hash b/port-hash
@@ -1 +1 @@
-fe2dd856e0f77647614a62a231148acdf0d9499e
+635d7abc3812b211d8d1f46af80c61e6f7a6f9ba

Original file line number	Diff line number	Diff line change
`@@ -37,9 +37,9 @@ public virtual void TestThatLocalFontIsReleasedAfterConversion() {`
`37`	`37`	`HtmlConverter.ConvertToPdf(new FileInfo(workDirHtmlFile), new FileInfo(targetPdfFile));`
`38`	`38`	`}`
`39`	`39`	`// The resource must be freed after the conversion`
`40`		`- FileInfo resourceToBeremoved = new FileInfo(workDirFontFile);`
`41`		`- resourceToBeremoved.Delete();`
`42`		`- NUnit.Framework.Assert.IsFalse(resourceToBeremoved.Exists);`
	`40`	`+ FileInfo resourceToBeRemoved = new FileInfo(workDirFontFile);`
	`41`	`+ resourceToBeRemoved.Delete();`
	`42`	`+ NUnit.Framework.Assert.IsFalse(resourceToBeRemoved.Exists);`
`43`	`43`	`}`
`44`	`44`	`}`
`45`	`45`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-fe2dd856e0f77647614a62a231148acdf0d9499e`
	`1`	`+635d7abc3812b211d8d1f46af80c61e6f7a6f9ba`