On Fri, Apr 11, 2008 at 10:29 PM, Matthew Toseland
<[EMAIL PROTECTED]> wrote:
> On Friday 11 April 2008 15:22, [EMAIL PROTECTED] wrote:
> > +
> > + /**
> > + * Test method for [EMAIL PROTECTED]
> freenet.support.SortedLongSet#remove(long)}.
> > + */
> > + public void testRemove() {
> > + // Construction and get
> > + SortedLongSet set = perpare(testArray);
> > +
> > + // Remove
> > + assertTrue(set.contains(0L));
> > + set.remove(0L);
> > + assertFalse(set.contains(0L));
> > +
> > + // Construction and get
> > + SortedLongSet set2 = perpare(testArray);
> > + // Remove non exist
> > + assertFalse(set2.contains(101L));
> > + set.remove(10L);
>
> Surely set.remove(101L) ? Otherwise the assertTrue below doesn't make sense.
> Have you tried the test?
>
fixed, thanks.
_______________________________________________
Devl mailing list
[email protected]
http://emu.freenetproject.org/cgi-bin/mailman/listinfo/devl
