[jira] [Commented] (SUREFIRE-1862) IntegrationTestMojo should not silently swallow exceptions thrown by providers

["Tibor Digana (Jira)" <ji...@apache.org>]

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

Tibor Digana commented on SUREFIRE-1862:
----------------------------------------

This method parameter _firstForkException_ is not needed and should be deleted because it is already processed in _AbstractSurefireMojo_

{code:java}
current = failure( current, firstForkException );
handleSummary( current, firstForkException );
{code}


> IntegrationTestMojo should not silently swallow exceptions thrown by providers
> ------------------------------------------------------------------------------
>
>                 Key: SUREFIRE-1862
>                 URL: https://issues.apache.org/jira/browse/SUREFIRE-1862
>             Project: Maven Surefire
>          Issue Type: Bug
>          Components: Maven Failsafe Plugin
>            Reporter: Marc Philipp
>            Priority: Major
>
> The {{IntegrationTestMojo}}'s {{handleSummary}} method currently does not handle the passed {{firstForkException}}:
> [https://github.com/apache/maven-surefire/blob/52ec2773d168d35dcc681b184f72c1053a1b9f16/maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java#L485-L508]
> That leads to provider failures being silently ignored.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)