Re: Proposal: Use decorators in test suite for XFail, Skip, etc

On 02/04/2011 02:09 PM, Greg Stein wrote:
> On Fri, Feb 4, 2011 at 12:15, Hyrum K Wright <hyrum_at_hyrumwright.org> wrote:
>> ...
>> We currently mark tests XFail (or Skip, or something else) by wrapping
>> them in the test_list in the test suite. Rather than doing it there,
>> I think it makes more sense to use Python's decorator syntax to mark
>> tests as XFail right at their definition, rather than down in the test
>> list. Keeping all attributes of a test in close proximity is a Good
>> Thing, imo. Attached is a patch which demonstrates this.
>
> Sure.
>
>> ...
>> +++ subversion/tests/cmdline/basic_tests.py (working copy)
>
> XFail = svntest.testcase.xfail_deco
>
>> @@ -1961,6 +1961,7 @@
>> expected_status)
>>
>> # Test for issue #1199
>> +@svntest.testcase.xfail_deco
>
> @XFail

Oh yes. Much, much nicer to read.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand