diff --git a/build.xml b/build.xml index 9ffdaf88e..883db323f 100644 --- a/build.xml +++ b/build.xml @@ -70,8 +70,7 @@ under the License. JVM system properties for running tests, user.language and user.country are required as we have locale-sensitive formatters --> - - + @@ -159,8 +158,6 @@ under the License. - - @@ -572,6 +569,7 @@ under the License. failureproperty="main.test.failed" showoutput="true"> + @@ -608,6 +606,7 @@ under the License. failureproperty="scratchpad.test.failed"> + @@ -642,6 +641,7 @@ under the License. + @@ -672,6 +672,7 @@ under the License. failureproperty="ooxml.test.failed"> + @@ -697,6 +698,7 @@ under the License. + diff --git a/src/testcases/org/apache/poi/ss/format/CellFormatTestBase.java b/src/testcases/org/apache/poi/ss/format/CellFormatTestBase.java index b8d692512..28e9cfe46 100644 --- a/src/testcases/org/apache/poi/ss/format/CellFormatTestBase.java +++ b/src/testcases/org/apache/poi/ss/format/CellFormatTestBase.java @@ -43,6 +43,8 @@ import org.apache.poi.ss.usermodel.Cell; import org.apache.poi.ss.usermodel.Row; import org.apache.poi.ss.usermodel.Sheet; import org.apache.poi.ss.usermodel.Workbook; +import org.apache.poi.util.POILogger; +import org.apache.poi.util.POILogFactory; /** * This class is a base class for spreadsheet-based tests, such as are used for @@ -58,6 +60,8 @@ import org.apache.poi.ss.usermodel.Workbook; @SuppressWarnings( {"JUnitTestCaseWithNoTests", "JUnitTestClassNamingConvention"}) public class CellFormatTestBase extends TestCase { + private static final POILogger logger = POILogFactory.getLogger(CellFormatTestBase.class); + private final ITestDataProvider _testDataProvider; protected Workbook workbook; @@ -212,13 +216,11 @@ public class CellFormatTestBase extends TestCase { label.setForeground(testColor); label.setText("xyzzy"); - System.out.printf("Row %d: \"%s\" -> \"%s\": expected \"%s\"", row + 1, - String.valueOf(value), desc, expectedText); - System.out.flush(); + logger.log(POILogger.INFO, String.format("Row %d: \"%s\" -> \"%s\": expected \"%s\"", row + 1, + String.valueOf(value), desc, expectedText)); String actualText = tryColor(desc, null, getter, value, expectedText, testColor); - System.out.printf(", actual \"%s\")%n", actualText); - System.out.flush(); + logger.log(POILogger.INFO, String.format(", actual \"%s\")%n", actualText)); if (tryAllColors && testColor != TEST_COLOR) { for (int i = 0; i < COLOR_NAMES.length; i++) {