]> source.dussan.org Git - poi.git/commitdiff
IDE and JavaDoc fixes, close() in tests
authorDominik Stadler <centic@apache.org>
Thu, 27 Dec 2018 20:51:31 +0000 (20:51 +0000)
committerDominik Stadler <centic@apache.org>
Thu, 27 Dec 2018 20:51:31 +0000 (20:51 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1849812 13f79535-47bb-0310-9956-ffa450edef68

src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
src/ooxml/testcases/org/apache/poi/extractor/ooxml/TestExtractorFactory.java
src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFSheetShiftRows.java
src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFSDT.java
src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFSmartTag.java
src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFStyles.java
src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFTableCell.java
src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFTableRow.java
src/testcases/org/apache/poi/hssf/record/TestBoundSheetRecord.java

index 1a4bef5939ceaccdf4092ed95fcba3018f52017f..9eaa7c3eae8f9f647d4de25047c50dfbf7bf789f 100644 (file)
@@ -1571,7 +1571,7 @@ public abstract class OPCPackage implements RelationshipSource, Closeable {
      *
      * @param oldContentType  the content type to be replaced
      * @param newContentType  the replacement
-     * @return whether replacement was succesfull
+     * @return whether replacement was successful
      * @since POI-3.8
      */
     public boolean replaceContentType(String oldContentType, String newContentType){
index 196499efdc1081460e36c95c1e7ffed6fab05608..851f9dd089ad4889ae6ce533819fedc6b0a03e7a 100644 (file)
@@ -470,7 +470,6 @@ public class TestExtractorFactory {
         ExtractorFactory.getEmbededDocsTextExtractors((POIXMLTextExtractor) null);
     }
 
-    @SuppressWarnings("deprecation")
     @Test(expected = IllegalStateException.class)
     public void testGetEmbeddedFromXMLExtractor() {
         // currently not implemented
index fc8e308cb72bc41f05a1fa04f2d08b341b3e1b96..23e02ff241c8bf5f7255d5dd5832c9c9b79ea764 100644 (file)
@@ -112,7 +112,6 @@ public final class TestXSSFSheetShiftRows extends BaseTestSheetShiftRows {
             return;
         }
         Cell readCell = readRow.getCell(0);
-        //noinspection deprecation
         if(readCell.getCellType() == CellType.NUMERIC) {
             assertEquals(expect, Double.toString(readCell.getNumericCellValue()));
         } else {
@@ -470,7 +469,7 @@ public final class TestXSSFSheetShiftRows extends BaseTestSheetShiftRows {
     // bug 60260: shift rows or rename a sheet containing a named range
     // that refers to formula with a unicode (non-ASCII) sheet name formula
     @Test
-    public void shiftRowsWithUnicodeNamedRange() throws IOException {
+    public void shiftRowsWithUnicodeNamedRange() {
         XSSFWorkbook wb = XSSFTestDataSamples.openSampleWorkbook("unicodeSheetName.xlsx");
         XSSFSheet sheet = wb.getSheetAt(0);
         sheet.shiftRows(1, 2, 3);
index 9388e087102658bf10e14d88f08a54a2848f8443..4bf87fcbb0f1420210337cd1c0000fa7adc7c776 100644 (file)
@@ -32,27 +32,26 @@ public final class TestXWPFSDT {
 
     /**
      * Test simple tag and title extraction from SDT
-     *
-     * @throws Exception
      */
     @Test
     public void testTagTitle() throws Exception {
-        XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("Bug54849.docx");
-        String tag = null;
-        String title = null;
-        List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
-        for (XWPFAbstractSDT sdt : sdts) {
-            if (sdt.getContent().toString().equals("Rich_text")) {
-                tag = "MyTag";
-                title = "MyTitle";
-                break;
+        try (XWPFDocument doc =XWPFTestDataSamples.openSampleDocument("Bug54849.docx")) {
+            String tag = null;
+            String title = null;
+            List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
+            for (XWPFAbstractSDT sdt : sdts) {
+                if (sdt.getContent().toString().equals("Rich_text")) {
+                    tag = "MyTag";
+                    title = "MyTitle";
+                    break;
+                }
+
             }
+            assertEquals("controls size", 13, sdts.size());
 
+            assertEquals("tag", "MyTag", tag);
+            assertEquals("title", "MyTitle", title);
         }
-        assertEquals("controls size", 13, sdts.size());
-
-        assertEquals("tag", "MyTag", tag);
-        assertEquals("title", "MyTitle", title);
     }
 
     @Test
@@ -73,14 +72,15 @@ public final class TestXWPFSDT {
                 "Endnote_sdt"
 
         };
-        XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("Bug54849.docx");
-        List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
+        try (XWPFDocument doc =XWPFTestDataSamples.openSampleDocument("Bug54849.docx")) {
+            List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
 
-        assertEquals("number of sdts", contents.length, sdts.size());
+            assertEquals("number of sdts", contents.length, sdts.size());
 
-        for (int i = 0; i < contents.length; i++) {
-            XWPFAbstractSDT sdt = sdts.get(i);
-            assertEquals(i + ": " + contents[i], contents[i], sdt.getContent().toString());
+            for (int i = 0; i < contents.length; i++) {
+                XWPFAbstractSDT sdt = sdts.get(i);
+                assertEquals(i + ": " + contents[i], contents[i], sdt.getContent().toString());
+            }
         }
     }
 
@@ -91,25 +91,26 @@ public final class TestXWPFSDT {
     public void testSDTAsCell() throws Exception {
         //Bug54771a.docx and Bug54771b.docx test slightly 
         //different recursion patterns. Keep both!
-        XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("Bug54771a.docx");
-        List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
-        String text = sdts.get(0).getContent().getText();
-        assertEquals(2, sdts.size());
-        assertContains(text, "Test");
-
-        text = sdts.get(1).getContent().getText();
-        assertContains(text, "Test Subtitle");
-        assertContains(text, "Test User");
-        assertTrue(text.indexOf("Test") < text.indexOf("Test Subtitle"));
-
-        doc = XWPFTestDataSamples.openSampleDocument("Bug54771b.docx");
-        sdts = extractAllSDTs(doc);
-        assertEquals(3, sdts.size());
-        assertContains(sdts.get(0).getContent().getText(), "Test");
+        try (XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("Bug54771a.docx")) {
+            List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
+            String text = sdts.get(0).getContent().getText();
+            assertEquals(2, sdts.size());
+            assertContains(text, "Test");
+
+            text = sdts.get(1).getContent().getText();
+            assertContains(text, "Test Subtitle");
+            assertContains(text, "Test User");
+            assertTrue(text.indexOf("Test") < text.indexOf("Test Subtitle"));
+        }
 
-        assertContains(sdts.get(1).getContent().getText(), "Test Subtitle");
-        assertContains(sdts.get(2).getContent().getText(), "Test User");
+        try (XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("Bug54771b.docx")) {
+             List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
+            assertEquals(3, sdts.size());
+            assertContains(sdts.get(0).getContent().getText(), "Test");
 
+            assertContains(sdts.get(1).getContent().getText(), "Test Subtitle");
+            assertContains(sdts.get(2).getContent().getText(), "Test User");
+        }
     }
 
     /**
@@ -117,35 +118,37 @@ public final class TestXWPFSDT {
      */
     @Test
     public void testNewLinesBetweenRuns() throws Exception {
-        XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("Bug55142.docx");
-        List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
-        List<String> targs = new ArrayList<>();
-        //these test newlines and tabs in paragraphs/body elements
-        targs.add("Rich-text1 abcdefghi");
-        targs.add("Rich-text2 abcd\t\tefgh");
-        targs.add("Rich-text3 abcd\nefg");
-        targs.add("Rich-text4 abcdefg");
-        targs.add("Rich-text5 abcdefg\nhijk");
-        targs.add("Plain-text1 abcdefg");
-        targs.add("Plain-text2 abcdefg\nhijk\nlmnop");
-        //this tests consecutive runs within a cell (not a paragraph)
-        //this test case was triggered by Tika-1130
-        targs.add("sdt_incell2 abcdefg");
-
-        for (int i = 0; i < sdts.size(); i++) {
-            XWPFAbstractSDT sdt = sdts.get(i);
-            assertEquals(targs.get(i), targs.get(i), sdt.getContent().getText());
+        try (XWPFDocument doc =XWPFTestDataSamples.openSampleDocument("Bug55142.docx")) {
+            List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
+            List<String> targs = new ArrayList<>();
+            //these test newlines and tabs in paragraphs/body elements
+            targs.add("Rich-text1 abcdefghi");
+            targs.add("Rich-text2 abcd\t\tefgh");
+            targs.add("Rich-text3 abcd\nefg");
+            targs.add("Rich-text4 abcdefg");
+            targs.add("Rich-text5 abcdefg\nhijk");
+            targs.add("Plain-text1 abcdefg");
+            targs.add("Plain-text2 abcdefg\nhijk\nlmnop");
+            //this tests consecutive runs within a cell (not a paragraph)
+            //this test case was triggered by Tika-1130
+            targs.add("sdt_incell2 abcdefg");
+
+            for (int i = 0; i < sdts.size(); i++) {
+                XWPFAbstractSDT sdt = sdts.get(i);
+                assertEquals(targs.get(i), targs.get(i), sdt.getContent().getText());
+            }
         }
     }
 
     @Test
     public void test60341() throws IOException {
         //handle sdtbody without an sdtpr
-        XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("Bug60341.docx");
-        List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
-        assertEquals(1, sdts.size());
-        assertEquals("", sdts.get(0).getTag());
-        assertEquals("", sdts.get(0).getTitle());
+        try (XWPFDocument doc =XWPFTestDataSamples.openSampleDocument("Bug60341.docx")) {
+            List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
+            assertEquals(1, sdts.size());
+            assertEquals("", sdts.get(0).getTag());
+            assertEquals("", sdts.get(0).getTitle());
+        }
     }
 
     @Test
@@ -154,15 +157,15 @@ public final class TestXWPFSDT {
         //it does test for a related issue, and the fix fixes both.
         //We should try to add the actual triggering document
         //to our test suite.
-        XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("Bug62859.docx");
-        List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
-        assertEquals(1, sdts.size());
-        assertEquals("", sdts.get(0).getTag());
-        assertEquals("", sdts.get(0).getTitle());
+        try (XWPFDocument doc =XWPFTestDataSamples.openSampleDocument("Bug62859.docx")) {
+            List<XWPFAbstractSDT> sdts = extractAllSDTs(doc);
+            assertEquals(1, sdts.size());
+            assertEquals("", sdts.get(0).getTag());
+            assertEquals("", sdts.get(0).getTitle());
+        }
     }
 
     private List<XWPFAbstractSDT> extractAllSDTs(XWPFDocument doc) {
-
         List<XWPFAbstractSDT> sdts = new ArrayList<>();
 
         List<XWPFHeader> headers = doc.getHeaderList();
index b3780cd2ae68c60124982493904603923a66018d..c3a0409023d9a6d18df4c190108ecd5d9f04f9c0 100644 (file)
@@ -30,10 +30,11 @@ import static org.apache.poi.POITestCase.assertContains;
 public final class TestXWPFSmartTag extends TestCase {
 
     public void testSmartTags() throws IOException {
-        XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("smarttag-snippet.docx");
-        XWPFParagraph p = doc.getParagraphArray(0);
-        assertContains(p.getText(), "Carnegie Mellon University School of Computer Science");
-        p = doc.getParagraphArray(2);
-        assertContains(p.getText(), "Alice's Adventures");
+        try (XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("smarttag-snippet.docx")) {
+            XWPFParagraph p = doc.getParagraphArray(0);
+            assertContains(p.getText(), "Carnegie Mellon University School of Computer Science");
+            p = doc.getParagraphArray(2);
+            assertContains(p.getText(), "Alice's Adventures");
+        }
     }
 }
index 8ac9c083d59e15ba1950bfdd1a06fba9ff51e870..996f50b4f8d5e69a7b41addce8a7d320bb71384c 100644 (file)
@@ -18,6 +18,7 @@
 package org.apache.poi.xwpf.usermodel;
 
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
@@ -39,18 +40,19 @@ import org.openxmlformats.schemas.wordprocessingml.x2006.main.STStyleType;
 public final class TestXWPFStyles {
     @Test
     public void testGetUsedStyles() throws IOException {
-        XWPFDocument sampleDoc = XWPFTestDataSamples.openSampleDocument("Styles.docx");
-        List<XWPFStyle> testUsedStyleList = new ArrayList<>();
-        XWPFStyles styles = sampleDoc.getStyles();
-        XWPFStyle style = styles.getStyle("berschrift1");
-        testUsedStyleList.add(style);
-        testUsedStyleList.add(styles.getStyle("Standard"));
-        testUsedStyleList.add(styles.getStyle("berschrift1Zchn"));
-        testUsedStyleList.add(styles.getStyle("Absatz-Standardschriftart"));
-        style.hasSameName(style);
-
-        List<XWPFStyle> usedStyleList = styles.getUsedStyleList(style);
-        assertEquals(usedStyleList, testUsedStyleList);
+        try (XWPFDocument sampleDoc = XWPFTestDataSamples.openSampleDocument("Styles.docx")) {
+            List<XWPFStyle> testUsedStyleList = new ArrayList<>();
+            XWPFStyles styles = sampleDoc.getStyles();
+            XWPFStyle style = styles.getStyle("berschrift1");
+            testUsedStyleList.add(style);
+            testUsedStyleList.add(styles.getStyle("Standard"));
+            testUsedStyleList.add(styles.getStyle("berschrift1Zchn"));
+            testUsedStyleList.add(styles.getStyle("Absatz-Standardschriftart"));
+            style.hasSameName(style);
+
+            List<XWPFStyle> usedStyleList = styles.getUsedStyleList(style);
+            assertEquals(usedStyleList, testUsedStyleList);
+        }
     }
 
     @Test
@@ -79,13 +81,14 @@ public final class TestXWPFStyles {
      */
     @Test
     public void test52449() throws Exception {
-        XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("52449.docx");
-        XWPFStyles styles = doc.getStyles();
-        assertNotNull(styles);
+        try (XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("52449.docx")) {
+            XWPFStyles styles = doc.getStyles();
+            assertNotNull(styles);
 
-        XWPFDocument docIn = XWPFTestDataSamples.writeOutAndReadBack(doc);
-        styles = docIn.getStyles();
-        assertNotNull(styles);
+            XWPFDocument docIn = XWPFTestDataSamples.writeOutAndReadBack(doc);
+            styles = docIn.getStyles();
+            assertNotNull(styles);
+        }
     }
 
 
@@ -121,8 +124,8 @@ public final class TestXWPFStyles {
         CTLsdException ex = latentStyles.addNewLsdException();
         ex.setName("ex1");
         XWPFLatentStyles ls = new XWPFLatentStyles(latentStyles);
-        assertEquals(true, ls.isLatentStyle("ex1"));
-        assertEquals(false, ls.isLatentStyle("notex1"));
+        assertTrue(ls.isLatentStyle("ex1"));
+        assertFalse(ls.isLatentStyle("notex1"));
     }
 
     @Test
@@ -147,88 +150,89 @@ public final class TestXWPFStyles {
 
     @Test
     public void testEasyAccessToStyles() throws IOException {
-        XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("SampleDoc.docx");
-        XWPFStyles styles = doc.getStyles();
-        assertNotNull(styles);
-
-        // Has 3 paragraphs on page one, a break, and 3 on page 2
-        assertEquals(7, doc.getParagraphs().size());
-
-        // Check the first three have no run styles, just default paragraph style
-        for (int i = 0; i < 3; i++) {
-            XWPFParagraph p = doc.getParagraphs().get(i);
-            assertEquals(null, p.getStyle());
-            assertEquals(null, p.getStyleID());
-            assertEquals(1, p.getRuns().size());
-
-            XWPFRun r = p.getRuns().get(0);
-            assertEquals(null, r.getColor());
-            assertEquals(null, r.getFontFamily());
-            assertEquals(null, r.getFontName());
-            assertEquals(-1, r.getFontSize());
-        }
-
-        // On page two, has explicit styles, but on runs not on
-        //  the paragraph itself
-        for (int i = 4; i < 7; i++) {
-            XWPFParagraph p = doc.getParagraphs().get(i);
-            assertEquals(null, p.getStyle());
-            assertEquals(null, p.getStyleID());
-            assertEquals(1, p.getRuns().size());
-
-            XWPFRun r = p.getRuns().get(0);
-            assertEquals("Arial Black", r.getFontFamily());
-            assertEquals("Arial Black", r.getFontName());
-            assertEquals(16, r.getFontSize());
-            assertEquals("548DD4", r.getColor());
-        }
-
-        // Check the document styles
-        // Should have a style defined for each type
-        assertEquals(4, styles.getNumberOfStyles());
-        assertNotNull(styles.getStyle("Normal"));
-        assertNotNull(styles.getStyle("DefaultParagraphFont"));
-        assertNotNull(styles.getStyle("TableNormal"));
-        assertNotNull(styles.getStyle("NoList"));
+        try (XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("SampleDoc.docx")) {
+            XWPFStyles styles = doc.getStyles();
+            assertNotNull(styles);
+
+            // Has 3 paragraphs on page one, a break, and 3 on page 2
+            assertEquals(7, doc.getParagraphs().size());
+
+            // Check the first three have no run styles, just default paragraph style
+            for (int i = 0; i < 3; i++) {
+                XWPFParagraph p = doc.getParagraphs().get(i);
+                assertNull(p.getStyle());
+                assertNull(p.getStyleID());
+                assertEquals(1, p.getRuns().size());
+
+                XWPFRun r = p.getRuns().get(0);
+                assertNull(r.getColor());
+                assertNull(r.getFontFamily());
+                assertNull(r.getFontName());
+                assertEquals(-1, r.getFontSize());
+            }
+
+            // On page two, has explicit styles, but on runs not on
+            //  the paragraph itself
+            for (int i = 4; i < 7; i++) {
+                XWPFParagraph p = doc.getParagraphs().get(i);
+                assertNull(p.getStyle());
+                assertNull(p.getStyleID());
+                assertEquals(1, p.getRuns().size());
+
+                XWPFRun r = p.getRuns().get(0);
+                assertEquals("Arial Black", r.getFontFamily());
+                assertEquals("Arial Black", r.getFontName());
+                assertEquals(16, r.getFontSize());
+                assertEquals("548DD4", r.getColor());
+            }
+
+            // Check the document styles
+            // Should have a style defined for each type
+            assertEquals(4, styles.getNumberOfStyles());
+            assertNotNull(styles.getStyle("Normal"));
+            assertNotNull(styles.getStyle("DefaultParagraphFont"));
+            assertNotNull(styles.getStyle("TableNormal"));
+            assertNotNull(styles.getStyle("NoList"));
 
-        // We can't do much yet with latent styles
-        assertEquals(137, styles.getLatentStyles().getNumberOfStyles());
+            // We can't do much yet with latent styles
+            assertEquals(137, styles.getLatentStyles().getNumberOfStyles());
 
-        // Check the default styles
-        assertNotNull(styles.getDefaultRunStyle());
-        assertNotNull(styles.getDefaultParagraphStyle());
+            // Check the default styles
+            assertNotNull(styles.getDefaultRunStyle());
+            assertNotNull(styles.getDefaultParagraphStyle());
 
-        assertEquals(11, styles.getDefaultRunStyle().getFontSize());
-        assertEquals(200, styles.getDefaultParagraphStyle().getSpacingAfter());
+            assertEquals(11, styles.getDefaultRunStyle().getFontSize());
+            assertEquals(200, styles.getDefaultParagraphStyle().getSpacingAfter());
+        }
     }
     
     // Bug 60329: style with missing StyleID throws NPE
     @Test
     public void testMissingStyleId() throws IOException {
-        XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("60329.docx");
-        XWPFStyles styles = doc.getStyles();
-        // Styles exist in the test document in this order, EmptyCellLayoutStyle
-        // is missing a StyleId
-        try {
-            assertNotNull(styles.getStyle("NoList"));
-            assertNull(styles.getStyle("EmptyCellLayoutStyle"));
-            assertNotNull(styles.getStyle("BalloonText"));
-        } catch (NullPointerException e) {
-            fail(e.toString());
+        try (XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("60329.docx")) {
+            XWPFStyles styles = doc.getStyles();
+            // Styles exist in the test document in this order, EmptyCellLayoutStyle
+            // is missing a StyleId
+            try {
+                assertNotNull(styles.getStyle("NoList"));
+                assertNull(styles.getStyle("EmptyCellLayoutStyle"));
+                assertNotNull(styles.getStyle("BalloonText"));
+            } catch (NullPointerException e) {
+                fail(e.toString());
+            }
         }
-
-        doc.close();
     }
     
     @Test
     public void testGetStyleByName() throws IOException {
-        XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("SampleDoc.docx");
-        XWPFStyles styles = doc.getStyles();
-        assertNotNull(styles);
-
-        String styleName = "Normal Table";
-        XWPFStyle style = styles.getStyleWithName(styleName);
-        assertNotNull("Expected to find style \"" + styleName + "\"", style);
-        assertEquals(styleName, style.getName());
+        try (XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("SampleDoc.docx")) {
+            XWPFStyles styles = doc.getStyles();
+            assertNotNull(styles);
+
+            String styleName = "Normal Table";
+            XWPFStyle style = styles.getStyleWithName(styleName);
+            assertNotNull("Expected to find style \"" + styleName + "\"", style);
+            assertEquals(styleName, style.getName());
+        }
     }
 }
index 4b711c6a77376d5bce64b1b5ff676e7fefe477a0..df3fa2a7a4ec68a66473809cff9c14d225c7cde5 100644 (file)
@@ -33,7 +33,7 @@ import java.util.List;
 public class TestXWPFTableCell {
 
     @Test
-    public void testSetGetVertAlignment() throws Exception {
+    public void testSetGetVertAlignment() {
         // instantiate the following classes so they'll get picked up by
         // the XmlBean process and added to the jar file. they are required
         // for the following XWPFTableCell methods.
@@ -62,7 +62,7 @@ public class TestXWPFTableCell {
     }
 
     @Test
-    public void testSetGetColor() throws Exception {
+    public void testSetGetColor() {
         // create a table
         XWPFDocument doc = new XWPFDocument();
         CTTbl ctTable = CTTbl.Factory.newInstance();
@@ -81,7 +81,6 @@ public class TestXWPFTableCell {
     /**
      * ensure that CTHMerge and CTTcBorders go in poi-ooxml.jar
      */
-    @SuppressWarnings("unused")
     @Test
     public void test54099() {
         XWPFDocument doc = new XWPFDocument();
@@ -96,24 +95,27 @@ public class TestXWPFTableCell {
         hMerge.setVal(STMerge.RESTART);
 
         CTTcBorders tblBorders = tcPr.addNewTcBorders();
+        assertNotNull(tblBorders);
         CTVMerge vMerge = tcPr.addNewVMerge();
+        assertNotNull(vMerge);
     }
 
     @Test
     public void testCellVerticalAlign() throws Exception{
-        XWPFDocument docx = XWPFTestDataSamples.openSampleDocument("59030.docx");
-        List<XWPFTable> tables = docx.getTables();
-        assertEquals(1, tables.size());
+        try (XWPFDocument docx = XWPFTestDataSamples.openSampleDocument("59030.docx")) {
+            List<XWPFTable> tables = docx.getTables();
+            assertEquals(1, tables.size());
 
-        XWPFTable table = tables.get(0);
+            XWPFTable table = tables.get(0);
 
-        List<XWPFTableRow> tableRows = table.getRows();
-        assertEquals(2, tableRows.size());
+            List<XWPFTableRow> tableRows = table.getRows();
+            assertEquals(2, tableRows.size());
 
-        assertNull(tableRows.get(0).getCell(0).getVerticalAlignment());
-        assertEquals(XWPFVertAlign.BOTTOM, tableRows.get(0).getCell(1).getVerticalAlignment());
-        assertEquals(XWPFVertAlign.CENTER, tableRows.get(1).getCell(0).getVerticalAlignment());
-        assertNull(tableRows.get(1).getCell(1).getVerticalAlignment()); // should return null since alignment isn't set
+            assertNull(tableRows.get(0).getCell(0).getVerticalAlignment());
+            assertEquals(XWPFVertAlign.BOTTOM, tableRows.get(0).getCell(1).getVerticalAlignment());
+            assertEquals(XWPFVertAlign.CENTER, tableRows.get(1).getCell(0).getVerticalAlignment());
+            assertNull(tableRows.get(1).getCell(1).getVerticalAlignment()); // should return null since alignment isn't set
+        }
     }
 
     // This is not a very useful test as written. It is not worth the execution time for a unit test
index 9367d628cba450e7aaacaf85923733bb86eae6b2..383e5c3c94adb77cad644372c6955f955e0cff42 100644 (file)
@@ -102,20 +102,21 @@ public class TestXWPFTableRow {
     // generated in Word
     @Test
     public void testIsRepeatHeader() throws Exception {
-        XWPFDocument doc = XWPFTestDataSamples
-                .openSampleDocument("Bug60337.docx");
-        XWPFTable table = doc.getTables().get(0);
-        XWPFTableRow tr = table.getRow(0);
-        boolean isRpt = tr.isRepeatHeader();
-        assertTrue(isRpt);
-        
-        tr = table.getRow(1);
-        isRpt = tr.isRepeatHeader();
-        assertFalse(isRpt);
-        
-        tr = table.getRow(2);
-        isRpt = tr.isRepeatHeader();
-        assertFalse(isRpt);
+        try (XWPFDocument doc = XWPFTestDataSamples
+                .openSampleDocument("Bug60337.docx")) {
+            XWPFTable table = doc.getTables().get(0);
+            XWPFTableRow tr = table.getRow(0);
+            boolean isRpt = tr.isRepeatHeader();
+            assertTrue(isRpt);
+
+            tr = table.getRow(1);
+            isRpt = tr.isRepeatHeader();
+            assertFalse(isRpt);
+
+            tr = table.getRow(2);
+            isRpt = tr.isRepeatHeader();
+            assertFalse(isRpt);
+        }
     }
     
     
@@ -123,20 +124,21 @@ public class TestXWPFTableRow {
     // generated in Word
     @Test
     public void testIsCantSplit() throws Exception {
-        XWPFDocument doc = XWPFTestDataSamples
-                .openSampleDocument("Bug60337.docx");
-        XWPFTable table = doc.getTables().get(0);
-        XWPFTableRow tr = table.getRow(0);
-        boolean isCantSplit = tr.isCantSplitRow();
-        assertFalse(isCantSplit);
-        
-        tr = table.getRow(1);
-        isCantSplit = tr.isCantSplitRow();
-        assertFalse(isCantSplit);
+        try (XWPFDocument doc = XWPFTestDataSamples
+                .openSampleDocument("Bug60337.docx")) {
+            XWPFTable table = doc.getTables().get(0);
+            XWPFTableRow tr = table.getRow(0);
+            boolean isCantSplit = tr.isCantSplitRow();
+            assertFalse(isCantSplit);
 
-        tr = table.getRow(2);
-        isCantSplit = tr.isCantSplitRow();
-        assertTrue(isCantSplit);
+            tr = table.getRow(1);
+            isCantSplit = tr.isCantSplitRow();
+            assertFalse(isCantSplit);
+
+            tr = table.getRow(2);
+            isCantSplit = tr.isCantSplitRow();
+            assertTrue(isCantSplit);
+        }
     }
 
     @Test
index b231e36781d7315fc94b91ea651af6c8e38c0a0d..6a1dccac280bd0ef84582532ee0d60984c4e74d1 100644 (file)
@@ -52,7 +52,7 @@ public final class TestBoundSheetRecord extends TestCase {
 
                try {
                        record.setSheetname("s//*s");
-                       throw new AssertionFailedError("Should have thrown IllegalArgumentException, but didnt");
+                       fail("Should have thrown IllegalArgumentException, but didnt");
                } catch (IllegalArgumentException e) {
                        // expected
                }