[ 
https://issues.apache.org/jira/browse/SUREFIRE-1259?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tibor Digana closed SUREFIRE-1259.
----------------------------------
    Resolution: Duplicate

> ParallelComputerBuilder should not filter out runners with zero children
> ------------------------------------------------------------------------
>
>                 Key: SUREFIRE-1259
>                 URL: https://issues.apache.org/jira/browse/SUREFIRE-1259
>             Project: Maven Surefire
>          Issue Type: Bug
>          Components: Maven Failsafe Plugin
>    Affects Versions: 2.19.1
>            Reporter: Fuud
>            Assignee: Tibor Digana
>
> ParallelComputerBuilder
> {code}
> private <T extends Runner> WrappedRunners wrapRunners( Collection<T> runners )
>             throws InitializationError
>         {
>             // Do NOT use allGroups here.
>             long childrenCounter = 0;
>             ArrayList<Runner> runs = new ArrayList<Runner>();
>             for ( T runner : runners )
>             {
>                 if ( runner != null )
>                 {
>                     int children = countChildren( runner );
>                     childrenCounter += children;
>                     if ( children != 0 )
>                     {
>                         runs.add( runner );
>                     }
>                 }
>             }
>             return runs.isEmpty() ? new WrappedRunners() : new 
> WrappedRunners( createSuite( runs ), childrenCounter );
>         }
> {code}
> JUnit does not enfore ParentRunner to report all it's children. Therefore 
> zero children count should not be criteria to skip this runner.
> I suggest to remove "if ( children != 0 )" condition and run runner anyway.
> Pull request https://github.com/apache/maven-surefire/pull/114



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to