Return-Path: Delivered-To: apmail-incubator-harmony-dev-archive@www.apache.org Received: (qmail 42279 invoked from network); 12 Jan 2006 12:38:47 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 12 Jan 2006 12:38:47 -0000 Received: (qmail 76438 invoked by uid 500); 12 Jan 2006 12:38:32 -0000 Delivered-To: apmail-incubator-harmony-dev-archive@incubator.apache.org Received: (qmail 76257 invoked by uid 500); 12 Jan 2006 12:38:31 -0000 Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: harmony-dev@incubator.apache.org Delivered-To: mailing list harmony-dev@incubator.apache.org Received: (qmail 76124 invoked by uid 99); 12 Jan 2006 12:38:30 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 12 Jan 2006 04:38:30 -0800 X-ASF-Spam-Status: No, hits=1.4 required=10.0 tests=ADVANCE_FEE_1,SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (asf.osuosl.org: 64.74.244.71 is neither permitted nor denied by domain of geir@pobox.com) Received: from [64.74.244.71] (HELO chi.mobile-health-diary.com) (64.74.244.71) by apache.org (qpsmtpd/0.29) with SMTP; Thu, 12 Jan 2006 04:38:30 -0800 Received: (qmail 22414 invoked from network); 12 Jan 2006 12:38:07 -0000 Received: from vbn.0050556.lodgenet.net (HELO ?10.2.1.146?) (geir@216.142.194.234) by b014.internal.mobile-health-diary.com with SMTP; 12 Jan 2006 12:38:07 -0000 Message-ID: <43C64DAD.4080608@pobox.com> Date: Thu, 12 Jan 2006 04:38:05 -0800 From: Geir Magnusson Jr Reply-To: geir@pobox.com User-Agent: Mozilla Thunderbird 1.0.7 (Windows/20050923) X-Accept-Language: en-us, en MIME-Version: 1.0 To: harmony-dev@incubator.apache.org Subject: Re: regression test suite References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed 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 George Harley1 wrote: > Hi, > > >>I think that regression tests should be marked in some way. > > > Agreed. But can we please *resist* the temptation to do this by > incorporating JIRA issue numbers into test case names (e.g. calling unit > test methods test_26() or test_JIRA_26()). I've seen this kind of approach > adopted in a couple of projects and, in my experience, it often leads to > the scattering of duplicated test code around the test harness. > > Better, methinks, to either create a new test method with a meaningful > name or else augment an existing method - whatever makes more sense for > the particular issue. Then marking certain code as being for regression > test purposes could be done in comments that include the URL of the JIRA > issue. Perhaps an agreed tag like "JIRA" or "BUG" etc could be used as an > eye-catcher as well ? > e.g. > > // BUG http://issues.apache.org/jira/browse/HARMONY-26 This topic really has gained a life of it's own. How about just a comment, like you say? geir > > > My 2 Euro Cents. > > Best regards, > George > ________________________________________ > George C. Harley > > > > > "Mishura, Stepan M" > 12/01/2006 04:56 > Please respond to > harmony-dev@incubator.apache.org > > > To > > cc > > Subject > RE: regression test suite > > > > > > > Hello, > > Tim Ellison wrote: > > [snip] > >>What is the useful distinction for regression tests being kept > > separate? > >>I can see that you may distinguish unit and 'system-level' tests just >>because of the difference in frameworks etc. required, but why do I > > care > >>if the test was written due to a JIRA issue or test-based development > > or > >>someone who get's kicks out of writing tests to break the code? >> > > > I agree that separating regression tests doesn't make sense. > However I think that regression tests should be marked in some way. > This will signal a developer that a test was created to track already > known issue. IMHO, a regression test should point out to a bug report > and a bug report (after resolving a bug) should contain a reference to > corresponding regression test in repository. > > Thanks, > Stepan Mishura > Intel Middleware Products Division > > >