gnodet commented on code in PR #14144:
URL: https://github.com/apache/camel/pull/14144#discussion_r1600227218
##########
parent/pom.xml:
##########
@@ -3535,8 +3535,8 @@
<failIfNoTests>false</failIfNoTests>
<failIfNoSpecifiedTests>false</failIfNoSpecifiedTests>
<runOrder>alphabetical</runOrder>
- <!-- lets re-run the failed test one more time,
just to be sure -->
- <rerunFailingTestsCount>0</rerunFailingTestsCount>
+ <!-- lets re-run the failed test, just to be sure
-->
+ <rerunFailingTestsCount>2</rerunFailingTestsCount>
Review Comment:
> BTW, it may be necessary to do the same in a few more places (and also for
the failsafe plugin). I looked briefly at the code and there's a few more
places where this is set.
>
> Good catch!!!
FWIW, the `rerunFailingTestsCount` is not explicitely set in the failsafe
plugin config, so that one is not a problem and the
`failsafe.rerunFailingTestsCount` system property should be honored.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
