From: Peter Edberg <pedberg@unicode.org>
Date: Wed, 6 Feb 2019 07:29:46 +0000 (-0800)
Subject: ICU-20347 Update ICU4J test to reflect expected behavior
X-Git-Tag: release-64-rc~129
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=01ecea2df0bede3fbcb61429b8decffd2a589064;p=icu

ICU-20347 Update ICU4J test to reflect expected behavior
---

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");
         }
      }