Issue Details (XML | Word | Printable)

Key: GRAILS-2521
Type: Test Test
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Graeme Rocher
Reporter: Jason Rudolph
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Grails

Patch: Improve Cobertura report to include src/groovy/* and exclude test/*

Created: 23/Feb/08 08:10 PM   Updated: 13/Mar/08 10:40 AM   Resolved: 13/Mar/08 10:40 AM
Return to search
Component/s: ContinuousBuild, Testing
Affects Version/s: 1.0.1
Fix Version/s: 1.0.2

Time Tracking:
Not Specified

File Attachments: 1. File GRAILS-2521.diff (1 kB)


Patch Submitted: Yes


 Description  « Hide

The attached patch improves the Cobertura report in two ways:

  • Include files in src/groovy in the Cobertura report. Without this patch, the Cobertura report cannot find the source code for those classes (e.g., org.codehaus.groovy.grails.plugins.converters.codecs.JSONCodec).
  • Exclude the test classes from the coverage report. (Including the test classes in the coverage report adversely affects the the coverage percentages, and as a general rule, one is typically only interested in measuring coverage for production code, not test cases.)


Graeme Rocher made changes - 25/Feb/08 07:56 AM
Field Original Value New Value
Fix Version/s 1.0.2 [ 14084 ]
Graeme Rocher added a comment - 13/Mar/08 10:40 AM

Thanks for the patch!


Graeme Rocher made changes - 13/Mar/08 10:40 AM
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]