You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by ce...@apache.org on 2023/02/05 21:11:09 UTC

svn commit: r1907445 - /poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFTheme.java

Author: centic
Date: Sun Feb  5 21:11:09 2023
New Revision: 1907445

URL: http://svn.apache.org/viewvc?rev=1907445&view=rev
Log:
Avoid a file-handle leak in tests

Modified:
    poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFTheme.java

Modified: poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFTheme.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFTheme.java?rev=1907445&r1=1907444&r2=1907445&view=diff
==============================================================================
--- poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFTheme.java (original)
+++ poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFTheme.java Sun Feb  5 21:11:09 2023
@@ -31,13 +31,14 @@ public final class TestXWPFTheme {
 
     @Test
     void testRead() throws IOException {
-        XWPFDocument docx = XWPFTestDataSamples.openSampleDocument("sample.docx");
-        XWPFTheme theme = docx.getTheme();
-        assertEquals("Office Theme", theme.getName());
-        assertEquals("Cambria", theme.getMajorFont());
-        assertEquals("Calibri", theme.getMinorFont());
-        CTColor accent1 = theme.getCTColor("accent1");
-        XSLFColor color = new XSLFColor(accent1, null, null, null);
-        assertEquals(new Color(79, 129, 189), color.getColor());
+        try (XWPFDocument docx = XWPFTestDataSamples.openSampleDocument("sample.docx")) {
+            XWPFTheme theme = docx.getTheme();
+            assertEquals("Office Theme", theme.getName());
+            assertEquals("Cambria", theme.getMajorFont());
+            assertEquals("Calibri", theme.getMinorFont());
+            CTColor accent1 = theme.getCTColor("accent1");
+            XSLFColor color = new XSLFColor(accent1, null, null, null);
+            assertEquals(new Color(79, 129, 189), color.getColor());
+        }
     }
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org