db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Knut Anders Hatlen (JIRA)" <derby-...@db.apache.org>
Subject [jira] Commented: (DERBY-1893) Convert largedata/lobLengthTests.java to junit
Date Mon, 16 Oct 2006 19:36:36 GMT
    [ http://issues.apache.org/jira/browse/DERBY-1893?page=comments#action_12442681 ] 
Knut Anders Hatlen commented on DERBY-1893:

I have two comments to the patch (sorry I didn't notice before it was committed):

 - the suite method should have used suite.addTestSuite(LobLengthTest.class) instead of suite.addTest(new
LobLengthTest("testXXX")). The way it is now, a new test case won't run automatically, and
one might be led to belive that it succeeded when in fact it did not run.

 - the tearDown() method should set conn to null and call super.tearDown(). Tests not doing
this have caused OutOfMemoryErrors before (DERBY-1845 and DERBY-1910). (It would be even better
if the test class didn't have its own instance variable for the connection since the BaseJDBCTestClass
already has many helper methods which remove the need for an explicit pointer to the connection.)

> Convert largedata/lobLengthTests.java to junit
> ----------------------------------------------
>                 Key: DERBY-1893
>                 URL: http://issues.apache.org/jira/browse/DERBY-1893
>             Project: Derby
>          Issue Type: Sub-task
>          Components: Test
>            Reporter: Øystein Grøvlen
>         Assigned To: Øystein Grøvlen
>            Priority: Trivial
>         Attachments: derby1893.diff, derby1893.stat, derby1893_v2.diff, derby1893_v2.stat

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


View raw message