Author: onealj
Date: Tue Jun 20 07:05:08 2017
New Revision: 1799307

URL: http://svn.apache.org/viewvc?rev=1799307&view=rev
Log:
rename createXls and createXlsx helper functions: these are specifically for 
testing text rotation.
Made the file creation compatible on Windows and Mac/Linux using POI TempFile 
instead of a hard-coded filename.

Modified:
    
poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java

Modified: 
poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java?rev=1799307&r1=1799306&r2=1799307&view=diff
==============================================================================
--- 
poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java 
(original)
+++ 
poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java 
Tue Jun 20 07:05:08 2017
@@ -2885,51 +2885,37 @@ public final class TestXSSFBugs extends
         wb.close();
     }
 
-    private void createXls() throws IOException {
-        Workbook workbook = new HSSFWorkbook();
-        FileOutputStream fileOut = new FileOutputStream("/tmp/rotated.xls");
-        Sheet sheet1 = workbook.createSheet();
-        Row row1 = sheet1.createRow((short) 0);
+    /**
+     * helper function for {@link #test58043()}
+     * Side-effects: closes the provided workbook!
+     *
+     * @param workbook the workbook to save for manual checking
+     * @param outputFile the output file location to save the workbook to
+     */
+    private void saveRotatedTextExample(Workbook workbook, File outputFile) 
throws IOException {
+        Sheet sheet = workbook.createSheet();
+        Row row = sheet.createRow((short) 0);
 
-        Cell cell1 = row1.createCell(0);
+        Cell cell = row.createCell(0);
 
-        cell1.setCellValue("Successful rotated text.");
+        cell.setCellValue("Unsuccessful rotated text.");
 
         CellStyle style = workbook.createCellStyle();
         style.setRotation((short) -90);
 
-        cell1.setCellStyle(style);
+        cell.setCellStyle(style);
 
-        workbook.write(fileOut);
-        fileOut.close();
-        workbook.close();
-    }
-
-    private void createXlsx() throws IOException {
-        Workbook workbook = new XSSFWorkbook();
-        FileOutputStream fileOut = new FileOutputStream("/tmp/rotated.xlsx");
-        Sheet sheet1 = workbook.createSheet();
-        Row row1 = sheet1.createRow((short) 0);
-
-        Cell cell1 = row1.createCell(0);
-
-        cell1.setCellValue("Unsuccessful rotated text.");
-
-        CellStyle style = workbook.createCellStyle();
-        style.setRotation((short) -90);
-
-        cell1.setCellStyle(style);
-
-        workbook.write(fileOut);
-        fileOut.close();
+        OutputStream fos = new FileOutputStream(outputFile);
+        workbook.write(fos);
+        fos.close();
         workbook.close();
     }
 
     @Ignore("Creates files for checking results manually, actual values are 
tested in Test*CellStyle")
     @Test
     public void test58043() throws IOException {
-        createXls();
-        createXlsx();
+        saveRotatedTextExample(new HSSFWorkbook(), 
TempFile.createTempFile("rotated", ".xls"));
+        saveRotatedTextExample(new XSSFWorkbook(), 
TempFile.createTempFile("rotated", ".xlsx"));
     }
 
     @Test



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to