Re: AsyncEventListenerDUnitTest

[Kirk Lund <kl...@apache.org>]

Sorry this is taking me so long. This test had even more issues.

1) There are 5 tests marked with @Ignore that I decided to fix:

  @Ignore("TODO: Disabled until I can sort out the hydra dependencies - see
bug 52214")
  @Test
  public void testReplicatedSerialAsyncEventQueueWithoutLocator() {

  @Ignore("TODO: Disabled for 52351")
  @Test
  public void
testReplicatedSerialAsyncEventQueueWithPeristenceEnabled_Restart() {

  @Ignore("TODO: Disabled for 52351")
  @Test
  public void
testReplicatedSerialAsyncEventQueueWithPeristenceEnabled_Restart2() {

  @Ignore("TODO: Disabled for 52349")
  @Test
  public void testParallelAsyncEventQueueHA_Scenario1() {

  @Ignore("Depends on hydra code. See bug ")
  @Test
  public void testParallelAsyncEventQueueHA_Scenario3() {

2) One test missing @Test that seems to work when I add the annotation:

  public void
testParallelAsyncEventQueueMovePrimaryAndMoveItBackDuringDispatching() {

3) Flakiness which prompted the disabling of tests in #1 caused by
concurrency bugs in the various implementations of AsyncEventListener.

On Fri, Jul 27, 2018 at 12:10 PM, Kirk Lund <kl...@apache.org> wrote:

> If you update to the new version of IntelliJ (or switch to using Eclipse),
> you may see an error message about AsyncEventListenerDUnitTest. I'm
> currently reworking this test to fix this issue.
>
>
>
> Eclipse:
>
>
> The code of method $deserializeLambda$(SerializedLambda) is exceeding the
> 65535 bytes limit AsyncEventListenerDUnitTest.java /geode-core/src/
> distributedTest/java/org/apache/geode/internal/cache/wan/asyncqueue line
> 74
>
>
> IntelliJ:
>
>
> /Users/klund/dev/gemfire1/open/geode-core/src/distributedTest/java/org/
> apache/geode/internal/cache/wan/asyncqueue/AsyncEventListenerDUnitTest.
> java
>
> Error:(143, 1) java: code too large
>
>
>
>