From depot-dev-return-811-apmail-incubator-depot-dev-archive=incubator.apache.org@incubator.apache.org Thu Jul 29 14:13:16 2004 Return-Path: Delivered-To: apmail-incubator-depot-dev-archive@www.apache.org Received: (qmail 41387 invoked from network); 29 Jul 2004 14:13:15 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur-2.apache.org with SMTP; 29 Jul 2004 14:13:15 -0000 Received: (qmail 57251 invoked by uid 500); 29 Jul 2004 14:12:57 -0000 Delivered-To: apmail-incubator-depot-dev-archive@incubator.apache.org Received: (qmail 57142 invoked by uid 500); 29 Jul 2004 14:12:56 -0000 Mailing-List: contact depot-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: list-post: Reply-To: "Depot Development" List-Id: Depot Development Delivered-To: mailing list depot-dev@incubator.apache.org Received: (qmail 57048 invoked by uid 99); 29 Jul 2004 14:12:55 -0000 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received: from [130.214.10.19] (HELO mx1.try.sybase.com) (130.214.10.19) by apache.org (qpsmtpd/0.27.1) with ESMTP; Thu, 29 Jul 2004 07:12:53 -0700 Received: from mail.try.sybase.com (mail.try.sybase.com [130.214.10.18]) by mx1.try.sybase.com (8.12.11/8.12.11) with ESMTP id i6TEBCTN031691 for ; Thu, 29 Jul 2004 08:11:12 -0600 Received: from tsws1 ([10.22.120.45]) by mail.try.sybase.com (8.12.11/8.12.11) with SMTP id i6TEAs3w012297 for ; Thu, 29 Jul 2004 08:10:56 -0600 Message-ID: <0ee101c47576$1d4a8430$14c5ea43@sybase.com> From: "Adam R. B. Jack" To: "Depot Development" References: <0e9401c47572$fca611c0$14c5ea43@sybase.com> <30782.1091109319@www52.gmx.net> Subject: Re: Deleting stuff Date: Thu, 29 Jul 2004 08:09:50 -0600 MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit X-Priority: 3 X-MSMail-Priority: Normal X-Mailer: Microsoft Outlook Express 6.00.2800.1437 X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2800.1441 X-Virus-Checked: Checked X-Spam-Rating: minotaur-2.apache.org 1.6.2 0/1000/N You mean the statement that the work was really boring? ;-) ;-) Ok, I've recovered it now. And yes, I'd love to see a better way to create mock objects. Right now the MockRepository (and some unit tests) need to create these objects, so -- so long as we can find a nice way to do that [preferably under src/test, and perhaps not shipped w/ releases], I'm definitely +1 to see the statics removed the core classes. BTW: Sorry if I was crabby, I'm frustrated that I'm not getting this back into shape quicker/sooner. I know we are close, I know we have a good foundation, I think it is just hidden behind some confusion. I like your Javadoc comments, and I'll try to add to those. I'm hoping we are 'clean' within no more than a week, and we -- total group -- understand the codebase we have, and how it works. That is the main short term goal, IMHO. Also, I think it is key that we get green and stay green -- not having working unit tests means we don't know when we delete/break something important. For me (in Eclipse) I couldn't run the Ant unit tests, but I think I now can. I'll work to get the tests working, and then we can rest easier that we can hack w/o breaking (undetected). regards, Adam ----- Original Message ----- From: "Markus M. May" To: "Depot Development" Sent: Thursday, July 29, 2004 7:55 AM Subject: Re: Deleting stuff > Hmm, this was the testing stuff I was working on currently. Sorry for the > delay :-( > I do prefer the creation of the testing objects in the test class. And I > definitly agree on your statement in the first mail :-) > > > Greetz > > Markus > > > Ok, I see that viewcvs can help me find old code. I can go borrow it > > again. > > > > Markus, the if the getTextXXXX stuff is ugly I'm game to change it to a > > better set of Mock objects (I just couldn't think of a better way at the > > time). I'll add them back (for now) but then happily work w/ folks to > > clean > > them out of important classes and get them into better places. > > > > regards > > > > Adam > > ----- Original Message ----- > > From: "Adam R. B. Jack" > > To: "Depot Dev" > > Sent: Thursday, July 29, 2004 7:40 AM > > Subject: Deleting stuff > > > > > > > Folks > > > > > > >From now on can we discuss something before deleting it? I know we had > > some > > > baggage, but I don't think we have much/any left (in Updater, and > > Version > > > I'd like to keep as my problem please). > > > > > > Right now I am trying to get some key unit tests working and I am > > finding > > > that the main Artifact*.getTestXXXX static helper methods appear > > missing. > > I > > > spent a lot of really boring time adding those, and the prospect of > > doing > > it > > > again is seriously disheartening. (SVN is, IMHO, a pain in the rear to > > > recover from, and I don't plan on trying to guess old version numbers to > > > copy over and manually merge from). > > > > > > Please, although I doubt anything more is remaining to be snipped, > > please > > > let's discuss twice, cut once. > > > > > > regards > > > > > > Adam > > > -- > > > Experience the Unwired Enterprise: > > > http://www.sybase.com/unwiredenterprise > > > Try Sybase: http://www.try.sybase.com > >