tamaashu commented on a change in pull request #372:
URL: https://github.com/apache/curator/pull/372#discussion_r475893740
##########
File path: curator-client/src/test/java/org/apache/curator/BasicTests.java
##########
@@ -30,15 +37,13 @@
import org.apache.zookeeper.ZooDefs;
import org.apache.zookeeper.ZooKeeper;
import org.mockito.Mockito;
-import org.testng.Assert;
-import org.testng.annotations.Test;
import java.util.concurrent.Callable;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.atomic.AtomicBoolean;
public class BasicTests extends BaseClassForTests
{
- @Test
+ @RepeatedIfExceptionsTest(repeats = BaseClassForTests.REPEATS)
Review comment:
There are multiple ways to repeat failed tests with jUnit5.
Most simple is Maven's surefire plugin, which works now with jUnit5.
But in Curator in the original solution not all of the tests have been
repeated when failed. Some of them have been tried only once, some not.
This extension provides an annotation which you can use on those tests which
you want to repeat when failed, but still have the option to use @Test on those
you are not interested in. I thinks this greater flexibility is valuable.
What do you think?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
