[jira] Updated: (DERBY-3178) DataSourceTest uses ConnectionEventListener but never verifies that events are getting fired

["Mamta A. Satoor (JIRA)" <ji...@apache.org>]

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

Mamta A. Satoor updated DERBY-3178:
-----------------------------------

    Attachment: DERBY_3178_diff_ver1_Nov_06_2007.txt

Committed attached patch DERBY_3178_diff_ver1_Nov_06_2007.txt to trunk using revision 592544. The commit comments which describes the fix are as follows

DERBY-3178
This commit modifies DataSourceTest.java to ensure that expected Connection events are fired to the
ConnectionEventListener. The changes have been implemented by having AssertEventCatcher in DataSourceTest.java keep track of what events it received. This state is kept in boolean gotConnectionClosed and boolean gotConnectionErrorOccured flags in AssertEventCatcher. The test then checks the state of AssertEventCatcher after the test has done firing the events to make sure AssertEventCatcher got the expected events.


> DataSourceTest uses ConnectionEventListener but never verifies that events are getting fired
> --------------------------------------------------------------------------------------------
>
>                 Key: DERBY-3178
>                 URL: https://issues.apache.org/jira/browse/DERBY-3178
>             Project: Derby
>          Issue Type: Test
>          Components: Test
>    Affects Versions: 10.4.0.0
>            Reporter: Mamta A. Satoor
>            Assignee: Mamta A. Satoor
>         Attachments: DERBY_3178_diff_ver1_Nov_06_2007.txt
>
>
> Derby has a junit test called DataSourceTest in jdbcapi package. This test registers ConnectionEventListener for various Connection objects but never verifies that events from those Connection objects are getting delivered to the listeners. Prior to this test's conversion to junit, there were System.out.println in the test but since this test is not cannon based anymore, those printlns have been commented out. I will soon submit a patch where this test will have asserts to make sure that Connection events are getting fired.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.