You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@maven.apache.org by joseph22 <yu...@gmail.com> on 2008/03/04 21:41:40 UTC
Re: Coverage report of web application integration test with
cobertura-maven-plugin
We have a very similar need. We have an integration-test module that has
dependencies on other modules containing unit tests and would like to have
the integration-test module use the instrumented classes from its
dependencies to generate a cobertura report. A few questions:
1. Has this issue been logged as a jira bug or task against the plugin? If
so, what is the jira number?
2. The work that you or jeff have been doing -- is it checked in anywhere?
If so, how could someone check it out.
tks
Joseph
Joshua Spiewak wrote:
>
> So in a simplistic attempt to get this to work, I looked at the Clover
> plugin to see what it did. I tried changing the execute phase of
> CoberturaCheckMojo and CoberturaReportMojo to "verify", like the Clover
> mojos of the same function. This fails in the package phase:
>
> [INFO] Error configuring: org.apache.maven.plugins:maven-war-plugin.
> Reason: ERROR: Cannot override read-only parameter: classesDirectory in
> goal: war:war
>
> Looking at the Clover plugin a little more, it looks like it manages its
> separate lifecycle a little differently and only the instrument mojo kicks
> off the separate lifecycle. And in fact, the instrument mojo is a shell,
> and there is an internal instrument mojo that does the real work.
>
> Do you think that the Cobertura mojo can be incrementally moved to this
> model? What else were you intending?
>
> I would love to get this working, as having code coverage (with checking)
> is that last step in converting our main project from Ant to Maven. So
> let me know if you need help!
>
> -- Josh
>
>
>
> Jeff MAURY wrote:
>>
>> Richard,
>>
>> I have the same problem and I think this is not feasible with the current
>> design of the Cobertura Maven2 plugin design as it is today. It does only
>> address the test phase and not the integration-test phase. I plan to work
>> on
>> another Cobertura plugin (inspired from the philosophy of the Clover
>> plugin)
>> that address this issue.
>>
>> Jeff
>>
>>
>> On Dec 13, 2007 3:29 PM, Brewster, Richard <ri...@pqa.com>
>> wrote:
>>
>>> I want to produce a coverage report for the integration test phase of a
>>> web application (war packaging). My integration tests are run by the
>>> Cargo maven plugin launching JBoss 4.0.5. I've been using the 2.0
>>> version of the cobertura-maven-plugin (and will try the upgrade to 2.2).
>>> I have a modular project with core and web modules. I get the expected
>>> coverage reports for my core module, but zero coverage for the web
>>> module. The instrumented classes are not packaged into the war, for
>>> one. According to the Cobertura FAQ, this sort of coverage testing can
>>> be done.
>>>
>>> http://cobertura.sourceforge.net/faq.html
>>>
>>> Has anyone done this? Any advice would be appreciated!
>>>
>>> Also, it would be great if integration test coverage could be documented
>>> here, too:
>>>
>>> http://mojo.codehaus.org/cobertura-maven-plugin/usage.html
>>>
>>> Thanks,
>>>
>>> Richard Brewster
>>> Senior Associate
>>> Perrin Quarles Associates
>>> richardbrewster@pqa.com
>>> (434) 817-2640
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscribe@maven.apache.org
>>> For additional commands, e-mail: users-help@maven.apache.org
>>>
>>>
>>
>>
>> --
>> La mélancolie c'est communiste
>> Tout le monde y a droit de temps en temps
>> La mélancolie n'est pas capitaliste
>> C'est même gratuit pour les perdants
>> La mélancolie c'est pacifiste
>> On ne lui rentre jamais dedans
>> La mélancolie oh tu sais ça existe
>> Elle se prend même avec des gants
>> La mélancolie c'est pour les syndicalistes
>> Il faut juste sa carte de permanent
>>
>> Miossec (2006)
>>
>> http://www.jeffmaury.com
>> http://riadiscuss.jeffmaury.com
>>
>>
>
>
--
View this message in context: http://www.nabble.com/-ANN--cobertura-maven-plugin-2.2-tp14310731s177p15836604.html
Sent from the Maven - Users mailing list archive at Nabble.com.
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@maven.apache.org
For additional commands, e-mail: users-help@maven.apache.org
Re: Coverage report of web application integration test with cobertura-maven-plugin
Posted by Jeff MAURY <je...@jeffmaury.com>.
On Thu, Mar 6, 2008 at 4:23 AM, Joshua Spiewak <js...@ivyventures.net> wrote:
>
> 1. Looks like an issue was just opened here:
>
> http://jira.codehaus.org/browse/MCOBERTURA-86
>
> 2. Not sure about Jeff, mine hasn't amounted to anything worth checking
> in,
> and my effort has fallen by the wayside for the past couple of months.
I did not check in my code because I did not have the committer status.
I still need to do more validation because I first developed it with an
intent to use it with Cargo but this one has some problems with classifier.
So I need to build a test POM for the validation.
However, if you need it, I can send the code.
Jeff
>
>
> joseph22 wrote:
> >
> > We have a very similar need. We have an integration-test module that
> has
> > dependencies on other modules containing unit tests and would like to
> have
> > the integration-test module use the instrumented classes from its
> > dependencies to generate a cobertura report. A few questions:
> >
> > 1. Has this issue been logged as a jira bug or task against the plugin?
> > If so, what is the jira number?
> > 2. The work that you or jeff have been doing -- is it checked in
> > anywhere? If so, how could someone check it out.
> >
> > tks
> > Joseph
> >
> >
> > Joshua Spiewak wrote:
> >>
> >> So in a simplistic attempt to get this to work, I looked at the Clover
> >> plugin to see what it did. I tried changing the execute phase of
> >> CoberturaCheckMojo and CoberturaReportMojo to "verify", like the Clover
> >> mojos of the same function. This fails in the package phase:
> >>
> >> [INFO] Error configuring: org.apache.maven.plugins:maven-war-plugin.
> >> Reason: ERROR: Cannot override read-only parameter: classesDirectory in
> >> goal: war:war
> >>
> >> Looking at the Clover plugin a little more, it looks like it manages
> its
> >> separate lifecycle a little differently and only the instrument mojo
> >> kicks off the separate lifecycle. And in fact, the instrument mojo is
> a
> >> shell, and there is an internal instrument mojo that does the real
> work.
> >>
> >> Do you think that the Cobertura mojo can be incrementally moved to this
> >> model? What else were you intending?
> >>
> >> I would love to get this working, as having code coverage (with
> checking)
> >> is that last step in converting our main project from Ant to Maven. So
> >> let me know if you need help!
> >>
> >> -- Josh
> >>
> >>
> >>
> >> Jeff MAURY wrote:
> >>>
> >>> Richard,
> >>>
> >>> I have the same problem and I think this is not feasible with the
> >>> current
> >>> design of the Cobertura Maven2 plugin design as it is today. It does
> >>> only
> >>> address the test phase and not the integration-test phase. I plan to
> >>> work on
> >>> another Cobertura plugin (inspired from the philosophy of the Clover
> >>> plugin)
> >>> that address this issue.
> >>>
> >>> Jeff
> >>>
> >>>
> >>> On Dec 13, 2007 3:29 PM, Brewster, Richard <ri...@pqa.com>
> >>> wrote:
> >>>
> >>>> I want to produce a coverage report for the integration test phase of
> a
> >>>> web application (war packaging). My integration tests are run by the
> >>>> Cargo maven plugin launching JBoss 4.0.5. I've been using the 2.0
> >>>> version of the cobertura-maven-plugin (and will try the upgrade to
> >>>> 2.2).
> >>>> I have a modular project with core and web modules. I get the
> expected
> >>>> coverage reports for my core module, but zero coverage for the web
> >>>> module. The instrumented classes are not packaged into the war, for
> >>>> one. According to the Cobertura FAQ, this sort of coverage testing
> can
> >>>> be done.
> >>>>
> >>>> http://cobertura.sourceforge.net/faq.html
> >>>>
> >>>> Has anyone done this? Any advice would be appreciated!
> >>>>
> >>>> Also, it would be great if integration test coverage could be
> >>>> documented
> >>>> here, too:
> >>>>
> >>>> http://mojo.codehaus.org/cobertura-maven-plugin/usage.html
> >>>>
> >>>> Thanks,
> >>>>
> >>>> Richard Brewster
> >>>> Senior Associate
> >>>> Perrin Quarles Associates
> >>>> richardbrewster@pqa.com
> >>>> (434) 817-2640
> >>>>
> >>>> ---------------------------------------------------------------------
> >>>> To unsubscribe, e-mail: users-unsubscribe@maven.apache.org
> >>>> For additional commands, e-mail: users-help@maven.apache.org
> >>>>
> >>>>
> >>>
> >>>
> >>> --
> >>> La mélancolie c'est communiste
> >>> Tout le monde y a droit de temps en temps
> >>> La mélancolie n'est pas capitaliste
> >>> C'est même gratuit pour les perdants
> >>> La mélancolie c'est pacifiste
> >>> On ne lui rentre jamais dedans
> >>> La mélancolie oh tu sais ça existe
> >>> Elle se prend même avec des gants
> >>> La mélancolie c'est pour les syndicalistes
> >>> Il faut juste sa carte de permanent
> >>>
> >>> Miossec (2006)
> >>>
> >>> http://www.jeffmaury.com
> >>> http://riadiscuss.jeffmaury.com
> >>>
> >>>
> >>
> >>
> >
> >
>
> --
> View this message in context:
> http://www.nabble.com/-ANN--cobertura-maven-plugin-2.2-tp14310731s177p15865771.html
> Sent from the Maven - Users mailing list archive at Nabble.com.
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@maven.apache.org
> For additional commands, e-mail: users-help@maven.apache.org
>
>
--
La mélancolie c'est communiste
Tout le monde y a droit de temps en temps
La mélancolie n'est pas capitaliste
C'est même gratuit pour les perdants
La mélancolie c'est pacifiste
On ne lui rentre jamais dedans
La mélancolie oh tu sais ça existe
Elle se prend même avec des gants
La mélancolie c'est pour les syndicalistes
Il faut juste sa carte de permanent
Miossec (2006)
http://www.jeffmaury.com
http://riadiscuss.jeffmaury.com
Re: Coverage report of web application integration test with
cobertura-maven-plugin
Posted by Joshua Spiewak <js...@ivyventures.net>.
1. Looks like an issue was just opened here:
http://jira.codehaus.org/browse/MCOBERTURA-86
2. Not sure about Jeff, mine hasn't amounted to anything worth checking in,
and my effort has fallen by the wayside for the past couple of months.
joseph22 wrote:
>
> We have a very similar need. We have an integration-test module that has
> dependencies on other modules containing unit tests and would like to have
> the integration-test module use the instrumented classes from its
> dependencies to generate a cobertura report. A few questions:
>
> 1. Has this issue been logged as a jira bug or task against the plugin?
> If so, what is the jira number?
> 2. The work that you or jeff have been doing -- is it checked in
> anywhere? If so, how could someone check it out.
>
> tks
> Joseph
>
>
> Joshua Spiewak wrote:
>>
>> So in a simplistic attempt to get this to work, I looked at the Clover
>> plugin to see what it did. I tried changing the execute phase of
>> CoberturaCheckMojo and CoberturaReportMojo to "verify", like the Clover
>> mojos of the same function. This fails in the package phase:
>>
>> [INFO] Error configuring: org.apache.maven.plugins:maven-war-plugin.
>> Reason: ERROR: Cannot override read-only parameter: classesDirectory in
>> goal: war:war
>>
>> Looking at the Clover plugin a little more, it looks like it manages its
>> separate lifecycle a little differently and only the instrument mojo
>> kicks off the separate lifecycle. And in fact, the instrument mojo is a
>> shell, and there is an internal instrument mojo that does the real work.
>>
>> Do you think that the Cobertura mojo can be incrementally moved to this
>> model? What else were you intending?
>>
>> I would love to get this working, as having code coverage (with checking)
>> is that last step in converting our main project from Ant to Maven. So
>> let me know if you need help!
>>
>> -- Josh
>>
>>
>>
>> Jeff MAURY wrote:
>>>
>>> Richard,
>>>
>>> I have the same problem and I think this is not feasible with the
>>> current
>>> design of the Cobertura Maven2 plugin design as it is today. It does
>>> only
>>> address the test phase and not the integration-test phase. I plan to
>>> work on
>>> another Cobertura plugin (inspired from the philosophy of the Clover
>>> plugin)
>>> that address this issue.
>>>
>>> Jeff
>>>
>>>
>>> On Dec 13, 2007 3:29 PM, Brewster, Richard <ri...@pqa.com>
>>> wrote:
>>>
>>>> I want to produce a coverage report for the integration test phase of a
>>>> web application (war packaging). My integration tests are run by the
>>>> Cargo maven plugin launching JBoss 4.0.5. I've been using the 2.0
>>>> version of the cobertura-maven-plugin (and will try the upgrade to
>>>> 2.2).
>>>> I have a modular project with core and web modules. I get the expected
>>>> coverage reports for my core module, but zero coverage for the web
>>>> module. The instrumented classes are not packaged into the war, for
>>>> one. According to the Cobertura FAQ, this sort of coverage testing can
>>>> be done.
>>>>
>>>> http://cobertura.sourceforge.net/faq.html
>>>>
>>>> Has anyone done this? Any advice would be appreciated!
>>>>
>>>> Also, it would be great if integration test coverage could be
>>>> documented
>>>> here, too:
>>>>
>>>> http://mojo.codehaus.org/cobertura-maven-plugin/usage.html
>>>>
>>>> Thanks,
>>>>
>>>> Richard Brewster
>>>> Senior Associate
>>>> Perrin Quarles Associates
>>>> richardbrewster@pqa.com
>>>> (434) 817-2640
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: users-unsubscribe@maven.apache.org
>>>> For additional commands, e-mail: users-help@maven.apache.org
>>>>
>>>>
>>>
>>>
>>> --
>>> La mélancolie c'est communiste
>>> Tout le monde y a droit de temps en temps
>>> La mélancolie n'est pas capitaliste
>>> C'est même gratuit pour les perdants
>>> La mélancolie c'est pacifiste
>>> On ne lui rentre jamais dedans
>>> La mélancolie oh tu sais ça existe
>>> Elle se prend même avec des gants
>>> La mélancolie c'est pour les syndicalistes
>>> Il faut juste sa carte de permanent
>>>
>>> Miossec (2006)
>>>
>>> http://www.jeffmaury.com
>>> http://riadiscuss.jeffmaury.com
>>>
>>>
>>
>>
>
>
--
View this message in context: http://www.nabble.com/-ANN--cobertura-maven-plugin-2.2-tp14310731s177p15865771.html
Sent from the Maven - Users mailing list archive at Nabble.com.
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@maven.apache.org
For additional commands, e-mail: users-help@maven.apache.org