[ 
https://issues.apache.org/jira/browse/GEODE-3180?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16166762#comment-16166762
 ] 

Nick Reich commented on GEODE-3180:
-----------------------------------

As much as I want to remove the checking for log messages, this entire test 
class is completely based on log checking. This is far from ideal, but it would 
take a long time (and possibly significant code refactoring elsewhere) to 
retool this test to follow better patterns. This seems to be a rare issue, so I 
am leaning towards refactoring away from Thread.sleep to the more preferred 
pattern (used elsewhere in this test) of awaitility to hopefully stabilize this 
test. This may have been a missed location in the previous attempt to stabilize 
this test. If that does not work, we will have to consider either ignoring the 
test or putting in the work to redo these tests.

> CI failure: 
> org.apache.geode.internal.cache.partitioned.PersistentColocatedPartitionedRegionDUnitTest.testFullTreeOfColocatedChildPRsWithMissingRegions
>  fails with org.mockito.exceptions.verification.NoInteractionsWanted
> ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: GEODE-3180
>                 URL: https://issues.apache.org/jira/browse/GEODE-3180
>             Project: Geode
>          Issue Type: Bug
>          Components: regions
>            Reporter: Lynn Gallinat
>            Assignee: Nick Reich
>              Labels: storage_3
>
> {noformat}
> org.apache.geode.internal.cache.partitioned.PersistentColocatedPartitionedRegionDUnitTest
>  > testFullTreeOfColocatedChildPRsWithMissingRegions FAILED
>     java.lang.AssertionError: An exception occurred during asynchronous 
> invocation.
>         at 
> org.apache.geode.test.dunit.AsyncInvocation.checkException(AsyncInvocation.java:148)
>         at 
> org.apache.geode.test.dunit.AsyncInvocation.get(AsyncInvocation.java:422)
>         at 
> org.apache.geode.internal.cache.partitioned.PersistentColocatedPartitionedRegionDUnitTest.testFullTreeOfColocatedChildPRsWithMissingRegions(PersistentColocatedPartitionedRegionDUnitTest.java:1298)
>         Caused by:
>         org.mockito.exceptions.verification.NoInteractionsWanted: 
>         No interactions wanted here:
>         -> at 
> org.apache.geode.internal.cache.partitioned.PersistentColocatedPartitionedRegionDUnitTest$14.call(PersistentColocatedPartitionedRegionDUnitTest.java:1232)
>         But found this interaction on mock 'appender':
>         -> at 
> org.apache.logging.log4j.core.config.AppenderControl.tryCallAppender(AppenderControl.java:156)
>         ***
>         For your reference, here is the list of all invocations ([?] - means 
> unverified).
>         1. -> at 
> org.apache.logging.log4j.core.config.AbstractConfiguration.addLoggerAppender(AbstractConfiguration.java:694)
>         2. -> at 
> org.apache.logging.log4j.core.config.AppenderControl.<init>(AppenderControl.java:51)
>         3. -> at 
> org.apache.logging.log4j.core.config.AppenderControl.ensureAppenderStarted(AppenderControl.java:134)
>         4. -> at 
> org.apache.logging.log4j.core.config.AppenderControl.tryCallAppender(AppenderControl.java:156)
>         5. -> at 
> org.apache.logging.log4j.core.config.AppenderControl.ensureAppenderStarted(AppenderControl.java:134)
>         6. [?]-> at 
> org.apache.logging.log4j.core.config.AppenderControl.tryCallAppender(AppenderControl.java:156){noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to