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/19 00:13:00 UTC

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

     [ https://issues.apache.org/jira/browse/GEODE-5863?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dan Smith reassigned GEODE-5863:
--------------------------------

    Assignee: Dan Smith

> 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
>            Assignee: Dan Smith
>            Priority: Major
>              Labels: swat
>
> 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)