You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Peter Janes (JIRA)" <ji...@codehaus.org> on 2013/01/11 17:34:13 UTC

[jira] (MCHECKSTYLE-185) Empty checkstyle-results.xml in a tests-only project

Peter Janes created MCHECKSTYLE-185:
---------------------------------------

             Summary: Empty checkstyle-results.xml in a tests-only project
                 Key: MCHECKSTYLE-185
                 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-185
             Project: Maven 2.x Checkstyle Plugin
          Issue Type: Bug
    Affects Versions: 2.9.1, 2.9, 2.8, 2.7
            Reporter: Peter Janes
         Attachments: check-tests-only.patch

MCHECKSTYLE-153 fixed canGenerateReport() being able to generate a report for projects, but still doesn't add the test sources.  This leads to an empty checkstyle-results.xml file.  The attached patch against 2.10-SNAPSHOT fixes the problem by not short-circuiting addFilesToProcess if there's no source directory.

(A workaround until this bug is fixed is to create an empty ${project.build.sourceDirectory}.)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] (MCHECKSTYLE-185) Empty checkstyle-results.xml in a tests-only project

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
     [ https://jira.codehaus.org/browse/MCHECKSTYLE-185?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dennis Lundberg closed MCHECKSTYLE-185.
---------------------------------------

    Resolution: Fixed
      Assignee: Dennis Lundberg

I have applied your patch in [r1438948|http://svn.apache.org/viewvc?view=revision&revision=1438948] together with a couple of other fixes to the handling of test classes. Thanks!
                
> Empty checkstyle-results.xml in a tests-only project
> ----------------------------------------------------
>
>                 Key: MCHECKSTYLE-185
>                 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-185
>             Project: Maven 2.x Checkstyle Plugin
>          Issue Type: Bug
>    Affects Versions: 2.7, 2.8, 2.9, 2.9.1
>            Reporter: Peter Janes
>            Assignee: Dennis Lundberg
>         Attachments: check-tests-only.patch
>
>
> MCHECKSTYLE-153 fixed canGenerateReport() being able to generate a report for projects, but still doesn't add the test sources.  This leads to an empty checkstyle-results.xml file.  The attached patch against 2.10-SNAPSHOT fixes the problem by not short-circuiting addFilesToProcess if there's no source directory.
> (A workaround until this bug is fixed is to create an empty ${project.build.sourceDirectory}.)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira