Are you sure that you have svn upped after this commit? Because maven
needs at least one test method in a test class. This was the reason I
added that method. And it was actually to make integration tests pass.

Don't worry, be happy..

On 11/25/06, Emmanuel Lecharny <[EMAIL PROTECTED]> wrote:
Ersin,

 while modifying this test, you have made Maven happy, but you have made me
unhappy ;)

 If you launch mvn install, everything is fine
 If you launch mvn -Dintegration test, then maven - and I - is not so happy
:(

 Sorry that you have to try again to make maven more happy :)

 Emmanuel


On 11/22/06, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
> Author: ersiner
> Date: Wed Nov 22 01:29:55 2006
> New Revision: 478099
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=478099
> Log:
> Added one test method to make maven happy. The main test was commented
out.
>
> Modified:
>
directory/trunks/apacheds/core-unit/src/test/java/org/apache/directory/server/core/subtree/SubentryServiceObjectClassChangeHandlingITest.java
>
> Modified:
directory/trunks/apacheds/core-unit/src/test/java/org/apache/directory/server/core/subtree/SubentryServiceObjectClassChangeHandlingITest.java
> URL:
http://svn.apache.org/viewvc/directory/trunks/apacheds/core-unit/src/test/java/org/apache/directory/server/core/subtree/SubentryServiceObjectClassChangeHandlingITest.java?view=diff&rev=478099&r1=478098&r2=478099
>
==============================================================================
> ---
directory/trunks/apacheds/core-unit/src/test/java/org/apache/directory/server/core/subtree/SubentryServiceObjectClassChangeHandlingITest.java
(original)
> +++
directory/trunks/apacheds/core-unit/src/test/java/org/apache/directory/server/core/subtree/SubentryServiceObjectClassChangeHandlingITest.java
Wed Nov 22 01:29:55 2006
> @@ -115,6 +115,15 @@
>
>
>      /*
> +     * Remove this test when you fix DIRSERVER-784 and activate the
following test.
> +     */
> +    public void testEmpty()
> +    {
> +        assertTrue( true );
> +    }
> +
> +
> +    /*
>       * FIXME: The following test fails.
>       */
>
>
>
>



--
Cordialement,
Emmanuel Lécharny


--
Ersin

Reply via email to