On 5 May 2006 at 11:33, Tim Ellison <[EMAIL PROTECTED]> wrote:
>
> [EMAIL PROTECTED] wrote:
> <snip>
>
> > Modified: incubator/harmony/enhanced/classlib/trunk/modules/text/src/test/j
> ava/org/apache/harmony/text/tests/java/text/BidiTest.java
> > URL: http://svn.apache.org/viewcvs/incubator/harmony/enhanced/classlib/trun
> k/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/or
> g/apache/harmony/text/tests/java/text/BidiTest.java (original)
> > +++ incubator/harmony/enhanced/classlib/trunk/modules/text/src/test/java/or
> g/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.
I mentioned that this should be removed in my most recent JIRA comment
for this issue.
> <snip>
>
> > + } catch (ParseException e) {
> > + assertEquals("java.text.ParseException", e.getClass().getName(
> ));
>
> Huh?
Oops! Didn't spot that one. Who wrote that rubbish? ;-)
-Mark.
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
