vert-x3 / vertx-ignite

Apache License 2.0
35 stars 28 forks source link

Test sub removed with killed node does not pass constantly #66

Closed vietj closed 4 years ago

vietj commented 6 years ago

The reproducer is to modify the IgniteClusteredEventbusTest with:

  @Rule
  public final RepeatRule rule = new RepeatRule();

  @Test
  @Repeat(times = 1000)
  @Override
  public void testSubsRemovedForKilledNode() throws Exception {
    super.testSubsRemovedForKilledNode();
  }

and run it

> mvn clean test -Dtest=IgniteClusteredEventbusTest#testSubsRemovedForKilledNode

the reported failure:

org.junit.ComparisonFailure: expected:<foo[2]> but was:<foo[1]>
    at org.junit.Assert.assertEquals(Assert.java:115)
    at org.junit.Assert.assertEquals(Assert.java:144)
    at io.vertx.test.core.AsyncTestBase.assertEquals(AsyncTestBase.java:350)
    at io.vertx.test.core.ClusteredEventBusTest.lambda$testSubsRemoved$15(ClusteredEventBusTest.java:231)
    at io.vertx.core.eventbus.impl.HandlerRegistration.deliver(HandlerRegistration.java:223)
    at io.vertx.core.eventbus.impl.HandlerRegistration.handle(HandlerRegistration.java:200)
    at io.vertx.core.eventbus.impl.EventBusImpl.lambda$deliverToHandler$3(EventBusImpl.java:533)
    at io.vertx.core.impl.ContextImpl.lambda$wrapTask$2(ContextImpl.java:339)
    at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:163)
    at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:404)
    at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:463)
    at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:886)
    at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
    at java.lang.Thread.run(Thread.java:748)
vietj commented 6 years ago

can you check this @agura ?

agura commented 6 years ago

I'll take a look.

zyclonite commented 4 years ago

this was resolved latest with the cluster spi refactoring...

tsegismont commented 4 years ago

Closed in #82