ticket:11674: Make ULocale tests work better with Android.

ticket:11674: Make ULocale tests work better with Android.

ULocale has been updated to work with scripts and Unicode extensions in
the Android Locale, so the tests can be updated to expect this to behave
essentially like Java 7.

R=markus.icu@gmail.com

Committed: http://bugs.icu-project.org/trac/changeset/37432

[markus.icu, 2015-05-12 18:05:15]

otherwise LGTM

https://codereview.appspot.com/235430043/diff/1/main/tests/core/src/com/ibm/i...
File main/tests/core/src/com/ibm/icu/dev/test/format/IntlTestDateFormatAPI.java
(right):

https://codereview.appspot.com/235430043/diff/1/main/tests/core/src/com/ibm/i...
main/tests/core/src/com/ibm/icu/dev/test/format/IntlTestDateFormatAPI.java:26:
import com.ibm.icu.dev.test.TestUtil.JavaVendor;
see comment next file

https://codereview.appspot.com/235430043/diff/1/main/tests/core/src/com/ibm/i...
File main/tests/core/src/com/ibm/icu/dev/test/util/ULocaleTest.java (right):

https://codereview.appspot.com/235430043/diff/1/main/tests/core/src/com/ibm/i...
main/tests/core/src/com/ibm/icu/dev/test/util/ULocaleTest.java:27: import
com.ibm.icu.dev.test.TestUtil.JavaVendor;
You should not need to import this, because you qualify the constant as
JavaVendor.Android, right?

[roubert (google), 2015-05-12 18:09:29]

https://codereview.appspot.com/235430043/diff/1/main/tests/core/src/com/ibm/i...
File main/tests/core/src/com/ibm/icu/dev/test/util/ULocaleTest.java (right):

https://codereview.appspot.com/235430043/diff/1/main/tests/core/src/com/ibm/i...
main/tests/core/src/com/ibm/icu/dev/test/util/ULocaleTest.java:27: import
com.ibm.icu.dev.test.TestUtil.JavaVendor;
On 2015/05/12 18:05:15, markus.icu wrote:
> You should not need to import this, because you qualify the constant as
> JavaVendor.Android, right?

I wonder what you're thinking about here. If I just remove this import
statement, compilation will fail like this:

ULocaleTest.java:50: error: cannot find symbol
symbol:   variable JavaVendor
location: class ULocaleTest

Do you mean that you'd prefer to use TestUtil.JavaVendor.Android instead of this
import statment and JavaVendor.Android, or something else?

[markus.icu, 2015-05-12 18:29:36]

https://codereview.appspot.com/235430043/diff/1/main/tests/core/src/com/ibm/i...
File main/tests/core/src/com/ibm/icu/dev/test/util/ULocaleTest.java (right):

https://codereview.appspot.com/235430043/diff/1/main/tests/core/src/com/ibm/i...
main/tests/core/src/com/ibm/icu/dev/test/util/ULocaleTest.java:27: import
com.ibm.icu.dev.test.TestUtil.JavaVendor;
Oh, never mind :-/
Please commit the change, and I will take a rest now...

[roubert (google), 2015-05-12 18:38:47]

Committed patchset #1 (id:1) manually as 37432 (presubmit successful).