[ 
https://jira.codehaus.org/browse/MOJO-1767?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ian Robertson updated MOJO-1767:
--------------------------------

    Attachment: mojo-1767.patch

Patch attached. In addition to adding the 
<addTestClassPath>true</addTestClassPath> line to the pom, I also added 
checkstyle configuration to check src/test/java files in addition to those in 
main; feel free to pull that out if that is discouraged by codehaus guidelines. 
There is also a long-line warning, due to the inclusion of a url in the javadoc.

Checkstyle does complain about the package-info.java file, because it's first 
line isn't the package declaration; unfortunately, javadoc for packages does 
belong above the package declaration.

The mojo-1744 test will likely need to be changed once MOJO-1772 is fixed, but 
for now, it passes on the code as is.

> Improve integration-test for MOJO-1744
> --------------------------------------
>
>                 Key: MOJO-1767
>                 URL: https://jira.codehaus.org/browse/MOJO-1767
>             Project: Mojo
>          Issue Type: Improvement
>          Components: extra-enforcer-rules
>            Reporter: Robert Scholte
>            Assignee: Robert Scholte
>            Priority: Minor
>         Attachments: LogParser.java, mojo-1744.it.2.patch, mojo-1767.patch
>
>
> With MOJO-1744 we have support for ignoring classes for specific dependencies.
> However, the IT seems unstable due to the usage of HashMaps (unpredictable 
> order).
> The IT needs an improvement by ignoring the order, though every entry must be 
> verified.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to