Export to GitHub

testability-explorer - issue #5

Maven plugin should only uses classes directory and not entire runtime classpath


Posted on Feb 27, 2009 by Massive Elephant

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.

Attachments

Comment #1

Posted on Mar 7, 2009 by Swift Cat

Thanks 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 Elephant

I 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 Cat

Ah, 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