aboutsummaryrefslogtreecommitdiffstats
path: root/src/testcases/org/apache/poi
diff options
context:
space:
mode:
authorAndreas Beeker <kiwiwings@apache.org>2020-08-15 00:18:42 +0000
committerAndreas Beeker <kiwiwings@apache.org>2020-08-15 00:18:42 +0000
commit60303201dcb715a7738cd5bff3a85ffb424a2a90 (patch)
tree1154b264901ee5e32e7001175063d64b81f8b73f /src/testcases/org/apache/poi
parente68bd7bd7ded6948a3289c6a6e3ad37453d12c42 (diff)
downloadpoi-60303201dcb715a7738cd5bff3a85ffb424a2a90.tar.gz
poi-60303201dcb715a7738cd5bff3a85ffb424a2a90.zip
use the forbidden-apis policies corresponding to the JRE
fix the forbidden apis issues git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1880866 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'src/testcases/org/apache/poi')
-rw-r--r--src/testcases/org/apache/poi/hpsf/basic/TestMetaDataIPI.java74
-rw-r--r--src/testcases/org/apache/poi/hssf/usermodel/DummyGraphics2d.java10
-rw-r--r--src/testcases/org/apache/poi/hssf/usermodel/TestDataValidation.java564
-rw-r--r--src/testcases/org/apache/poi/ss/formula/constant/TestConstantValueParser.java14
-rw-r--r--src/testcases/org/apache/poi/ss/formula/function/ExcelFileFormatDocFunctionExtractor.java40
-rw-r--r--src/testcases/org/apache/poi/ss/formula/functions/TestStatsLib.java2
-rw-r--r--src/testcases/org/apache/poi/ss/formula/ptg/TestArrayPtg.java2
7 files changed, 328 insertions, 378 deletions
diff --git a/src/testcases/org/apache/poi/hpsf/basic/TestMetaDataIPI.java b/src/testcases/org/apache/poi/hpsf/basic/TestMetaDataIPI.java
index de4dea27ef..9347699566 100644
--- a/src/testcases/org/apache/poi/hpsf/basic/TestMetaDataIPI.java
+++ b/src/testcases/org/apache/poi/hpsf/basic/TestMetaDataIPI.java
@@ -18,6 +18,7 @@
package org.apache.poi.hpsf.basic;
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;
@@ -55,7 +56,7 @@ public final class TestMetaDataIPI {
public void tearDown() throws Exception {
poifs.close();
}
-
+
/**
* Setup is used to get the document ready. Gets the DocumentSummaryInformation and the
* SummaryInformation to reasonable values
@@ -96,12 +97,12 @@ public final class TestMetaDataIPI {
/* Insert some custom properties into the container. */
customProperties.put("Key1", "Value1");
customProperties.put("Schl\u00fcssel2", "Wert2");
- customProperties.put("Sample Integer", new Integer(12345));
- customProperties.put("Sample Boolean", Boolean.TRUE);
+ customProperties.put("Sample Integer", 12345);
+ customProperties.put("Sample Boolean", true);
Date date = new Date();
customProperties.put("Sample Date", date);
- customProperties.put("Sample Double", new Double(-1.0001));
- customProperties.put("Sample Negative Integer", new Integer(-100000));
+ customProperties.put("Sample Double", -1.0001);
+ customProperties.put("Sample Negative Integer", -100000);
dsi.setCustomProperties(customProperties);
@@ -136,17 +137,17 @@ public final class TestMetaDataIPI {
String a2 = (String) customProperties.get("Schl\u00fcssel2");
assertEquals("Schl\u00fcssel2", "Wert2", a2);
Integer a3 = (Integer) customProperties.get("Sample Integer");
- assertEquals("Sample Number", new Integer(12345), a3);
+ assertEquals("Sample Number", 12345, (int)a3);
Boolean a4 = (Boolean) customProperties.get("Sample Boolean");
- assertEquals("Sample Boolean", Boolean.TRUE, a4);
+ assertTrue("Sample Boolean", a4);
Date a5 = (Date) customProperties.get("Sample Date");
assertEquals("Custom Date:", date, a5);
Double a6 = (Double) customProperties.get("Sample Double");
- assertEquals("Custom Float", new Double(-1.0001), a6);
+ assertEquals("Custom Float", -1.0001, a6, 0);
Integer a7 = (Integer) customProperties.get("Sample Negative Integer");
- assertEquals("Neg", new Integer(-100000), a7);
+ assertEquals("Neg", -100000, (int)a7);
}
/**
@@ -185,7 +186,7 @@ public final class TestMetaDataIPI {
/* Insert some custom properties into the container. */
customProperties.put(k1, p1);
customProperties.put(k2, p2);
- customProperties.put("Sample Number", new Integer(12345));
+ customProperties.put("Sample Number", 12345);
customProperties.put("Sample Boolean", Boolean.TRUE);
Date date = new Date();
customProperties.put("Sample Date", date);
@@ -226,9 +227,9 @@ public final class TestMetaDataIPI {
String a2 = (String) customProperties.get(k2);
assertEquals("Schl\u00fcssel2", p2, a2);
Integer a3 = (Integer) customProperties.get("Sample Number");
- assertEquals("Sample Number", new Integer(12345), a3);
+ assertEquals("Sample Number", 12345, (int)a3);
Boolean a4 = (Boolean) customProperties.get("Sample Boolean");
- assertEquals("Sample Boolean", Boolean.TRUE, a4);
+ assertTrue("Sample Boolean", a4);
Date a5 = (Date) customProperties.get("Sample Date");
assertEquals("Custom Date:", date, a5);
@@ -271,8 +272,8 @@ public final class TestMetaDataIPI {
/* Insert some custom properties into the container. */
customProperties.put(k1, p1);
customProperties.put(k2, p2);
- customProperties.put("Sample Number", new Integer(12345));
- customProperties.put("Sample Boolean", Boolean.FALSE);
+ customProperties.put("Sample Number", 12345);
+ customProperties.put("Sample Boolean", false);
Date date = new Date(0);
customProperties.put("Sample Date", date);
@@ -313,9 +314,9 @@ public final class TestMetaDataIPI {
String a2 = (String) customProperties.get(k2);
assertEquals("Schl\u00fcssel2", p2, a2);
Integer a3 = (Integer) customProperties.get("Sample Number");
- assertEquals("Sample Number", new Integer(12345), a3);
+ assertEquals("Sample Number", 12345, (int)a3);
Boolean a4 = (Boolean) customProperties.get("Sample Boolean");
- assertEquals("Sample Boolean", Boolean.FALSE, a4);
+ assertFalse("Sample Boolean", a4);
Date a5 = (Date) customProperties.get("Sample Date");
assertEquals("Custom Date:", date, a5);
@@ -359,13 +360,13 @@ public final class TestMetaDataIPI {
si.setComments(comments);
si.setKeywords(keywords);
si.setSubject(subject);
-
+
CustomProperties customProperties = new CustomProperties();
/* Insert some custom properties into the container. */
customProperties.put(k1, p1);
customProperties.put(k2, p2);
- customProperties.put("Sample Number", new Integer(12345));
- customProperties.put("Sample Boolean", Boolean.TRUE);
+ customProperties.put("Sample Number", 12345);
+ customProperties.put("Sample Boolean", true);
Date date = new Date();
customProperties.put("Sample Date", date);
@@ -404,9 +405,9 @@ public final class TestMetaDataIPI {
String a2 = (String) customProperties.get(k2);
assertEquals("Schl\u00fcssel2", p2, a2);
Integer a3 = (Integer) customProperties.get("Sample Number");
- assertEquals("Sample Number", new Integer(12345), a3);
+ assertEquals("Sample Number", 12345, (int)a3);
Boolean a4 = (Boolean) customProperties.get("Sample Boolean");
- assertEquals("Sample Boolean", Boolean.TRUE, a4);
+ assertTrue("Sample Boolean", a4);
Date a5 = (Date) customProperties.get("Sample Date");
assertEquals("Custom Date:", date, a5);
}
@@ -437,16 +438,16 @@ public final class TestMetaDataIPI {
}
/* Insert some custom properties into the container. */
- customProperties.put("int", new Integer(12345));
- customProperties.put("negint", new Integer(-12345));
- customProperties.put("long", new Long(12345));
- customProperties.put("neglong", new Long(-12345));
- customProperties.put("boolean", Boolean.TRUE);
+ customProperties.put("int", 12345);
+ customProperties.put("negint", -12345);
+ customProperties.put("long", 12345L);
+ customProperties.put("neglong", -12345L);
+ customProperties.put("boolean", true);
customProperties.put("string", "a String");
// customProperties.put("float", new Float(12345.0)); is not valid
// customProperties.put("negfloat", new Float(-12345.1)); is not valid
- customProperties.put("double", new Double(12345.2));
- customProperties.put("negdouble", new Double(-12345.3));
+ customProperties.put("double", 12345.2);
+ customProperties.put("negdouble", -12345.3);
// customProperties.put("char", new Character('a')); is not valid
Date date = new Date();
@@ -485,28 +486,28 @@ public final class TestMetaDataIPI {
/* Insert some custom properties into the container. */
Integer a3 = (Integer) customProperties.get("int");
- assertEquals("int", new Integer(12345), a3);
+ assertEquals("int", 12345, (int)a3);
a3 = (Integer) customProperties.get("negint");
- assertEquals("negint", new Integer(-12345), a3);
+ assertEquals("negint", -12345, (int)a3);
Long al = (Long) customProperties.get("neglong");
- assertEquals("neglong", new Long(-12345), al);
+ assertEquals("neglong", -12345L, (long)al);
al = (Long) customProperties.get("long");
- assertEquals("long", new Long(12345), al);
+ assertEquals("long", 12345L, (long)al);
Boolean a4 = (Boolean) customProperties.get("boolean");
- assertEquals("boolean", Boolean.TRUE, a4);
+ assertTrue("boolean", a4);
Date a5 = (Date) customProperties.get("date");
assertEquals("Custom Date:", date, a5);
Double d = (Double) customProperties.get("double");
- assertEquals("int", new Double(12345.2), d);
+ assertEquals("int", 12345.2, d, 0);
d = (Double) customProperties.get("negdouble");
- assertEquals("string", new Double(-12345.3), d);
+ assertEquals("string", -12345.3, d, 0);
String s = (String) customProperties.get("string");
assertEquals("sring", "a String", s);
@@ -543,7 +544,7 @@ public final class TestMetaDataIPI {
/* Read the document summary information. */
DirectoryEntry dir = poifs.getRoot();
-
+
dsi = (DocumentSummaryInformation)PropertySetFactory.create(dir, DocumentSummaryInformation.DEFAULT_STREAM_NAME);
si = (SummaryInformation)PropertySetFactory.create(dir, SummaryInformation.DEFAULT_STREAM_NAME);
}
@@ -595,7 +596,6 @@ public final class TestMetaDataIPI {
sb.append(" ");
char j = (char) rand.nextInt(220);
j += 33;
- // System.out.println(j);
sb.append(">");
sb.append(Character.valueOf(j));
sb.append("=");
diff --git a/src/testcases/org/apache/poi/hssf/usermodel/DummyGraphics2d.java b/src/testcases/org/apache/poi/hssf/usermodel/DummyGraphics2d.java
index 6275fc0d02..a10cc1309d 100644
--- a/src/testcases/org/apache/poi/hssf/usermodel/DummyGraphics2d.java
+++ b/src/testcases/org/apache/poi/hssf/usermodel/DummyGraphics2d.java
@@ -56,13 +56,13 @@ public class DummyGraphics2d extends Graphics2D {
public DummyGraphics2d() {
this(System.out);
}
-
+
public DummyGraphics2d(PrintStream log) {
bufimg = new BufferedImage(1000, 1000, 2);
g2D = (Graphics2D)bufimg.getGraphics();
this.log = log;
}
-
+
public DummyGraphics2d(PrintStream log, Graphics2D g2D) {
this.g2D = g2D;
this.log = log;
@@ -86,7 +86,7 @@ public class DummyGraphics2d extends Graphics2D {
public void draw(Shape s) {
String l =
- "draw(Shape):" +
+ "draw(Shape):" +
"\n s = " + s;
log.println( l );
g2D.draw( s );
@@ -715,8 +715,8 @@ public class DummyGraphics2d extends Graphics2D {
@Override
public final void finalize() {
log.println( "finalize():" );
- g2D.finalize(); // NOSOLAR
- super.finalize();
+ g2D.dispose();
+ dispose();
}
public Shape getClip() {
diff --git a/src/testcases/org/apache/poi/hssf/usermodel/TestDataValidation.java b/src/testcases/org/apache/poi/hssf/usermodel/TestDataValidation.java
index e5ddcaa7f7..1e3fc381cc 100644
--- a/src/testcases/org/apache/poi/hssf/usermodel/TestDataValidation.java
+++ b/src/testcases/org/apache/poi/hssf/usermodel/TestDataValidation.java
@@ -61,14 +61,14 @@ public final class TestDataValidation extends BaseTestDataValidation {
public void assertDataValidation(Workbook wb) {
- ByteArrayOutputStream baos = new ByteArrayOutputStream(22000);
- try {
+ byte[] generatedContent;
+ try (ByteArrayOutputStream baos = new ByteArrayOutputStream(22000)) {
wb.write(baos);
- baos.close();
+ generatedContent = baos.toByteArray();
} catch (IOException e) {
throw new RuntimeException(e);
}
- byte[] generatedContent = baos.toByteArray();
+
boolean isSame;
// if (false) {
// // TODO - add proof spreadsheet and compare
@@ -76,23 +76,21 @@ public final class TestDataValidation extends BaseTestDataValidation {
// isSame = compareStreams(proofStream, generatedContent);
// }
isSame = true;
-
+
if (isSame) {
return;
}
File tempDir = new File(System.getProperty("java.io.tmpdir"));
File generatedFile = new File(tempDir, "GeneratedTestDataValidation.xls");
- try {
- FileOutputStream fileOut = new FileOutputStream(generatedFile);
+ try (FileOutputStream fileOut = new FileOutputStream(generatedFile)) {
fileOut.write(generatedContent);
- fileOut.close();
} catch (IOException e) {
throw new RuntimeException(e);
}
-
+
PrintStream ps = System.out;
-
- ps.println("This test case has failed because the generated file differs from proof copy '"
+
+ ps.println("This test case has failed because the generated file differs from proof copy '"
); // TODO+ proofFile.getAbsolutePath() + "'.");
ps.println("The cause is usually a change to this test, or some common spreadsheet generation code. "
+ "The developer has to decide whether the changes were wanted or unwanted.");
@@ -104,68 +102,43 @@ public final class TestDataValidation extends BaseTestDataValidation {
ps.println("One other possible (but less likely) cause of a failed test is a problem in the "
+ "comparison logic used here. Perhaps some extra file regions need to be ignored.");
ps.println("The generated file has been saved to '" + generatedFile.getAbsolutePath() + "' for manual inspection.");
-
- fail("Generated file differs from proof copy. See sysout comments for details on how to fix.");
-
- }
-
-// private static boolean compareStreams(InputStream isA, byte[] generatedContent) {
-//
-// InputStream isB = new ByteArrayInputStream(generatedContent);
-//
-// // The allowable regions where the generated file can differ from the
-// // proof should be small (i.e. much less than 1K)
-// int[] allowableDifferenceRegions = {
-// 0x0228, 16, // a region of the file containing the OS username
-// 0x506C, 8, // See RootProperty (super fields _seconds_2 and _days_2)
-// };
-// int[] diffs = StreamUtility.diffStreams(isA, isB, allowableDifferenceRegions);
-// if (diffs == null) {
-// return true;
-// }
-// System.err.println("Diff from proof: ");
-// for (int i = 0; i < diffs.length; i++) {
-// System.err.println("diff at offset: 0x" + Integer.toHexString(diffs[i]));
-// }
-// return false;
-// }
-
-
+ fail("Generated file differs from proof copy. See sysout comments for details on how to fix.");
+ }
/* package */ static void setCellValue(HSSFCell cell, String text) {
cell.setCellValue(new HSSFRichTextString(text));
-
+
}
-
+
@Test
public void testAddToExistingSheet() throws Exception {
- // dvEmpty.xls is a simple one sheet workbook. With a DataValidations header record but no
+ // dvEmpty.xls is a simple one sheet workbook. With a DataValidations header record but no
// DataValidations. It's important that the example has one SHEETPROTECTION record.
// Such a workbook can be created in Excel (2007) by adding datavalidation for one cell
// and then deleting the row that contains the cell.
- HSSFWorkbook wb = HSSFTestDataSamples.openSampleWorkbook("dvEmpty.xls");
- int dvRow = 0;
- Sheet sheet = wb.getSheetAt(0);
- DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
- DataValidationConstraint dc = dataValidationHelper.createIntegerConstraint(OperatorType.EQUAL, "42", null);
- DataValidation dv = dataValidationHelper.createValidation(dc,new CellRangeAddressList(dvRow, dvRow, 0, 0));
-
- dv.setEmptyCellAllowed(false);
- dv.setErrorStyle(ErrorStyle.STOP);
- dv.setShowPromptBox(true);
- dv.createErrorBox("Xxx", "Yyy");
- dv.setSuppressDropDownArrow(true);
-
- sheet.addValidationData(dv);
-
- ByteArrayOutputStream baos = new ByteArrayOutputStream();
- wb.write(baos);
-
- byte[] wbData = baos.toByteArray();
-
+ try (HSSFWorkbook wb = HSSFTestDataSamples.openSampleWorkbook("dvEmpty.xls")) {
+ int dvRow = 0;
+ Sheet sheet = wb.getSheetAt(0);
+ DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
+ DataValidationConstraint dc = dataValidationHelper.createIntegerConstraint(OperatorType.EQUAL, "42", null);
+ DataValidation dv = dataValidationHelper.createValidation(dc, new CellRangeAddressList(dvRow, dvRow, 0, 0));
+
+ dv.setEmptyCellAllowed(false);
+ dv.setErrorStyle(ErrorStyle.STOP);
+ dv.setShowPromptBox(true);
+ dv.createErrorBox("Xxx", "Yyy");
+ dv.setSuppressDropDownArrow(true);
+
+ sheet.addValidationData(dv);
+
+ ByteArrayOutputStream baos = new ByteArrayOutputStream();
+ wb.write(baos);
+
+ byte[] wbData = baos.toByteArray();
+
// if (false) { // TODO (Jul 2008) fix EventRecordFactory to process unknown records, (and DV records for that matter)
//
// ERFListener erfListener = null; // new MyERFListener();
@@ -179,27 +152,25 @@ public final class TestDataValidation extends BaseTestDataValidation {
// throw new RuntimeException(e);
// }
// }
- // else verify record ordering by navigating the raw bytes
-
- byte[] dvHeaderRecStart= { (byte)0xB2, 0x01, 0x12, 0x00, };
- int dvHeaderOffset = findIndex(wbData, dvHeaderRecStart);
- assertTrue(dvHeaderOffset > 0);
- int nextRecIndex = dvHeaderOffset + 22;
- int nextSid
- = ((wbData[nextRecIndex + 0] << 0) & 0x00FF)
- + ((wbData[nextRecIndex + 1] << 8) & 0xFF00)
- ;
- // nextSid should be for a DVRecord. If anything comes between the DV header record
- // and the DV records, Excel will not be able to open the workbook without error.
-
- if (nextSid == 0x0867) {
- fail("Identified bug 45519");
- }
- assertEquals(DVRecord.sid, nextSid);
-
- wb.close();
+ // else verify record ordering by navigating the raw bytes
+
+ byte[] dvHeaderRecStart = {(byte) 0xB2, 0x01, 0x12, 0x00,};
+ int dvHeaderOffset = findIndex(wbData, dvHeaderRecStart);
+ assertTrue(dvHeaderOffset > 0);
+ int nextRecIndex = dvHeaderOffset + 22;
+ int nextSid
+ = ((wbData[nextRecIndex + 0] << 0) & 0x00FF)
+ + ((wbData[nextRecIndex + 1] << 8) & 0xFF00);
+ // nextSid should be for a DVRecord. If anything comes between the DV header record
+ // and the DV records, Excel will not be able to open the workbook without error.
+
+ if (nextSid == 0x0867) {
+ fail("Identified bug 45519");
+ }
+ assertEquals(DVRecord.sid, nextSid);
+ }
}
-
+
private int findIndex(byte[] largeData, byte[] searchPattern) {
byte firstByte = searchPattern[0];
for (int i = 0; i < largeData.length; i++) {
@@ -222,256 +193,243 @@ public final class TestDataValidation extends BaseTestDataValidation {
@Test
public void testGetDataValidationsAny() throws Exception {
- HSSFWorkbook wb = new HSSFWorkbook();
- HSSFSheet sheet = wb.createSheet();
- List<HSSFDataValidation> list = sheet.getDataValidations();
- assertEquals(0, list.size());
-
- DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
- DataValidationConstraint constraint = dataValidationHelper.createNumericConstraint(ValidationType.ANY,
- OperatorType.IGNORED, null, null);
- CellRangeAddressList addressList = new CellRangeAddressList(1, 2, 3, 4);
- DataValidation validation = dataValidationHelper.createValidation(constraint, addressList);
- validation.setEmptyCellAllowed(true);
- validation.createErrorBox("error-title", "error-text");
- validation.createPromptBox("prompt-title", "prompt-text");
- sheet.addValidationData(validation);
-
- list = sheet.getDataValidations(); // <-- works
- assertEquals(1, list.size());
-
- HSSFDataValidation dv = list.get(0);
- {
- CellRangeAddressList regions = dv.getRegions();
- assertEquals(1, regions.countRanges());
-
- CellRangeAddress address = regions.getCellRangeAddress(0);
- assertEquals(1, address.getFirstRow());
- assertEquals(2, address.getLastRow());
- assertEquals(3, address.getFirstColumn());
- assertEquals(4, address.getLastColumn());
+ try (HSSFWorkbook wb = new HSSFWorkbook()) {
+ HSSFSheet sheet = wb.createSheet();
+ List<HSSFDataValidation> list = sheet.getDataValidations();
+ assertEquals(0, list.size());
+
+ DataValidationHelper dvh = sheet.getDataValidationHelper();
+ DataValidationConstraint constraint = dvh.createNumericConstraint(ValidationType.ANY, OperatorType.IGNORED, null, null);
+ CellRangeAddressList addressList = new CellRangeAddressList(1, 2, 3, 4);
+ DataValidation validation = dvh.createValidation(constraint, addressList);
+ validation.setEmptyCellAllowed(true);
+ validation.createErrorBox("error-title", "error-text");
+ validation.createPromptBox("prompt-title", "prompt-text");
+ sheet.addValidationData(validation);
+
+ list = sheet.getDataValidations(); // <-- works
+ assertEquals(1, list.size());
+
+ HSSFDataValidation dv = list.get(0);
+ {
+ CellRangeAddressList regions = dv.getRegions();
+ assertEquals(1, regions.countRanges());
+
+ CellRangeAddress address = regions.getCellRangeAddress(0);
+ assertEquals(1, address.getFirstRow());
+ assertEquals(2, address.getLastRow());
+ assertEquals(3, address.getFirstColumn());
+ assertEquals(4, address.getLastColumn());
+ }
+ assertTrue(dv.getEmptyCellAllowed());
+ assertFalse(dv.getSuppressDropDownArrow());
+ assertTrue(dv.getShowErrorBox());
+ assertEquals("error-title", dv.getErrorBoxTitle());
+ assertEquals("error-text", dv.getErrorBoxText());
+ assertTrue(dv.getShowPromptBox());
+ assertEquals("prompt-title", dv.getPromptBoxTitle());
+ assertEquals("prompt-text", dv.getPromptBoxText());
+
+ DataValidationConstraint c = dv.getValidationConstraint();
+ assertEquals(ValidationType.ANY, c.getValidationType());
+ assertEquals(OperatorType.IGNORED, c.getOperator());
}
- assertTrue(dv.getEmptyCellAllowed());
- assertFalse(dv.getSuppressDropDownArrow());
- assertTrue(dv.getShowErrorBox());
- assertEquals("error-title", dv.getErrorBoxTitle());
- assertEquals("error-text", dv.getErrorBoxText());
- assertTrue(dv.getShowPromptBox());
- assertEquals("prompt-title", dv.getPromptBoxTitle());
- assertEquals("prompt-text", dv.getPromptBoxText());
-
- DataValidationConstraint c = dv.getValidationConstraint();
- assertEquals(ValidationType.ANY, c.getValidationType());
- assertEquals(OperatorType.IGNORED, c.getOperator());
-
- wb.close();
}
@Test
public void testGetDataValidationsIntegerFormula() throws Exception {
- HSSFWorkbook wb = new HSSFWorkbook();
- HSSFSheet sheet = wb.createSheet();
- List<HSSFDataValidation> list = sheet.getDataValidations();
- assertEquals(0, list.size());
-
- DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
- DataValidationConstraint constraint = dataValidationHelper.createIntegerConstraint(OperatorType.BETWEEN, "=A2",
- "=A3");
- CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
- DataValidation validation = dataValidationHelper.createValidation(constraint, addressList);
- sheet.addValidationData(validation);
-
- list = sheet.getDataValidations(); // <-- works
- assertEquals(1, list.size());
-
- HSSFDataValidation dv = list.get(0);
- DVConstraint c = dv.getConstraint();
- assertEquals(ValidationType.INTEGER, c.getValidationType());
- assertEquals(OperatorType.BETWEEN, c.getOperator());
- assertEquals("A2", c.getFormula1());
- assertEquals("A3", c.getFormula2());
- assertNull(c.getValue1());
- assertNull(c.getValue2());
-
- wb.close();
+ try (HSSFWorkbook wb = new HSSFWorkbook()) {
+ HSSFSheet sheet = wb.createSheet();
+ List<HSSFDataValidation> list = sheet.getDataValidations();
+ assertEquals(0, list.size());
+
+ DataValidationHelper dvh = sheet.getDataValidationHelper();
+ DataValidationConstraint constraint = dvh.createIntegerConstraint(OperatorType.BETWEEN, "=A2", "=A3");
+ CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
+ DataValidation validation = dvh.createValidation(constraint, addressList);
+ sheet.addValidationData(validation);
+
+ list = sheet.getDataValidations(); // <-- works
+ assertEquals(1, list.size());
+
+ HSSFDataValidation dv = list.get(0);
+ DVConstraint c = dv.getConstraint();
+ assertEquals(ValidationType.INTEGER, c.getValidationType());
+ assertEquals(OperatorType.BETWEEN, c.getOperator());
+ assertEquals("A2", c.getFormula1());
+ assertEquals("A3", c.getFormula2());
+ assertNull(c.getValue1());
+ assertNull(c.getValue2());
+ }
}
@Test
public void testGetDataValidationsIntegerValue() throws Exception {
- HSSFWorkbook wb = new HSSFWorkbook();
- HSSFSheet sheet = wb.createSheet();
- List<HSSFDataValidation> list = sheet.getDataValidations();
- assertEquals(0, list.size());
-
- DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
- DataValidationConstraint constraint = dataValidationHelper.createIntegerConstraint(OperatorType.BETWEEN, "100",
- "200");
- CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
- DataValidation validation = dataValidationHelper.createValidation(constraint, addressList);
- sheet.addValidationData(validation);
-
- list = sheet.getDataValidations(); // <-- works
- assertEquals(1, list.size());
-
- HSSFDataValidation dv = list.get(0);
- DVConstraint c = dv.getConstraint();
- assertEquals(ValidationType.INTEGER, c.getValidationType());
- assertEquals(OperatorType.BETWEEN, c.getOperator());
- assertNull(c.getFormula1());
- assertNull(c.getFormula2());
- assertEquals(new Double("100"), c.getValue1());
- assertEquals(new Double("200"), c.getValue2());
-
- wb.close();
+ try (HSSFWorkbook wb = new HSSFWorkbook()) {
+ HSSFSheet sheet = wb.createSheet();
+ List<HSSFDataValidation> list = sheet.getDataValidations();
+ assertEquals(0, list.size());
+
+ DataValidationHelper dvh = sheet.getDataValidationHelper();
+ DataValidationConstraint constraint = dvh.createIntegerConstraint(OperatorType.BETWEEN, "100", "200");
+ CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
+ DataValidation validation = dvh.createValidation(constraint, addressList);
+ sheet.addValidationData(validation);
+
+ list = sheet.getDataValidations(); // <-- works
+ assertEquals(1, list.size());
+
+ HSSFDataValidation dv = list.get(0);
+ DVConstraint c = dv.getConstraint();
+ assertEquals(ValidationType.INTEGER, c.getValidationType());
+ assertEquals(OperatorType.BETWEEN, c.getOperator());
+ assertNull(c.getFormula1());
+ assertNull(c.getFormula2());
+ assertEquals(100d, c.getValue1(), 0);
+ assertEquals(200d, c.getValue2(), 0);
+ }
}
@Test
public void testGetDataValidationsDecimal() throws Exception {
- HSSFWorkbook wb = new HSSFWorkbook();
- HSSFSheet sheet = wb.createSheet();
- List<HSSFDataValidation> list = sheet.getDataValidations();
- assertEquals(0, list.size());
-
- DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
- DataValidationConstraint constraint = dataValidationHelper.createDecimalConstraint(OperatorType.BETWEEN, "=A2",
- "200");
- CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
- DataValidation validation = dataValidationHelper.createValidation(constraint, addressList);
- sheet.addValidationData(validation);
-
- list = sheet.getDataValidations(); // <-- works
- assertEquals(1, list.size());
-
- HSSFDataValidation dv = list.get(0);
- DVConstraint c = dv.getConstraint();
- assertEquals(ValidationType.DECIMAL, c.getValidationType());
- assertEquals(OperatorType.BETWEEN, c.getOperator());
- assertEquals("A2", c.getFormula1());
- assertNull(c.getFormula2());
- assertNull(c.getValue1());
- assertEquals(new Double("200"), c.getValue2());
-
- wb.close();
+ try (HSSFWorkbook wb = new HSSFWorkbook()) {
+ HSSFSheet sheet = wb.createSheet();
+ List<HSSFDataValidation> list = sheet.getDataValidations();
+ assertEquals(0, list.size());
+
+ DataValidationHelper dvh = sheet.getDataValidationHelper();
+ DataValidationConstraint constraint = dvh.createDecimalConstraint(OperatorType.BETWEEN, "=A2", "200");
+ CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
+ DataValidation validation = dvh.createValidation(constraint, addressList);
+ sheet.addValidationData(validation);
+
+ list = sheet.getDataValidations(); // <-- works
+ assertEquals(1, list.size());
+
+ HSSFDataValidation dv = list.get(0);
+ DVConstraint c = dv.getConstraint();
+ assertEquals(ValidationType.DECIMAL, c.getValidationType());
+ assertEquals(OperatorType.BETWEEN, c.getOperator());
+ assertEquals("A2", c.getFormula1());
+ assertNull(c.getFormula2());
+ assertNull(c.getValue1());
+ assertEquals(200, c.getValue2(), 0);
+ }
}
@Test
public void testGetDataValidationsDate() throws Exception {
- HSSFWorkbook wb = new HSSFWorkbook();
- HSSFSheet sheet = wb.createSheet();
- List<HSSFDataValidation> list = sheet.getDataValidations();
- assertEquals(0, list.size());
-
- DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
- DataValidationConstraint constraint = dataValidationHelper.createDateConstraint(OperatorType.EQUAL,
- "2014/10/25", null, null);
- CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
- DataValidation validation = dataValidationHelper.createValidation(constraint, addressList);
- sheet.addValidationData(validation);
-
- list = sheet.getDataValidations(); // <-- works
- assertEquals(1, list.size());
-
- HSSFDataValidation dv = list.get(0);
- DVConstraint c = dv.getConstraint();
- assertEquals(ValidationType.DATE, c.getValidationType());
- assertEquals(OperatorType.EQUAL, c.getOperator());
- assertNull(c.getFormula1());
- assertNull(c.getFormula2());
- assertEquals(DateUtil.getExcelDate(DateUtil.parseYYYYMMDDDate("2014/10/25")), c.getValue1(), 0);
- assertNull(c.getValue2());
-
- wb.close();
+ try (HSSFWorkbook wb = new HSSFWorkbook()) {
+ HSSFSheet sheet = wb.createSheet();
+ List<HSSFDataValidation> list = sheet.getDataValidations();
+ assertEquals(0, list.size());
+
+ DataValidationHelper dvh = sheet.getDataValidationHelper();
+ DataValidationConstraint constraint = dvh.createDateConstraint(OperatorType.EQUAL, "2014/10/25", null, null);
+ CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
+ DataValidation validation = dvh.createValidation(constraint, addressList);
+ sheet.addValidationData(validation);
+
+ list = sheet.getDataValidations(); // <-- works
+ assertEquals(1, list.size());
+
+ HSSFDataValidation dv = list.get(0);
+ DVConstraint c = dv.getConstraint();
+ assertEquals(ValidationType.DATE, c.getValidationType());
+ assertEquals(OperatorType.EQUAL, c.getOperator());
+ assertNull(c.getFormula1());
+ assertNull(c.getFormula2());
+ assertEquals(DateUtil.getExcelDate(DateUtil.parseYYYYMMDDDate("2014/10/25")), c.getValue1(), 0);
+ assertNull(c.getValue2());
+ }
}
@Test
public void testGetDataValidationsListExplicit() throws Exception {
- HSSFWorkbook wb = new HSSFWorkbook();
- HSSFSheet sheet = wb.createSheet();
- List<HSSFDataValidation> list = sheet.getDataValidations();
- assertEquals(0, list.size());
-
- DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
- DataValidationConstraint constraint = dataValidationHelper.createExplicitListConstraint(new String[] { "aaa",
- "bbb", "ccc" });
- CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
- DataValidation validation = dataValidationHelper.createValidation(constraint, addressList);
- validation.setSuppressDropDownArrow(true);
- sheet.addValidationData(validation);
-
- list = sheet.getDataValidations(); // <-- works
- assertEquals(1, list.size());
-
- HSSFDataValidation dv = list.get(0);
- assertTrue(dv.getSuppressDropDownArrow());
-
- DVConstraint c = dv.getConstraint();
- assertEquals(ValidationType.LIST, c.getValidationType());
- assertNull(c.getFormula1());
- assertNull(c.getFormula2());
- assertNull(c.getValue1());
- assertNull(c.getValue2());
- String[] values = c.getExplicitListValues();
- assertEquals(3, values.length);
- assertEquals("aaa", values[0]);
- assertEquals("bbb", values[1]);
- assertEquals("ccc", values[2]);
-
- wb.close();
+ try (HSSFWorkbook wb = new HSSFWorkbook()) {
+ HSSFSheet sheet = wb.createSheet();
+ List<HSSFDataValidation> list = sheet.getDataValidations();
+ assertEquals(0, list.size());
+
+ DataValidationHelper dvh = sheet.getDataValidationHelper();
+ DataValidationConstraint constraint = dvh.createExplicitListConstraint(new String[]{"aaa", "bbb", "ccc"});
+ CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
+ DataValidation validation = dvh.createValidation(constraint, addressList);
+ validation.setSuppressDropDownArrow(true);
+ sheet.addValidationData(validation);
+
+ list = sheet.getDataValidations(); // <-- works
+ assertEquals(1, list.size());
+
+ HSSFDataValidation dv = list.get(0);
+ assertTrue(dv.getSuppressDropDownArrow());
+
+ DVConstraint c = dv.getConstraint();
+ assertEquals(ValidationType.LIST, c.getValidationType());
+ assertNull(c.getFormula1());
+ assertNull(c.getFormula2());
+ assertNull(c.getValue1());
+ assertNull(c.getValue2());
+ String[] values = c.getExplicitListValues();
+ assertEquals(3, values.length);
+ assertEquals("aaa", values[0]);
+ assertEquals("bbb", values[1]);
+ assertEquals("ccc", values[2]);
+ }
}
@Test
public void testGetDataValidationsListFormula() throws Exception {
- HSSFWorkbook wb = new HSSFWorkbook();
- HSSFSheet sheet = wb.createSheet();
- List<HSSFDataValidation> list = sheet.getDataValidations();
- assertEquals(0, list.size());
-
- DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
- DataValidationConstraint constraint = dataValidationHelper.createFormulaListConstraint("A2");
- CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
- DataValidation validation = dataValidationHelper.createValidation(constraint, addressList);
- validation.setSuppressDropDownArrow(true);
- sheet.addValidationData(validation);
-
- list = sheet.getDataValidations(); // <-- works
- assertEquals(1, list.size());
-
- HSSFDataValidation dv = list.get(0);
- assertTrue(dv.getSuppressDropDownArrow());
-
- DVConstraint c = dv.getConstraint();
- assertEquals(ValidationType.LIST, c.getValidationType());
- assertEquals("A2", c.getFormula1());
- assertNull(c.getFormula2());
- assertNull(c.getValue1());
- assertNull(c.getValue2());
-
- wb.close();
+ try (HSSFWorkbook wb = new HSSFWorkbook()) {
+ HSSFSheet sheet = wb.createSheet();
+ List<HSSFDataValidation> list = sheet.getDataValidations();
+ assertEquals(0, list.size());
+
+ DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
+ DataValidationConstraint constraint = dataValidationHelper.createFormulaListConstraint("A2");
+ CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
+ DataValidation validation = dataValidationHelper.createValidation(constraint, addressList);
+ validation.setSuppressDropDownArrow(true);
+ sheet.addValidationData(validation);
+
+ list = sheet.getDataValidations(); // <-- works
+ assertEquals(1, list.size());
+
+ HSSFDataValidation dv = list.get(0);
+ assertTrue(dv.getSuppressDropDownArrow());
+
+ DVConstraint c = dv.getConstraint();
+ assertEquals(ValidationType.LIST, c.getValidationType());
+ assertEquals("A2", c.getFormula1());
+ assertNull(c.getFormula2());
+ assertNull(c.getValue1());
+ assertNull(c.getValue2());
+ }
}
@Test
public void testGetDataValidationsFormula() throws Exception {
- HSSFWorkbook wb = new HSSFWorkbook();
- HSSFSheet sheet = wb.createSheet();
- List<HSSFDataValidation> list = sheet.getDataValidations();
- assertEquals(0, list.size());
-
- DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
- DataValidationConstraint constraint = dataValidationHelper.createCustomConstraint("A2:A3");
- CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
- DataValidation validation = dataValidationHelper.createValidation(constraint, addressList);
- sheet.addValidationData(validation);
-
- list = sheet.getDataValidations(); // <-- works
- assertEquals(1, list.size());
-
- HSSFDataValidation dv = list.get(0);
- DVConstraint c = dv.getConstraint();
- assertEquals(ValidationType.FORMULA, c.getValidationType());
- assertEquals("A2:A3", c.getFormula1());
- assertNull(c.getFormula2());
- assertNull(c.getValue1());
- assertNull(c.getValue2());
- wb.close();
+ try (HSSFWorkbook wb = new HSSFWorkbook()) {
+ HSSFSheet sheet = wb.createSheet();
+ List<HSSFDataValidation> list = sheet.getDataValidations();
+ assertEquals(0, list.size());
+
+ DataValidationHelper dataValidationHelper = sheet.getDataValidationHelper();
+ DataValidationConstraint constraint = dataValidationHelper.createCustomConstraint("A2:A3");
+ CellRangeAddressList addressList = new CellRangeAddressList(0, 0, 0, 0);
+ DataValidation validation = dataValidationHelper.createValidation(constraint, addressList);
+ sheet.addValidationData(validation);
+
+ list = sheet.getDataValidations(); // <-- works
+ assertEquals(1, list.size());
+
+ HSSFDataValidation dv = list.get(0);
+ DVConstraint c = dv.getConstraint();
+ assertEquals(ValidationType.FORMULA, c.getValidationType());
+ assertEquals("A2:A3", c.getFormula1());
+ assertNull(c.getFormula2());
+ assertNull(c.getValue1());
+ assertNull(c.getValue2());
+ }
}
}
diff --git a/src/testcases/org/apache/poi/ss/formula/constant/TestConstantValueParser.java b/src/testcases/org/apache/poi/ss/formula/constant/TestConstantValueParser.java
index 08ce2d96e4..72a9bcc0d8 100644
--- a/src/testcases/org/apache/poi/ss/formula/constant/TestConstantValueParser.java
+++ b/src/testcases/org/apache/poi/ss/formula/constant/TestConstantValueParser.java
@@ -34,7 +34,7 @@ public final class TestConstantValueParser {
private static final Object[] SAMPLE_VALUES = {
Boolean.TRUE,
null,
- new Double(1.1),
+ 1.1,
"Sample text",
ErrorConstant.valueOf(FormulaError.DIV0.getCode()),
};
@@ -44,29 +44,29 @@ public final class TestConstantValueParser {
"01 9A 99 99 99 99 99 F1 3F " +
"02 0B 00 00 53 61 6D 70 6C 65 20 74 65 78 74 " +
"10 07 00 00 00 00 00 00 00");
-
+
@Test
public void testGetEncodedSize() {
int actual = ConstantValueParser.getEncodedSize(SAMPLE_VALUES);
assertEquals(51, actual);
}
-
+
@Test
public void testEncode() {
int size = ConstantValueParser.getEncodedSize(SAMPLE_VALUES);
byte[] data = new byte[size];
-
+
ConstantValueParser.encode(new LittleEndianByteArrayOutputStream(data, 0), SAMPLE_VALUES);
-
+
if (!Arrays.equals(data, SAMPLE_ENCODING)) {
fail("Encoding differs");
}
}
-
+
@Test
public void testDecode() {
LittleEndianInput in = TestcaseRecordInputStream.createLittleEndian(SAMPLE_ENCODING);
-
+
Object[] values = ConstantValueParser.parse(in, 4);
for (int i = 0; i < values.length; i++) {
if(!isEqual(SAMPLE_VALUES[i], values[i])) {
diff --git a/src/testcases/org/apache/poi/ss/formula/function/ExcelFileFormatDocFunctionExtractor.java b/src/testcases/org/apache/poi/ss/formula/function/ExcelFileFormatDocFunctionExtractor.java
index ac2551665d..c408dd527b 100644
--- a/src/testcases/org/apache/poi/ss/formula/function/ExcelFileFormatDocFunctionExtractor.java
+++ b/src/testcases/org/apache/poi/ss/formula/function/ExcelFileFormatDocFunctionExtractor.java
@@ -39,19 +39,20 @@ import java.util.List;
import java.util.Map;
import java.util.Set;
import java.util.Stack;
-import java.util.zip.ZipException;
import java.util.zip.ZipFile;
+import javax.xml.parsers.ParserConfigurationException;
+import javax.xml.parsers.SAXParser;
+import javax.xml.parsers.SAXParserFactory;
+
import org.apache.poi.poifs.crypt.CryptoFunctions;
import org.apache.poi.poifs.crypt.HashAlgorithm;
import org.apache.poi.util.TempFile;
+import org.apache.poi.util.XMLHelper;
import org.xml.sax.Attributes;
-import org.xml.sax.ContentHandler;
-import org.xml.sax.InputSource;
import org.xml.sax.Locator;
import org.xml.sax.SAXException;
-import org.xml.sax.XMLReader;
-import org.xml.sax.helpers.XMLReaderFactory;
+import org.xml.sax.helpers.DefaultHandler;
/**
* This class is not used during normal POI run-time but is used at development time to generate
@@ -229,7 +230,7 @@ public final class ExcelFileFormatDocFunctionExtractor {
/**
* To avoid drag-in - parse XML using only JDK.
*/
- private static class EFFDocHandler implements ContentHandler {
+ private static class EFFDocHandler extends DefaultHandler {
private static final String[] HEADING_PATH_NAMES = {
"office:document-content", "office:body", "office:text", "text:h",
};
@@ -339,7 +340,7 @@ public final class ExcelFileFormatDocFunctionExtractor {
processFunction(cellData, noteFlags, 0);
processFunction(cellData, noteFlags, 8);
}
-
+
public void processFunction(String[] cellData, Boolean[] noteFlags, int i) {
String funcIxStr = cellData[i + 0];
if (funcIxStr.length() < 1) {
@@ -428,27 +429,18 @@ public final class ExcelFileFormatDocFunctionExtractor {
}
private static void extractFunctionData(FunctionDataCollector fdc, InputStream is) {
- XMLReader xr;
+ SAXParserFactory sf = XMLHelper.getSaxParserFactory();
+ SAXParser xr;
try {
// First up, try the default one
- xr = XMLReaderFactory.createXMLReader();
- } catch (SAXException e) {
- // Try one for java 1.4
- System.setProperty("org.xml.sax.driver", "org.apache.crimson.parser.XMLReaderImpl");
- try {
- xr = XMLReaderFactory.createXMLReader();
- } catch (SAXException e2) {
- throw new RuntimeException(e2);
- }
+ xr = sf.newSAXParser();
+ } catch (SAXException | ParserConfigurationException e) {
+ throw new RuntimeException(e);
}
- xr.setContentHandler(new EFFDocHandler(fdc));
- InputSource inSrc = new InputSource(is);
-
- try {
- xr.parse(inSrc);
- is.close();
+ try (InputStream is2 = is) {
+ xr.parse(is2, new EFFDocHandler(fdc));
} catch (IOException | SAXException e) {
throw new RuntimeException(e);
}
@@ -463,7 +455,7 @@ public final class ExcelFileFormatDocFunctionExtractor {
public SimpleAsciiOutputStream(OutputStream os) {
_os = os;
}
-
+
@Override
public void write(int b) throws IOException {
checkByte(b);
diff --git a/src/testcases/org/apache/poi/ss/formula/functions/TestStatsLib.java b/src/testcases/org/apache/poi/ss/formula/functions/TestStatsLib.java
index bb8b8e62bb..9c4fdcb2f2 100644
--- a/src/testcases/org/apache/poi/ss/formula/functions/TestStatsLib.java
+++ b/src/testcases/org/apache/poi/ss/formula/functions/TestStatsLib.java
@@ -220,7 +220,7 @@ public class TestStatsLib extends BaseTestNumeric {
}
private static void confirmMode(double[] v, double expectedResult) {
- confirmMode(v, new Double(expectedResult));
+ confirmMode(v, (Double)expectedResult);
}
private static void confirmMode(double[] v, Double expectedResult) {
diff --git a/src/testcases/org/apache/poi/ss/formula/ptg/TestArrayPtg.java b/src/testcases/org/apache/poi/ss/formula/ptg/TestArrayPtg.java
index 960209681b..170c06719b 100644
--- a/src/testcases/org/apache/poi/ss/formula/ptg/TestArrayPtg.java
+++ b/src/testcases/org/apache/poi/ss/formula/ptg/TestArrayPtg.java
@@ -67,7 +67,7 @@ public final class TestArrayPtg {
assertEquals(Boolean.TRUE, values[0][0]);
assertEquals("ABCD", values[0][1]);
- assertEquals(new Double(0), values[1][0]);
+ assertEquals(0d, values[1][0]);
assertEquals(Boolean.FALSE, values[1][1]);
assertEquals("FG", values[1][2]);