Don't use a hardcoded temporary file (and also not a hardcoded /tmp directory)
This could cause issues in CI or when multiple different users run tests of POI on one machine!
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1887998 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/ooxml/testcases/org/apache/poi/xddf/usermodel/chart/TestXDDFChart.java b/src/ooxml/testcases/org/apache/poi/xddf/usermodel/chart/TestXDDFChart.java
index 66196d3..45ff5d7 100644
--- a/src/ooxml/testcases/org/apache/poi/xddf/usermodel/chart/TestXDDFChart.java
+++ b/src/ooxml/testcases/org/apache/poi/xddf/usermodel/chart/TestXDDFChart.java
@@ -20,16 +20,19 @@
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
import org.apache.poi.ooxml.POIXMLFactory;
import org.apache.poi.ooxml.POIXMLRelation;
import org.apache.poi.ss.util.CellRangeAddress;
+import org.apache.poi.util.TempFile;
import org.apache.poi.xssf.XSSFTestDataSamples;
import org.apache.poi.xssf.usermodel.XSSFSheet;
import org.apache.poi.xssf.usermodel.XSSFWorkbook;
import org.junit.jupiter.api.Test;
import org.openxmlformats.schemas.drawingml.x2006.chart.CTChartSpace;
+import java.io.File;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.OutputStream;
@@ -83,14 +86,19 @@
chart.plot(data);
- try (OutputStream out = new FileOutputStream("/tmp/chart20201220.xlsx")) {
- wb.write(out);
+ File file = TempFile.createTempFile("chart20201220", ".xlsx");
+ try {
+ try (OutputStream out = new FileOutputStream(file)) {
+ wb.write(out);
+ }
+ } finally {
+ assertTrue(!file.exists() || file.delete());
}
}
}
private XDDFChart newXDDFChart() {
- XDDFChart xddfChart = new XDDFChart() {
+ return new XDDFChart() {
@Override
protected POIXMLRelation getChartRelation() {
return null;
@@ -106,6 +114,5 @@
return null;
}
};
- return xddfChart;
}
}
