[ http://issues.apache.org/jira/browse/DERBY-1555?page=all ]
Kristian Waagan updated DERBY-1555:
-----------------------------------
Attachment: derby-1555-removecomment.diff
I don't think this comment in BaseJDBCTestCase is valid anymore, and can be
removed:
// TEMP NAME - WILL BE getConnection() once all uses of the
// static getConnection() have been converted to openDefaultConnection
As far as I can see, the getXConnection has been renamed to getConnection.
I have made a little patch :)
> API for configuration information in Derby's JUnit tests should through
> instance methods of the base classes, not static methods and static fields
> --------------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: DERBY-1555
> URL: http://issues.apache.org/jira/browse/DERBY-1555
> Project: Derby
> Issue Type: Improvement
> Components: Test
> Reporter: Daniel John Debrunner
> Assigned To: Daniel John Debrunner
> Attachments: derby-1555-removecomment.diff
>
>
> The heavy use of static state information in the existing Derby (non-Junit)
> harness was a major mistake, it should not be repeated in the Junit model.
> It should be changed before too many tests are written.
> A static configuration will limit the ability for the tests to be flexible
> and have different instances running with different configurations,
> a instance based scheme will be more flexible.
> This change will just change the api, the current implementation of a single
> static configuration will remain but be hidden within the base-classes.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
