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 2017/09/16 08:20:26 UTC

svn commit: r1808502 - in /poi/trunk: src/testcases/org/apache/poi/hpsf/extractor/TestHPSFPropertiesExtractor.java src/testcases/org/apache/poi/hssf/usermodel/TestPOIFSProperties.java test-data/poifs/61300.bin

Author: centic
Date: Sat Sep 16 08:20:25 2017
New Revision: 1808502

URL: http://svn.apache.org/viewvc?rev=1808502&view=rev
Log:
Tried to reproduce bug 61300, but could not.

Added:
    poi/trunk/test-data/poifs/61300.bin
Modified:
    poi/trunk/src/testcases/org/apache/poi/hpsf/extractor/TestHPSFPropertiesExtractor.java
    poi/trunk/src/testcases/org/apache/poi/hssf/usermodel/TestPOIFSProperties.java

Modified: poi/trunk/src/testcases/org/apache/poi/hpsf/extractor/TestHPSFPropertiesExtractor.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/hpsf/extractor/TestHPSFPropertiesExtractor.java?rev=1808502&r1=1808501&r2=1808502&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/hpsf/extractor/TestHPSFPropertiesExtractor.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/hpsf/extractor/TestHPSFPropertiesExtractor.java Sat Sep 16 08:20:25 2017
@@ -18,33 +18,36 @@
 package org.apache.poi.hpsf.extractor;
 
 import static org.apache.poi.POITestCase.assertContains;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
 
 import java.io.IOException;
-
-import junit.framework.TestCase;
+import java.io.InputStream;
 
 import org.apache.poi.POIDataSamples;
-import org.apache.poi.hpsf.Thumbnail;
+import org.apache.poi.hpsf.*;
 import org.apache.poi.hssf.HSSFTestDataSamples;
 import org.apache.poi.hssf.extractor.ExcelExtractor;
 import org.apache.poi.hssf.usermodel.HSSFWorkbook;
 import org.apache.poi.poifs.filesystem.POIFSFileSystem;
+import org.junit.Test;
 
