Minor tweaks to the Cobertura integration
----------------------------------------- Key: OFBIZ-3651 URL: https://issues.apache.org/jira/browse/OFBIZ-3651 Project: OFBiz Issue Type: Bug Components: framework Affects Versions: SVN trunk Reporter: Bob Morley Fix For: SVN trunk Attachments: OFBIZ-3651_CoberturaMinorEnhancements.patch When enabling Cobertura I ran into a few things that seemed like candidates for changing. They were: 1) When starting up tests w/o Cobertura (which is our standard) two stack traces would be dumped because the Class could not be found in the classpath. This is standard execution so I felt it was better if we put out a single message to the system console indicating that instrumentation had been disabled. Message is: InstrumenterWorker.instrument - Code instrumentation has been disabled, unable to find instrumenter class " + instrumenterClassName. 2) We know up front if the main configuration / container has been configured to not have an instrumentation so I do a check up front rather than have it go down into the implementation. This is consistent with how other things are set in Start.java. 3) Current code base is not compatible with the latest version of Cobertura. They have removed the static method ProjectData.getGlobalProjectData(). We would actually only ever need this if the instrumenter was not opened before trying to instrument a class (this is Cobertura implementation specific). I wrapped the instrumenting in a check similar to the close method. If you grab Cobertura 1.9.4 you can see this compile error. Intent is to discuss with Ofbiz experts in this area to make sure these are the right things to do. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
[ https://issues.apache.org/jira/browse/OFBIZ-3651?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bob Morley updated OFBIZ-3651: ------------------------------ Attachment: OFBIZ-3651_CoberturaMinorEnhancements.patch > Minor tweaks to the Cobertura integration > ----------------------------------------- > > Key: OFBIZ-3651 > URL: https://issues.apache.org/jira/browse/OFBIZ-3651 > Project: OFBiz > Issue Type: Bug > Components: framework > Affects Versions: SVN trunk > Reporter: Bob Morley > Fix For: SVN trunk > > Attachments: OFBIZ-3651_CoberturaMinorEnhancements.patch > > > When enabling Cobertura I ran into a few things that seemed like candidates for changing. They were: > 1) When starting up tests w/o Cobertura (which is our standard) two stack traces would be dumped because the Class could not be found in the classpath. This is standard execution so I felt it was better if we put out a single message to the system console indicating that instrumentation had been disabled. Message is: InstrumenterWorker.instrument - Code instrumentation has been disabled, unable to find instrumenter class " + instrumenterClassName. > 2) We know up front if the main configuration / container has been configured to not have an instrumentation so I do a check up front rather than have it go down into the implementation. This is consistent with how other things are set in Start.java. > 3) Current code base is not compatible with the latest version of Cobertura. They have removed the static method ProjectData.getGlobalProjectData(). We would actually only ever need this if the instrumenter was not opened before trying to instrument a class (this is Cobertura implementation specific). I wrapped the instrumenting in a check similar to the close method. If you grab Cobertura 1.9.4 you can see this compile error. > Intent is to discuss with Ofbiz experts in this area to make sure these are the right things to do. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
Free forum by Nabble | Edit this page |