Kirk Lund created GEODE-5914:
--------------------------------

             Summary: DUnitLauncher.closeAndCheckForSuspects() needs test 
coverage
                 Key: GEODE-5914
                 URL: https://issues.apache.org/jira/browse/GEODE-5914
             Project: Geode
          Issue Type: Test
          Components: tests
            Reporter: Kirk Lund


DUnitLauncher.closeAndCheckForSuspects() is executed after every distributed 
test class is run. This should catch any suspicious strings that were logged or 
printed to system out and cause the test to fail.

I recently noticed DistributedSystemDUnitTest producing the following stack 
trace but it doesn't seem to fail closeAndCheckForSuspects(). I checked to see 
if we have any test coverage for closeAndCheckForSuspects() that would fail if 
we break the check-for-suspects but there isn't any such test.
{noformat}
[vm0] [warn 2018/10/23 12:51:35.339 PDT <Pooled Message Processor 2> tid=0x58] 
attempt to unlock read lock, not locked by current thread
[vm0] java.lang.IllegalMonitorStateException: attempt to unlock read lock, not 
locked by current thread
[vm0]   at 
java.util.concurrent.locks.ReentrantReadWriteLock$Sync.unmatchedUnlockException(ReentrantReadWriteLock.java:444)
[vm0]   at 
java.util.concurrent.locks.ReentrantReadWriteLock$Sync.tryReleaseShared(ReentrantReadWriteLock.java:428)
[vm0]   at 
java.util.concurrent.locks.AbstractQueuedSynchronizer.releaseShared(AbstractQueuedSynchronizer.java:1341)
[vm0]   at 
java.util.concurrent.locks.ReentrantReadWriteLock$ReadLock.unlock(ReentrantReadWriteLock.java:881)
[vm0]   at 
org.apache.geode.management.internal.beans.ManagementListener.handleEvent(ManagementListener.java:232)
[vm0]   at 
org.apache.geode.distributed.internal.InternalDistributedSystem.notifyResourceEventListeners(InternalDistributedSystem.java:2219)
[vm0]   at 
org.apache.geode.distributed.internal.InternalDistributedSystem.handleResourceEvent(InternalDistributedSystem.java:595)
[vm0]   at 
org.apache.geode.internal.admin.remote.AlertListenerMessage.process(AlertListenerMessage.java:106)
[vm0]   at 
org.apache.geode.distributed.internal.DistributionMessage.scheduleAction(DistributionMessage.java:367)
[vm0]   at 
org.apache.geode.distributed.internal.DistributionMessage$1.run(DistributionMessage.java:432)
[vm0]   at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
[vm0]   at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
[vm0]   at 
org.apache.geode.distributed.internal.ClusterDistributionManager.runUntilShutdown(ClusterDistributionManager.java:954)
[vm0]   at 
org.apache.geode.distributed.internal.ClusterDistributionManager.doProcessingThread(ClusterDistributionManager.java:820)
[vm0]   at 
org.apache.geode.internal.logging.LoggingThreadFactory.lambda$newThread$0(LoggingThreadFactory.java:121)
[vm0]   at java.lang.Thread.run(Thread.java:748)
{noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to