diff --git a/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java b/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java index a4bd4a5cf..6df39bbe2 100644 --- a/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java +++ b/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java @@ -2493,7 +2493,7 @@ public final class TestBugs extends BaseTestBugzillaIssues { // How close the sizing should be, given that not all // systems will have quite the same fonts on them - int fontAccuracy = 25; + float fontAccuracy = 0.15f; // x% CellStyle iPercent = wb.createCellStyle(); @@ -2527,7 +2527,6 @@ public final class TestBugs extends BaseTestBugzillaIssues { } for (int i=0; i<12; i++) { s.autoSizeColumn(i); - System.out.println(i + " => " + s.getColumnWidth(i)); } // Check the 0(.00)% ones diff --git a/src/testcases/org/apache/poi/ss/usermodel/BaseTestBugzillaIssues.java b/src/testcases/org/apache/poi/ss/usermodel/BaseTestBugzillaIssues.java index 9083b7048..bc80b64fa 100644 --- a/src/testcases/org/apache/poi/ss/usermodel/BaseTestBugzillaIssues.java +++ b/src/testcases/org/apache/poi/ss/usermodel/BaseTestBugzillaIssues.java @@ -40,8 +40,9 @@ public abstract class BaseTestBugzillaIssues { _testDataProvider = testDataProvider; } - public static void assertAlmostEquals(double expected, double actual, double fuzz) { + public static void assertAlmostEquals(double expected, double actual, float factor) { double diff = Math.abs(expected - actual); + double fuzz = expected * factor; if (diff > fuzz) fail(actual + " not within " + fuzz + " of " + expected); }