From b2206fc40f82bcaa2a1cbb4b463544646a3b3628 Mon Sep 17 00:00:00 2001 From: Javen O'Neal Date: Fri, 17 Feb 2017 16:57:43 +0000 Subject: [PATCH] bug 60742: write out to unit test TempFile rather than Unix-only /tmp/60512.xlsx; patch from Chiara Marcheschi git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1783426 13f79535-47bb-0310-9956-ffa450edef68 --- .../org/apache/poi/xssf/TestXSSFCloneSheet.java | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/ooxml/testcases/org/apache/poi/xssf/TestXSSFCloneSheet.java b/src/ooxml/testcases/org/apache/poi/xssf/TestXSSFCloneSheet.java index 1c3cfe25d..c2c2375e7 100644 --- a/src/ooxml/testcases/org/apache/poi/xssf/TestXSSFCloneSheet.java +++ b/src/ooxml/testcases/org/apache/poi/xssf/TestXSSFCloneSheet.java @@ -25,11 +25,13 @@ import org.apache.poi.hssf.HSSFITestDataProvider; import org.apache.poi.ss.usermodel.BaseTestCloneSheet; import org.apache.poi.ss.usermodel.Sheet; import org.apache.poi.ss.usermodel.Workbook; +import org.apache.poi.util.TempFile; import org.apache.poi.xssf.usermodel.XSSFSheet; import org.apache.poi.xssf.usermodel.XSSFWorkbook; import org.junit.Before; import org.junit.Test; +import java.io.File; import java.io.FileOutputStream; import java.io.IOException; import java.io.OutputStream; @@ -81,11 +83,13 @@ public class TestXSSFCloneSheet extends BaseTestCloneSheet { assertNotNull(wbBack); wbBack.close(); - OutputStream str = new FileOutputStream("/tmp/60512.xlsx"); + // FIXME: why are we doing both writeOutAndReadBack and write? Was the temp file created for manual debugging? + File tmp = TempFile.createTempFile("60512.xlsx", ".tmp"); + OutputStream ostream = new FileOutputStream(tmp); try { - wb.write(str); + wb.write(ostream); } finally { - str.close(); + ostream.close(); } wb.close();