Return-Path: Delivered-To: apmail-incubator-geronimo-dev-archive@incubator.apache.org Received: (qmail 89531 invoked by uid 500); 16 Aug 2003 17:18:28 -0000 Mailing-List: contact geronimo-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: list-post: Reply-To: geronimo-dev@incubator.apache.org Delivered-To: mailing list geronimo-dev@incubator.apache.org Received: (qmail 89488 invoked from network); 16 Aug 2003 17:18:26 -0000 Received: from unknown (HELO reason.planet57.com) (202.183.249.230) by daedalus.apache.org with SMTP; 16 Aug 2003 17:18:26 -0000 Received: from coredevelopers.net (localhost [127.0.0.1]) by reason.planet57.com (Postfix) with ESMTP id 1C7F9374B00 for ; Sun, 17 Aug 2003 00:18:49 +0700 (ICT) Date: Sun, 17 Aug 2003 00:18:48 +0700 Subject: Re: [JSR-88] Recent Repackaging Content-Type: text/plain; charset=US-ASCII; format=flowed Mime-Version: 1.0 (Apple Message framework v552) From: Jason Dillon To: geronimo-dev@incubator.apache.org Content-Transfer-Encoding: 7bit In-Reply-To: Message-Id: X-Mailer: Apple Mail (2.552) X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N I think I have done that, double checking that a clean checkout functions. I think there was only one test which depended upon core (DeploymentFactoryManagerTest). If we want to test an impl then the core should create a test which extends the spec test and overrides the setup method to use an impl object instead of the mock. --jason On Sunday, August 17, 2003, at 01:30 AM, Aaron Mulder wrote: > On Sat, 16 Aug 2003, Jason Dillon wrote: >> I do not think they should be removed, the mock objects removed the >> dependency on j2ee-deployment test from the core module. > > I think you have that backward, because now j2ee-deployment/test > depends on the core module. And if you want to keep the Mock objects, > then we need to revert some of the tests, because they currently use > the > real objects. We'd probably also want to enhance the mock objects a > bit. > > Aaron >