You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Benedikt Ritter <br...@apache.org> on 2013/03/14 08:37:04 UTC

Re: svn commit: r1456130 - /commons/proper/logging/trunk/src/test/java/org/apache/commons/logging/impl/WeakHashtableTestCase.java

2013/3/13 <tn...@apache.org>

> Author: tn
> Date: Wed Mar 13 21:21:07 2013
> New Revision: 1456130
>
> URL: http://svn.apache.org/r1456130
> Log:
> Disable flawed test.
>
> Modified:
>
> commons/proper/logging/trunk/src/test/java/org/apache/commons/logging/impl/WeakHashtableTestCase.java
>
> Modified:
> commons/proper/logging/trunk/src/test/java/org/apache/commons/logging/impl/WeakHashtableTestCase.java
> URL:
> http://svn.apache.org/viewvc/commons/proper/logging/trunk/src/test/java/org/apache/commons/logging/impl/WeakHashtableTestCase.java?rev=1456130&r1=1456129&r2=1456130&view=diff
>
> ==============================================================================
> ---
> commons/proper/logging/trunk/src/test/java/org/apache/commons/logging/impl/WeakHashtableTestCase.java
> (original)
> +++
> commons/proper/logging/trunk/src/test/java/org/apache/commons/logging/impl/WeakHashtableTestCase.java
> Wed Mar 13 21:21:07 2013
> @@ -220,7 +220,14 @@ public class WeakHashtableTestCase exten
>          assertTrue(values.contains(valueThree));
>      }
>
> -    public void testRelease() throws Exception {
> +    /**
> +     * Disables this tests as it makes wrong assumptions wrt the GC.
>

Little typo here? Should be "Disabled", right?


> +     * This test especially fails with:
> +     *
> +     * Java(TM) SE Runtime Environment (build
> pxi3260sr12-20121025_01(SR12))
> +     * IBM J9 VM (build 2.4, JRE 1.6.0 IBM J9 2.4 Linux x86-32
> jvmxi3260sr12-20121024_1
> +     */
> +    public void xxxIgnoretestRelease() throws Exception {
>          assertNotNull(weakHashtable.get(new Long(1)));
>          ReferenceQueue testQueue = new ReferenceQueue();
>          WeakReference weakKeyOne = new WeakReference(keyOne, testQueue);
>
>
>


-- 
http://people.apache.org/~britter/
http://www.systemoutprintln.de/
http://twitter.com/BenediktRitter
http://github.com/britter