[EMAIL PROTECTED] wrote: <snip>
> Modified: > incubator/harmony/enhanced/classlib/trunk/modules/text/src/test/java/org/apache/harmony/text/tests/java/text/BidiTest.java > URL: > http://svn.apache.org/viewcvs/incubator/harmony/enhanced/classlib/trunk/modules/text/src/test/java/org/apache/harmony/text/tests/java/text/BidiTest.java?rev=400008&r1=400007&r2=400008&view=diff > ============================================================================== > --- > incubator/harmony/enhanced/classlib/trunk/modules/text/src/test/java/org/apache/harmony/text/tests/java/text/BidiTest.java > (original) > +++ > incubator/harmony/enhanced/classlib/trunk/modules/text/src/test/java/org/apache/harmony/text/tests/java/text/BidiTest.java > Fri May 5 00:55:46 2006 > @@ -15,10 +15,8 @@ > > package org.apache.harmony.text.tests.java.text; > > -import java.text.AttributedString; > import java.text.Bidi; > import java.util.Arrays; > -import java.util.List; > > import junit.framework.TestCase; > > @@ -59,6 +57,13 @@ > } catch (IllegalArgumentException e) { > // expected > } > + > + try { > + bd = new Bidi(null); > + fail("should throw IAE"); > + } catch (IllegalArgumentException e) { > + assertEquals("paragraph is null", e.getMessage()); This assertion is bogus, the API spec doesn't require this text, and it will fail when we switch to Russian locale messages. <snip> > + } catch (ParseException e) { > + assertEquals("java.text.ParseException", e.getClass().getName()); Huh? Regards, Tim > + assertEquals(0, e.getErrorOffset()); > + } > + } > + > +} > \ No newline at end of file > > > -- Tim Ellison ([EMAIL PROTECTED]) IBM Java technology centre, UK. --------------------------------------------------------------------- Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]