The vtest contributors get credit for this pattern! :-) We also have a Ruby script that scans the test source and, among other things, lists ignored tests and the associated blocking Jiras.
On Wed, Jun 11, 2008 at 11:49 AM, Simon Laws <[EMAIL PROTECTED]> wrote: > On Wed, Jun 11, 2008 at 6:43 PM, Vamsavardhana Reddy <[EMAIL PROTECTED]> > wrote: > >> Thanks Raymond for pointing this out. We never used this strategy in >> Geronimo (infact, I have never put any JIRA number in any of the source >> files other that Release Notes may be). I have come across this in Tuscany >> source code and have been following it consistently from then on. Should >> give credit to who ever started this in Tuscany :) >> >> ++Vamsi >> >> On Wed, Jun 11, 2008 at 11:02 PM, Raymond Feng <[EMAIL PROTECTED]> >> wrote: >> >> > Hi, >> > >> > I see Vamsi uses the following strategy to disable failing unit test >> cases. >> > >> > @Test >> > @Ignore("TUSCANY-xxxx") // Ignore the test case due to JIRA TUSCANY-xxxx >> > public void testMySrtuff() { >> > } >> > >> > I think it's a very good practice we should follow to maintain a clean >> > build while keep track of the ignored failing test cases with JIRAs. >> > >> > Thanks, >> > Raymond >> > >> > > +1 good idea > > Simon >