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

[Matthew Toseland]

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?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://emu.freenetproject.org/pipermail/devl/attachments/20080411/36a87143/attachment.pgp