On Fri, Apr 11, 2008 at 10:29 PM, Matthew Toseland
<toad at amphibian.dyndns.org> wrote:
> On Friday 11 April 2008 15:22, j16sdiz at freenetproject.org wrote:
> > +
> > + /**
> > + * Test method for {@link
> 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.