-public final class TestHPSFPropertiesExtractor extends TestCase {
+public final class TestHPSFPropertiesExtractor {
 	private static final POIDataSamples _samples = POIDataSamples.getHPSFInstance();
 
+	@Test
 	public void testNormalProperties() throws Exception {
 		POIFSFileSystem fs = new POIFSFileSystem(_samples.openResourceAsStream("TestMickey.doc"));
 		HPSFPropertiesExtractor ext = new HPSFPropertiesExtractor(fs);
 		try {
     		// Check each bit in turn
     		String summary = ext.getSummaryInformationText();
-    		String docsummary = ext.getDocumentSummaryInformationText();
+    		String docSummary = ext.getDocumentSummaryInformationText();
 
     		assertContains(summary, "TEMPLATE = Normal");
     		assertContains(summary, "SUBJECT = sample subject");
-    		assertContains(docsummary, "MANAGER = sample manager");
-    		assertContains(docsummary, "COMPANY = sample company");
+    		assertContains(docSummary, "MANAGER = sample manager");
+    		assertContains(docSummary, "COMPANY = sample company");
     
     		// Now overall
     		String text = ext.getText();
@@ -57,18 +60,19 @@ public final class TestHPSFPropertiesExt
 		}
 	}
 
+	@Test
 	public void testNormalUnicodeProperties() throws Exception {
 		POIFSFileSystem fs = new POIFSFileSystem(_samples.openResourceAsStream("TestUnicode.xls"));
 		HPSFPropertiesExtractor ext = new HPSFPropertiesExtractor(fs);
 		try {
     		// Check each bit in turn
     		String summary = ext.getSummaryInformationText();
-    		String docsummary = ext.getDocumentSummaryInformationText();
+    		String docSummary = ext.getDocumentSummaryInformationText();
     
     		assertContains(summary, "AUTHOR = marshall");
     		assertContains(summary, "TITLE = Titel: \u00c4h");
-    		assertContains(docsummary, "COMPANY = Schreiner");
-    		assertContains(docsummary, "SCALE = false");
+    		assertContains(docSummary, "COMPANY = Schreiner");
+    		assertContains(docSummary, "SCALE = false");
     
     		// Now overall
     		String text = ext.getText();
@@ -81,6 +85,7 @@ public final class TestHPSFPropertiesExt
 		}
 	}
 
+	@Test
 	public void testCustomProperties() throws Exception {
 		POIFSFileSystem fs = new POIFSFileSystem(
 				_samples.openResourceAsStream("TestMickey.doc")
@@ -100,6 +105,7 @@ public final class TestHPSFPropertiesExt
         }
 	}
 
+	@Test
 	public void testConstructors() throws IOException {
 		POIFSFileSystem fs;
 		HSSFWorkbook wb;
@@ -146,6 +152,7 @@ public final class TestHPSFPropertiesExt
 		assertContains(fsText, "TITLE = Titel: \u00c4h");
 	}
 
+	@Test
 	public void test42726() throws IOException {
 		HPSFPropertiesExtractor ext = new HPSFPropertiesExtractor(HSSFTestDataSamples.openSampleWorkbook("42726.xls"));
 		try {
@@ -158,7 +165,8 @@ public final class TestHPSFPropertiesExt
             ext.close();
         }
 	}
-	
+
+	@Test
     public void testThumbnail() throws Exception {
         POIFSFileSystem fs = new POIFSFileSystem(_samples.openResourceAsStream("TestThumbnail.xls"));
         HSSFWorkbook wb = new HSSFWorkbook(fs);
@@ -169,6 +177,7 @@ public final class TestHPSFPropertiesExt
         wb.close();
     }
 
+	@Test
     public void test52258() throws Exception {
         POIFSFileSystem fs = new POIFSFileSystem(_samples.openResourceAsStream("TestVisioWithCodepage.vsd"));
         HPSFPropertiesExtractor ext = new HPSFPropertiesExtractor(fs);
@@ -182,4 +191,11 @@ public final class TestHPSFPropertiesExt
             ext.close();
         }
     }
+
+	@Test
+	public void test61300Extractor() throws NoPropertySetStreamException, MarkUnsupportedException, IOException {
+		HPSFPropertiesExtractor.main(new String[]{
+				POIDataSamples.getPOIFSInstance().getFile("61300.bin").getAbsolutePath()
+		});
+	}
 }

Modified: poi/trunk/src/testcases/org/apache/poi/hssf/usermodel/TestPOIFSProperties.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/hssf/usermodel/TestPOIFSProperties.java?rev=1808502&r1=1808501&r2=1808502&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/hssf/usermodel/TestPOIFSProperties.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/hssf/usermodel/TestPOIFSProperties.java Sat Sep 16 08:20:25 2017
@@ -25,11 +25,7 @@ import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.InputStream;
 
-import org.apache.poi.hpsf.MarkUnsupportedException;
-import org.apache.poi.hpsf.NoPropertySetStreamException;
-import org.apache.poi.hpsf.PropertySetFactory;
-import org.apache.poi.hpsf.SummaryInformation;
-import org.apache.poi.hpsf.WritingNotSupportedException;
+import org.apache.poi.hpsf.*;
 import org.apache.poi.hssf.HSSFTestDataSamples;
 import org.apache.poi.poifs.filesystem.POIFSFileSystem;
 import org.apache.poi.util.HexDump;

Added: poi/trunk/test-data/poifs/61300.bin
URL: http://svn.apache.org/viewvc/poi/trunk/test-data/poifs/61300.bin?rev=1808502&view=auto
==============================================================================
Binary files poi/trunk/test-data/poifs/61300.bin (added) and poi/trunk/test-data/poifs/61300.bin Sat Sep 16 08:20:25 2017 differ



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