Return-Path: Delivered-To: apmail-db-derby-dev-archive@www.apache.org Received: (qmail 1007 invoked from network); 16 Aug 2006 11:25:36 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 16 Aug 2006 11:25:36 -0000 Received: (qmail 29695 invoked by uid 500); 16 Aug 2006 11:25:22 -0000 Delivered-To: apmail-db-derby-dev-archive@db.apache.org Received: (qmail 29657 invoked by uid 500); 16 Aug 2006 11:25:21 -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 29636 invoked by uid 99); 16 Aug 2006 11:25:21 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 16 Aug 2006 04:25:21 -0700 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests=UNPARSEABLE_RELAY X-Spam-Check-By: apache.org Received-SPF: pass (asf.osuosl.org: local policy) Received: from [192.18.98.43] (HELO brmea-mail-2.sun.com) (192.18.98.43) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 16 Aug 2006 04:25:19 -0700 Received: from phys-epost-1 ([129.159.136.14]) by brmea-mail-2.sun.com (8.13.6+Sun/8.12.9) with ESMTP id k7GBOw8f025148 for ; Wed, 16 Aug 2006 05:24:58 -0600 (MDT) Received: from conversion-daemon.epost-mail1.sweden.sun.com by epost-mail1.sweden.sun.com (iPlanet Messaging Server 5.2 HotFix 1.24 (built Dec 19 2003)) id <0J4300H018YZPQ@epost-mail1.sweden.sun.com> (original mail from Andreas.Korneliussen@Sun.COM) for derby-dev@db.apache.org; Wed, 16 Aug 2006 13:24:58 +0200 (MEST) Received: from [129.159.112.247] (khepri35.Norway.Sun.COM [129.159.112.247]) by epost-mail1.sweden.sun.com (iPlanet Messaging Server 5.2 HotFix 1.24 (built Dec 19 2003)) with ESMTPA id <0J4300AO791LVR@epost-mail1.sweden.sun.com> for derby-dev@db.apache.org; Wed, 16 Aug 2006 13:24:57 +0200 (MEST) Date: Wed, 16 Aug 2006 13:24:57 +0200 From: Andreas Korneliussen Subject: Re: [junit] Move JUnit base/utility classes??? In-reply-to: <44E237B7.60206@sun.com> To: derby-dev@db.apache.org Reply-to: Andreas.Korneliussen@Sun.COM Message-id: <44E30089.9010802@sun.com> Organization: Sun Microsystems MIME-version: 1.0 Content-type: text/plain; charset=ISO-8859-1; format=flowed Content-transfer-encoding: 7BIT User-Agent: Thunderbird 1.5.0.2 (X11/20060427) References: <44E21B61.8060402@apache.org> <44E237B7.60206@sun.com> X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N David Van Couvering wrote: > Sounds good to me... > > David > > Daniel John Debrunner wrote: >> Currently the JUnit base and utility classes are in this package: >> >> org.apache.derbyTesting.functionTests.util >> >> (See http://wiki.apache.org/db-derby/DerbyJUnitTesting) >> >> I was wondering if they should be moved, for two reasons: >> >> 1) That package is cluttered up with other stuff, it's more or less a >> dumping ground. Utilites, JUnit base classes, "user level" classes for >> procedures and vtis, etc. >> >> 2) JUnit tests can be much more than funcional tests, e.g. having >> system tests as JUnit tests would make them easy for everyone to run. >> >> I was thinking of the following package: >> >> org.apache.derbyTesting.junit >> >> The package would be limited to base classes for JUnit tests and JUnit >> related utilities such as the JDBC class. Classes for specific tests, or >> those that implement Java procedures for tests etc. would not be allowed. >> >> The functional tests would continue to live in their current location, >> just that the super-class BaseJDBCTestCase would be in the new package. >> >> Thoughts? I support moving the utility classes to a new package. Below are some thoughts on how Junit tests could be structured, in case we would like to also move tests: Package for derby-specific utility classes: org.apache.derbyTesting.junit.util -> TestConfiguration, JDBCClient, JDBC,.. Package for common testcase/testsetup classes: org.apache.derbyTesting.junit.common -> BaseTestCase, BaseJDBCTestCase, BaseJDBCTestSetup Packages for tests: org.apache.derbyTesting.junit.tests org.apache.derbyTesting.junit.tests.jdbcapi org.apache.derbyTesting.junit.tests.lang org.apache.derbyTesting.junit.tests.store org.apache.derbyTesting.junit.tests. . . Andreas