Re: svn commit: r513588 - /maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java

[Brett Porter]

was the NPE junitArtifact, or baseVersion?

- Brett

On 02/03/2007, at 10:09 AM, [EMAIL PROTECTED] wrote:

Author: jvanzyl
Date: Thu Mar  1 18:09:47 2007
New Revision: 513588

URL: http://svn.apache.org/viewvc?view=rev&rev=513588
Log:

SUREFIRE-300 Fixing the NPE for cases where projects have no test cases.

Modified:

maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/ apache/maven/plugin/surefire/SurefirePlugin.java

Modified: maven/surefire/trunk/maven-surefire-plugin/src/main/java/ org/apache/maven/plugin/surefire/SurefirePlugin.java URL: http://svn.apache.org/viewvc/maven/surefire/trunk/maven- surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/ SurefirePlugin.java?view=diff&rev=513588&r1=513587&r2=513588 ====================================================================== ======== --- maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/ apache/maven/plugin/surefire/SurefirePlugin.java (original) +++ maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/ apache/maven/plugin/surefire/SurefirePlugin.java Thu Mar 1 18:09:47 2007

@@ -591,7 +591,7 @@
             else
             {
                 String junitDirectoryTestSuite;

- if ( junitArtifact.getBaseVersion().startsWith ( "4" ) ) + if ( junitArtifact != null && junitArtifact.getBaseVersion().startsWith( "4" ) )

                 {

junitDirectoryTestSuite = "org.apache.maven.surefire.junit4.JUnit4DirectoryTestSuite";

                 }