From 01ecea2df0bede3fbcb61429b8decffd2a589064 Mon Sep 17 00:00:00 2001 From: Peter Edberg Date: Tue, 5 Feb 2019 23:29:46 -0800 Subject: [PATCH] ICU-20347 Update ICU4J test to reflect expected behavior --- .../com/ibm/icu/dev/test/format/NumberFormatTest.java | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/icu4j/main/tests/core/src/com/ibm/icu/dev/test/format/NumberFormatTest.java b/icu4j/main/tests/core/src/com/ibm/icu/dev/test/format/NumberFormatTest.java index 2079bdebdf5..bb951e8a000 100644 --- a/icu4j/main/tests/core/src/com/ibm/icu/dev/test/format/NumberFormatTest.java +++ b/icu4j/main/tests/core/src/com/ibm/icu/dev/test/format/NumberFormatTest.java @@ -1738,10 +1738,13 @@ public class NumberFormatTest extends TestFmwk { Number value = null; try { value = numfmt.parse(parsetxt, ppos); - // Currently this succeeds (no exception) but returns null (for value). - logln("NumberFormat.parse empty string succeeds, ppos " + ppos.getIndex() + ", value " + value); + if (value==null) { + logln("NumberFormat.parse empty string succeeds (no exception) with null return as expected, ppos " + ppos.getIndex()); + } else { + errln("NumberFormat.parse empty string succeeds (no exception) but returns non-null value " + value + ", ppos " + ppos.getIndex()); + } } catch (IllegalArgumentException e){ - logln("NumberFormat.parse empty string sets IllegalArgumentException, ppos " + ppos.getIndex() + ", value " + value); + errln("NumberFormat.parse empty string throws IllegalArgumentException"); } }