db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mamta A. Satoor (JIRA)" <j...@apache.org>
Subject [jira] Updated: (DERBY-3178) DataSourceTest uses ConnectionEventListener but never verifies that events are getting fired
Date Tue, 06 Nov 2007 21:18:51 GMT

     [ 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.


Mime
View raw message