Return-Path: Delivered-To: apmail-db-derby-dev-archive@www.apache.org Received: (qmail 32292 invoked from network); 24 Aug 2006 20:19:11 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 24 Aug 2006 20:19:11 -0000 Received: (qmail 85483 invoked by uid 500); 24 Aug 2006 20:19:00 -0000 Delivered-To: apmail-db-derby-dev-archive@db.apache.org Received: (qmail 85439 invoked by uid 500); 24 Aug 2006 20:19:00 -0000 Mailing-List: contact derby-dev-help@db.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: Delivered-To: mailing list derby-dev@db.apache.org Received: (qmail 85242 invoked by uid 99); 24 Aug 2006 20:18:59 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 24 Aug 2006 13:18:59 -0700 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received: from [209.237.227.198] (HELO brutus.apache.org) (209.237.227.198) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 24 Aug 2006 13:18:56 -0700 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 3A2F841002F for ; Thu, 24 Aug 2006 20:15:47 +0000 (GMT) Message-ID: <8838618.1156450547229.JavaMail.jira@brutus> Date: Thu, 24 Aug 2006 13:15:47 -0700 (PDT) From: "Kristian Waagan (JIRA)" To: derby-dev@db.apache.org Subject: [jira] Updated: (DERBY-1555) API for configuration information in Derby's JUnit tests should through instance methods of the base classes, not static methods and static fields In-Reply-To: <5310598.1153434253931.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N [ 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