[freenet-dev] [freenet-cvs] r19178 - in trunk/freenet: src/freenet/support test/freenet/support

[Daniel Cheng (SDiZ)]

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.