
testability-explorer - issue #5
Maven plugin should only uses classes directory and not entire runtime classpath
What steps will reproduce the problem? 1. run the maven 2 plugin 2. 3.
What is the expected output? What do you see instead?
I expect the plugin will only run testability on the classes for a give project not all of the dependencies of that project.
Also, the plugin should have a better artifactId that identifies the plugin.
I have fixed these issues in the attached patch.
- maven2-plugin.patch 5.83KB
Comment #1
Posted on Mar 7, 2009 by Swift CatThanks for reporting this and especially for the patch. I'm leaving the artifactId, since it makes sense with the groupId and matches the ant task. See http://build.hevery.com/job/Testability-Explorer/259/changes#detail0
Comment #2
Posted on Mar 7, 2009 by Massive ElephantI understand the point that it makes sense in terms of the groupId, but I know that you can specify some plugins without groupIds. This may be only for org.apache.maven.plugins....I cannot recall at this time. If you can do it for any groupId mapped in your settings.xml such as
com.google.testability-explorer
then you might have a problem with this approach because then they would only need to put the following the pom:
maven2-plugin
and that can be really confusing. While I think this is always a bad way to define plugins without groupIds, I see it all the time.
Comment #3
Posted on Mar 18, 2009 by Swift CatAh, I see, thank you for the explanation. I agree that it would suck if the groupId wasn't always listed with the artifactId.
I submitted that change as http://code.google.com/p/testability-explorer/source/detail?r=328
Thanks again for the help!!
Status: Fixed
Labels:
Type-Defect
Priority-Medium