[
https://issues.apache.org/jira/browse/SLING-2604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Carsten Ziegeler resolved SLING-2604.
-------------------------------------
Resolution: Won't Fix
> Method getChildren from the SlingRemoteTestRunner is breaking the execution
> of the Surefire plugin when using JUnit categories
> ------------------------------------------------------------------------------------------------------------------------------
>
> Key: SLING-2604
> URL: https://issues.apache.org/jira/browse/SLING-2604
> Project: Sling
> Issue Type: Bug
> Components: Scripting, Testing
> Affects Versions: JUnit Remote Tests Runners 1.0.6
> Environment: Windows 7
> Maven Surefire 2.12.3
> Reporter: Cristian Vazzolla
> Priority: Major
> Attachments: SlingRemoteTestRunner.patch,
> log_bug_sling_remote_runner.txt
>
>
> Method getChildren from the SlingRemoteTestRunner is breaking the execution
> of the Surefire plugin when using JUnit categories and executing the
> ServerSideIT tests if these throw an error in the maybeExecuteTests method.
> The getChildren method from the SlingRemoteTestRunner which overrides the
> getChildren method of the ParentRunner is not supposed to throw an error or
> at least it does not take into account that possibility when using JUnit
> categories with the Maven Surefire plugin. Because in that method an error is
> thrown , the error will go up and break the execution of the Surefire plugin
> when using JUnit categories. The method is called from the
> getFilteredChildren filter method of the ParentRunner. I'm not quite familiar
> with the SlingRemoteTestRunner but I think that we should remove the
> maybeExecuteTests call from the getChildren method of the
> SlingRemoteTestRunner.
> Attached is the log file and a patch for this which contains the method which
> just returns the children.
> Tried this also with the last version from the repo and it is the same, the
> method is not modified.
> We are not able to run the integration tests with the JUnit categories if we
> want to also run the ServerSideIt tests.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
