[jira] Created: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
25 messages Options
12
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
Download cobertura code coverage jar so it can be used to compile and execute tests with
----------------------------------------------------------------------------------------

                 Key: OFBIZ-3659
                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
             Project: OFBiz
          Issue Type: Bug
          Components: framework
    Affects Versions: SVN trunk
            Reporter: Bob Morley
             Fix For: SVN trunk


Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.

There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bob Morley updated OFBIZ-3659:
------------------------------

    Attachment: OFBIZ-3659_AutomateDownloadOfCobertura.patch

Minor changes but please review the ant fairly closely (it isn't something I do very often).  The intent here is to execute this before compilation (so that the CoberturaInstrumenter gets compiled if the jar has been deployed); but this should be done optionally -- if it fails the build will continue on without instrumentation.  Developer impact is that when they do a build it will pull down this jar (if possible) and when they run the test container it will instrument and generate code coverage metrics.

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854791#action_12854791 ]

Scott Gray commented on OFBIZ-3659:
-----------------------------------

My only concern is that an automatic and transparent download may not constitute an optional dependency (since the user isn't really given an option).

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854794#action_12854794 ]

Adrian Crum commented on OFBIZ-3659:
------------------------------------

Downloading Cobertura should be a part of the test process, not the build process. Ideally, we should have a new ant target - "reports" or "metrics" - that would include this and other code-quality reports.


> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Adrian Crum updated OFBIZ-3659:
-------------------------------

      Priority: Minor  (was: Major)
    Issue Type: New Feature  (was: Bug)

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854797#action_12854797 ]

Scott Gray commented on OFBIZ-3659:
-----------------------------------

It just seems to me that downloading an incompatibly licensed jar is something that the user should do knowingly and not have it be some sort of transparent process.
Download OFBiz: Apache licensed distribution
Run tests: No longer an Apache licensed distribution but the user doesn't know that

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854800#action_12854800 ]

Bob Morley commented on OFBIZ-3659:
-----------------------------------

I agree; my initial approach was to include the download in the test process.  The trouble is the cobertura jar is required to do our compile; if it is not present then our implementation of the instrumenter is not compiled and included in the (I think) base jar.  There is another ant target for the "reports" already existing; this bug does not do anything with that -- this is the ability to compile with Cobertura so that when the test process executes it can instrument and generate the code coverage metrics.

We can do some refactoring on the pluggable instrumentation implementation so that it has a separate build process from the main build.  That is to say when you execute tests we could do an "init-tests" target that would download cobertura and build our integration class at that time.

If you guys want me to bite off that then that is fine -- but what I would ask is that we remove the line that calls "download-cobertura" in my patch so that the other two targets are available to call manually.

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854803#action_12854803 ]

Bob Morley commented on OFBIZ-3659:
-----------------------------------

Scott -- is there a way we can parametrize the download process such that our automated ASF build will do the download (to ultimately get our frequent code coverage metrics) but have it remain manual for developers / end-users?  That is really what I would want anyway (with no impact to developers / end-users).

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854801#action_12854801 ]

Adrian Crum commented on OFBIZ-3659:
------------------------------------

Huh? If it's a local copy, you're not distributing it. If you're distributing it, you would not include metrics libraries in the distribution.


> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854804#action_12854804 ]

Adrian Crum commented on OFBIZ-3659:
------------------------------------

Bob,

Take a look at how Apache Commons does things. They download and execute a number of reporting tools that aren't included in the final library. It's pretty slick and extremely convenient.


> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854806#action_12854806 ]

Scott Gray commented on OFBIZ-3659:
-----------------------------------

Adrian - Just because it's a "local" copy doesn't mean you're not going to distribute it and if you don't know that the metrics library is there then how would you know to exclude it?

Example: Someone may decide to distribute a precompiled version of OFBiz for the convenience of the community.

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854807#action_12854807 ]

Scott Gray commented on OFBIZ-3659:
-----------------------------------

Bob: Buildbot can run any number of tasks in sequence, currently I think it does:
svn co ...
ant run-install
ant run-tests
ant docs-all
then some other stuff for the nightly builds

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854808#action_12854808 ]

Adrian Crum commented on OFBIZ-3659:
------------------------------------

Scott,

If the user is running a separate ant task that downloads and executes a third-party library, then the user will be aware of it. We can even include that bit of information in the ant task's description.

Compiling OFBiz != Downloading and executing code metrics.


> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854811#action_12854811 ]

Scott Gray commented on OFBIZ-3659:
-----------------------------------

Adrian,

You obviously haven't looked at the patch so I guess we're talking about different things.  In the patch Compiling OFBiz == Downloading code metrics tools.

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854812#action_12854812 ]

Adrian Crum commented on OFBIZ-3659:
------------------------------------

Scott,

I looked at the patch, and I commented on it - "Ideally, we should have a new ant target - "reports" or "metrics" - that would include this and other code-quality reports. "

Then Bob said - "We can do some refactoring on the pluggable instrumentation implementation so that it has a separate build process from the main build."


> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12854815#action_12854815 ]

Scott Gray commented on OFBIZ-3659:
-----------------------------------

Okay, I still think it should be it's own manual step though and not automated.  The new targets created in the patch look good to me, I just don't want them called from any processing targets.

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bob Morley updated OFBIZ-3659:
------------------------------

    Attachment: OFBIZ-3659_AutomateDownloadOfCobertura.patch

Replacement that makes the ant targets manual.

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch, OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bob Morley updated OFBIZ-3659:
------------------------------

    Attachment:     (was: OFBIZ-3659_AutomateDownloadOfCobertura.patch)

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bob Morley updated OFBIZ-3659:
------------------------------

    Attachment:     (was: OFBIZ-3659_AutomateDownloadOfCobertura.patch)

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-3659) Download cobertura code coverage jar so it can be used to compile and execute tests with

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-3659?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bob Morley updated OFBIZ-3659:
------------------------------

    Attachment: OFBIZ-3659_AutomateDownloadOfCobertura.patch

Ok this is the real proper patch now (the last one I manually modified and messed it up).

> Download cobertura code coverage jar so it can be used to compile and execute tests with
> ----------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3659
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3659
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3659_AutomateDownloadOfCobertura.patch
>
>
> Framework executed around allowing use of cobertura to create code coverage metrics as part of the Ofbiz build process.  Licensing restricts distribution of cobertura with Ofbiz, so we have taken the approach to attempt to download it before doing our build so it is available for any container that wants instrumentation turned on (currently only test-container).  Net result is when we run our tests we will now get very nice code coverage metrics being generated.
> There is an existing target to generate the cobertura report (html) which is not automatically being executed.  Once we get more reasonable code coverage via unit testing we can automatically generate this and expose it as appropriate.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

12