[jira] [Comment Edited] (RAT-168) Add integration test for apache-rat:rat to catch NPE

["Philipp Ottlinger (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/RAT-168?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14101358#comment-14101358 ] 

Philipp Ottlinger edited comment on RAT-168 at 8/18/14 9:41 PM:
----------------------------------------------------------------

{noformat}
@@ -199,7 +199,7 @@
     /**
     * Holds the maven-internal project to allow resolution of artifact properties during mojo runs.
     */
-    @Parameter(defaultValue = "${project}", required = true, readonly = true)
+    @Parameter(property = "project", required = true, readonly = true)
     private MavenProject project;
{noformat}
yields no NPE during rat:rat and rat:check.

Reverted older patch before building 0.11-RC2 as revision 1618753


was (Author: hugo.hirsch):
{noformat}
@@ -199,7 +199,7 @@
     /**
     * Holds the maven-internal project to allow resolution of artifact properties during mojo runs.
     */
-    @Parameter(defaultValue = "${project}", required = true, readonly = true)
+    @Parameter(property = "project", required = true, readonly = true)
     private MavenProject project;
{noformat}
yields no NPE during rat:rat and rat:check.

Reverted older patch before building 0.11-RC2.

> Add integration test for apache-rat:rat to catch NPE
> ----------------------------------------------------
>
>                 Key: RAT-168
>                 URL: https://issues.apache.org/jira/browse/RAT-168
>             Project: Apache Rat
>          Issue Type: Bug
>    Affects Versions: 0.11
>            Reporter: Sebb
>             Fix For: 0.11
>
>
> The unit/IT tests clearly don't test the Maven CLI functionality.
> mvn apache-rat:rat is currently failing with an NPE, even on the RAT project itself.
> To show this, do the following:
> {noformat}
> mvn or mvn clean install
>  mvn -o org.apache.rat:apache-rat-plugin::0.12-SNAPSHOT:rat 
> (change the version as required)
> {noformat}
> Note that the -o is essential, otherwise Maven will try to download the current snapshot



--
This message was sent by Atlassian JIRA
(v6.2#6252)