You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@geode.apache.org by "Dan Smith (JIRA)" <ji...@apache.org> on 2018/10/12 19:45:00 UTC

[jira] [Created] (GEODE-5863) CqDataDUnitTest.testMultipleExecuteWithInitialResults ignores assertion failures

Dan Smith created GEODE-5863:
--------------------------------

             Summary: CqDataDUnitTest.testMultipleExecuteWithInitialResults ignores assertion failures
                 Key: GEODE-5863
                 URL: https://issues.apache.org/jira/browse/GEODE-5863
             Project: Geode
          Issue Type: Improvement
            Reporter: Dan Smith


This test has assertions in an asynchronous thread. However, it never actually checks the results from that thread, it just asserts that the thread finishes. In practice, processCqs always throws an Awaitility exception, which is ignored and the test passes.

This is the problematic code. ThreadUtils.join does not check the result of the async invocation:

{code}
ThreadUtils.join(processCqs, 60 * 1000);
{code}



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