Author: centic
Date: Sun Jun 2 09:39:43 2024
New Revision: 1918115
URL: http://svn.apache.org/viewvc?rev=1918115&view=rev
Log:
Add simple test for font-width
Modified:
poi/trunk/poi/src/test/java/org/apache/poi/ss/util/TestSheetUtil.java
Modified: poi/trunk/poi/src/test/java/org/apache/poi/ss/util/TestSheetUtil.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi/src/test/java/org/apache/poi/ss/util/TestSheetUtil.java?rev=1918115&r1=1918114&r2=1918115&view=diff
==============================================================================
--- poi/trunk/poi/src/test/java/org/apache/poi/ss/util/TestSheetUtil.java
(original)
+++ poi/trunk/poi/src/test/java/org/apache/poi/ss/util/TestSheetUtil.java Sun
Jun 2 09:39:43 2024
@@ -32,6 +32,7 @@ import java.io.IOException;
import org.apache.poi.hssf.usermodel.HSSFWorkbook;
import org.apache.poi.ss.usermodel.Cell;
import org.apache.poi.ss.usermodel.CellType;
+import org.apache.poi.ss.usermodel.Font;
import org.apache.poi.ss.usermodel.Row;
import org.apache.poi.ss.usermodel.Sheet;
import org.apache.poi.ss.usermodel.Workbook;
@@ -114,6 +115,7 @@ final class TestSheetUtil {
Cell cell = row.createCell(0);
// no contents: cell.setCellValue("sometext");
+ //noinspection deprecation
assertEquals(-1.0, SheetUtil.getCellWidth(cell, 1, null, true),
0.01);
assertEquals(-1.0, SheetUtil.getCellWidth(cell, 1.0f, null, true),
0.01);
@@ -132,6 +134,7 @@ final class TestSheetUtil {
final double width = SheetUtil.getCellWidth(cell, 1.0f, null,
true);
assertTrue(width > 0);
+ //noinspection deprecation
assertEquals(width, SheetUtil.getCellWidth(cell, 1, null, true));
}
}
@@ -145,7 +148,9 @@ final class TestSheetUtil {
cell.setCellValue(88.234);
+ //noinspection deprecation
assertTrue(SheetUtil.getCellWidth(cell, 1, null, true) > 0);
+ assertTrue(SheetUtil.getCellWidth(cell, 1.0f, null, true) > 0);
}
}
@@ -158,7 +163,9 @@ final class TestSheetUtil {
cell.setCellValue(false);
+ //noinspection deprecation
assertTrue(SheetUtil.getCellWidth(cell, 1, null, false) > 0);
+ assertTrue(SheetUtil.getCellWidth(cell, 1.0f, null, false) > 0);
}
}
@@ -255,11 +262,22 @@ final class TestSheetUtil {
// verify that a call with an unknown font-name returns a useful value
// (likely the font-system falls back to a default font here)
// (may fail if font-system is missing, but then many other tests fail
as well)
-
try (Workbook wb = new HSSFWorkbook()) {
wb.getFontAt(0).setFontName("invalid font");
- final float width = SheetUtil.getDefaultCharWidthAsFloat(wb);
+ float width = SheetUtil.getDefaultCharWidthAsFloat(wb);
+ assertTrue(width > 0,
+ "Should get some useful char width, but had: " + width);
+
+ wb.getFontAt(0).setFontName("");
+
+ width = SheetUtil.getDefaultCharWidthAsFloat(wb);
+ assertTrue(width > 0,
+ "Should get some useful char width, but had: " + width);
+
+ wb.getFontAt(0).setFontName(null);
+
+ width = SheetUtil.getDefaultCharWidthAsFloat(wb);
assertTrue(width > 0,
"Should get some useful char width, but had: " + width);
}
@@ -386,4 +404,34 @@ final class TestSheetUtil {
SheetUtil.setIgnoreMissingFontSystem(previous);
}
}
+
+ @Test
+ void testGetDefaultCharWidthAsFloat() throws IOException {
+ try (Workbook wb = new HSSFWorkbook()) {
+ float width = SheetUtil.getDefaultCharWidthAsFloat(new
HSSFWorkbook());
+ assertTrue(width > 0,
+ "Expected a non-zero and positive font width");
+
+ //Font font = wb.createFont();
+ Font font = wb.getFontAt(0);
+ assertNotNull(font);
+
+ // verify that the system falls back to some default font for
unknown/incorrect font-families
+
+ font.setFontName("not-existing");
+ float range = SheetUtil.getDefaultCharWidthAsFloat(wb);
+ assertTrue(range > 5.5);
+ assertTrue(range < 6.7);
+
+ font.setFontName("");
+ range = SheetUtil.getDefaultCharWidthAsFloat(wb);
+ assertTrue(range > 5.5);
+ assertTrue(range < 6.7);
+
+ font.setFontName(null);
+ range = SheetUtil.getDefaultCharWidthAsFloat(wb);
+ assertTrue(range > 5.5);
+ assertTrue(range < 6.7);
+ }
+ }
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
