Details
-
Suggestion
-
Resolution: Fixed
-
None
Description
From looking at the _Events.groovy hooks in the plugin, there are several println statements which will execute in all contexts, not just testing and not just when clover.on is specified. Specifically, the set classpath and compile start/end hooks will happen everytime any Grails command is run.
This is bad practice for a plugin. The println statements should be changed to use the grailsConsole which is available automatically
println "foo" => grailsConsole.log "foo"
or grailsConsole.updateStatus "foo"
Secondly, this should probably be wrapped in a helper which only prints if the the clover is enabled, ie:
def logStuff(msg)
{ if( config.on ) grailsConsole.log msg }