Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: harmony-dev@incubator.apache.org
Received-SPF: neutral (asf.osuosl.org: 64.74.244.71 is neither permitted nor
 denied by domain of geir@pobox.com)
Message-ID: <44294BFD.1010705@pobox.com>
Date: Tue, 28 Mar 2006 16:45:17 +0200
From: Geir Magnusson Jr <geir@pobox.com>
Reply-To: geir@pobox.com
User-Agent: Thunderbird 1.5 (Windows/20051201)
MIME-Version: 1.0
To: harmony-dev@incubator.apache.org
Subject: Re: Regression tests (was: svn commit: r389475 - in
 /incubator/harmony/enhanced/classlib/trunk/modules/text/src:
 main/java/java/text/SimpleDateFormat.java
 test/java/tests/api/java/text/SimpleDateFormatTest.java)
References: <6e47b64f0603280346u3dc491edp8f06bacdbd04c626@mail.gmail.com>
 <fcb9f9160603280504o4b2a5bedu2051e24ad1521f7a@mail.gmail.com>
In-Reply-To: <fcb9f9160603280504o4b2a5bedu2051e24ad1521f7a@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit


Mark Hindess wrote:
> Are you expecting committers to do this or submitters?
> 
> Do we really want to raise the bar for submitters?

Yes, because a submitter is on his or her way to be committer.  One of 
the things I look for in potential committers is alignment with the 
other committers.

> 
> As a submitter, my current process is:
> 
> 0) find problem
> 1) create test
> 2) check test fails
> 3) fix code
> 4) check test passes
> 5) create patch
> 6) create jira
> 7) attach patch
> 
> Often I save steps 6 and 7 to submit a bunch of JIRA's in one go.
> It's going to be a pain to make this process:
> 
> 0) find problem
> 1) create test
> 2) check test fails
> 3) fix code
> 4) check test passes
> 5) create JIRA
> 6) edit test
> 7) check test still passes
> 8) create patch
> 9) attach patch
> 
> 
> With the new process I'm much more likely to get interrupted between
> the create and attach step. I think it is better to keep the
> "create/attach" JIRA steps together since then we avoid the
> possibility that someone sees the JIRA and thinks there is no patch
> forthcoming.  (Really I wish that JIRA create/attach could be an
> atomic operation when I have a patch ready.)
> 
> Do we really want to raise the bar for submitters?

Yep. :)

As Tim said, we won't reject because of this, but it sure would be nice.

geir

> 
> Regards,
>  Mark.
> 
> On 3/28/06, Stepan Mishura <stepan.mishura@gmail.com> wrote:
>> Hi,
>>
>> We agreed some time ago to mark regression tests with comments that includes
>> a reference to JIRA issue.
>>  There were no objections and I believe this is the good rule so I'd like to
>> ask everybody to follow it.
>> I've update testing wiki page to fix this agreement (see
>> http://wiki.apache.org/harmony/Testing_Convention)
>>
>> Thanks,
>> Stepan.
> 
> --
> Mark Hindess <mark.hindess@googlemail.com>
> IBM Java Technology Centre, UK.
> 
>