maven-surefire-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brett Porter <br...@apache.org>
Subject Re: svn commit: r513588 - /maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
Date Fri, 02 Mar 2007 02:23:32 GMT
was the NPE junitArtifact, or baseVersion?

- Brett

On 02/03/2007, at 10:09 AM, jvanzyl@apache.org 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";
>                  }
>

Mime
View raw message