You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@maven.apache.org by Dominik Dahlem <Do...@cs.tcd.ie> on 2003/09/09 12:33:14 UTC

[Fwd: Maven plug-in - feedback.]

Some feedback on jcoverage...

Dominik

-----Forwarded Message-----
> From: Benoit Xhenseval <be...@yahoo.com>
> To: jcoverage-user@jcoverage.com
> Cc: Dominik.Dahlem@cs.tcd.ie
> Subject: Maven plug-in - feedback.
> Date: Tue, 09 Sep 2003 03:06:30 -0700
> 
> Hi to all,
> 
> We've noticed the following issues with the maven jCoverage plugin.  Some of them might relate to
> jCoverage itself.
> 
> 1/ html-report: If you have define a class in the default package (no package statement), 
>    it throws an exception PackageComparator as the package name is null. 
> 
> 2/ html-report: On the package overview, it lists inner classes as well. However, when you 
>    click on the inner class name it fails (as it attempts to load a file matching the inner 
>    class name). It should either display the parent class, or be removed from the list altogether.
> 
> 
> 3/ html-report: The %branch figure is not correct. It appears to always think 1 branch has 
>    been executed, even if there is 0% coverage. Common figures it lists are 100%, 75%, 67%, 
>    50% 25%, etc. Note: this is after doing a clean and removing the .ser file. 
> 
> 4/ clean: The jcoverage.ser file is not placed in the target directory. Therefore you do not 
>    get correct results are a clean - you have to manually delete the .ser file.
> 
> In any case, a very good start! Thanks a lot.
> 
> Best regards,
> 
> Benoit & Andrew.


---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@maven.apache.org
For additional commands, e-mail: users-help@maven.apache.org


Re: Maven plug-in - feedback.]

Posted by Emmanuel Venisse <ev...@ifrance.com>.
----- Original Message ----- 
From: "Dominik Dahlem" <Do...@cs.tcd.ie>
To: <us...@maven.apache.org>
Sent: Tuesday, September 09, 2003 12:33 PM
Subject: [Fwd: Maven plug-in - feedback.]


> Some feedback on jcoverage...
>
> Dominik
>
> -----Forwarded Message-----
> > From: Benoit Xhenseval <be...@yahoo.com>
> > To: jcoverage-user@jcoverage.com
> > Cc: Dominik.Dahlem@cs.tcd.ie
> > Subject: Maven plug-in - feedback.
> > Date: Tue, 09 Sep 2003 03:06:30 -0700
> >
> > Hi to all,
> >
> > We've noticed the following issues with the maven jCoverage plugin.
Some of them might relate to
> > jCoverage itself.
> >
> > 1/ html-report: If you have define a class in the default package (no
package statement),
> >    it throws an exception PackageComparator as the package name is null.

Done in CVS

> >
> > 2/ html-report: On the package overview, it lists inner classes as well.
However, when you
> >    click on the inner class name it fails (as it attempts to load a file
matching the inner
> >    class name). It should either display the parent class, or be removed
from the list altogether.


Done in CVS

> >
> >
> > 3/ html-report: The %branch figure is not correct. It appears to always
think 1 branch has
> >    been executed, even if there is 0% coverage. Common figures it lists
are 100%, 75%, 67%,
> >    50% 25%, etc. Note: this is after doing a clean and removing the .ser
file.

It's a jcoverage bug.

> >
> > 4/ clean: The jcoverage.ser file is not placed in the target directory.
Therefore you do not
> >    get correct results are a clean - you have to manually delete the
.ser file.


Done in CVS

> >
> > In any case, a very good start! Thanks a lot.
> >
> > Best regards,
> >
> > Benoit & Andrew.
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@maven.apache.org
> For additional commands, e-mail: users-help@maven.apache.org
>
> _____________________________________________________________________
> Envie de discuter en "live" avec vos amis ? Télécharger MSN Messenger
> http://www.ifrance.com/_reloc/m la 1ère messagerie instantanée de France

_____________________________________________________________________
Envie de discuter en "live" avec vos amis ? Télécharger MSN Messenger
http://www.ifrance.com/_reloc/m la 1ère messagerie instantanée de France


---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@maven.apache.org
For additional commands, e-mail: users-help@maven.apache.org