gnodet commented on code in PR #14144: URL: https://github.com/apache/camel/pull/14144#discussion_r1600191069
########## 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: I don't think the `surefire.rerunFailingTestsCount` system property is used, because the plugin has a specific configuration set to 0 and the property is only used as a default. If we want the system property to take precedence, we need to remove the line or use something like: ``` <rerunFailingTestsCount>${surefire.rerunFailingTestsCount}</rerunFailingTestsCount> ``` and eventually define it with a default value: ``` <properties> <surefire.rerunFailingTestsCount>2</surefire.rerunFailingTestsCount> </properties> ``` -- 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