commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From S├ębastien Brisard <sebastien.bris...@m4x.org>
Subject Re: svn commit: r1361164 - /commons/proper/math/trunk/src/test/java/org/apache/commons/math3/linear/SparseRealVectorTest.java
Date Fri, 13 Jul 2012 11:36:35 GMT
Hi Gilles,

2012/7/13  <erans@apache.org>:
> Author: erans
> Date: Fri Jul 13 11:20:42 2012
> New Revision: 1361164
>
> URL: http://svn.apache.org/viewvc?rev=1361164&view=rev
> Log:
> Two failing tests temporarily disabled until MATH-821 is fixed.
>
> Modified:
>     commons/proper/math/trunk/src/test/java/org/apache/commons/math3/linear/SparseRealVectorTest.java
>
> Modified: commons/proper/math/trunk/src/test/java/org/apache/commons/math3/linear/SparseRealVectorTest.java
> URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/test/java/org/apache/commons/math3/linear/SparseRealVectorTest.java?rev=1361164&r1=1361163&r2=1361164&view=diff
> ==============================================================================
> --- commons/proper/math/trunk/src/test/java/org/apache/commons/math3/linear/SparseRealVectorTest.java
(original)
> +++ commons/proper/math/trunk/src/test/java/org/apache/commons/math3/linear/SparseRealVectorTest.java
Fri Jul 13 11:20:42 2012
> @@ -97,4 +97,22 @@ public class SparseRealVectorTest extend
>          u.ebeMultiply(v1);
>          u.ebeDivide(v1);
>      }
> +
> +    /**
> +     * XXX This test is disabled because it currently fails.
> +     * The bug must still be fixed in the sparse vector implementation.
> +     * When this is done, this override should be deleted.
> +     */
> +    @Test
> +    @Override
> +    public void testMap() {}
> +
> +    /**
> +     * XXX This test is disabled because it currently fails.
> +     * The bug must still be fixed in the sparse vector implementation.
> +     * When this is done, this override should be deleted.
> +     */
> +    @Test
> +    @Override
> +    public void testMapToSelf() {}
>  }
>
>

We could simply have overriden createFunctions() in
RealVectorAbstractTest(), but your solution is fine! I just took the
liberty to add a @Ignore tag, which shows a message in the tests
reports, just in case we forget to reenable this test.

Best regards,
S├ębastien


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message