[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r26009 - trunk/subversion/tests/cmdline

From: Karl Fogel <kfogel_at_red-bean.com>
Date: 2007-08-09 17:58:34 CEST

kameshj@tigris.org writes:
> Log:
> Follow-up to r22257.
>
> Add 'switch_with_obstructing_local_adds' test to the 'test_list'.
>
> * subversion/tests/cmdline/switch_tests.py
> (test_list): Add 'switch_with_obstructing_local_adds' to test_list
>
> Patch by: Senthil Kumaran <senthil@collab.net>
>
> --- trunk/subversion/tests/cmdline/switch_tests.py (original)
> +++ trunk/subversion/tests/cmdline/switch_tests.py Thu Aug 9 04:17:28 2007
> @@ -1715,6 +1715,7 @@
> XFail(relocate_and_propset, svntest.main.is_ra_type_dav),
> forced_switch,
> forced_switch_failures,
> + switch_with_obstructing_local_adds,
> switch_scheduled_add,
> mergeinfo_switch_elision,
> ]

I think it might be good to always add new tests to the end of the
list. That way if there is an email thread referring "test 17" in
foo_tests.py, we will not affect the numbering and confuse that
thread.

Of course, in theory, we should always refer to tests by name anyway,
but sometimes people get careless. If there's no reason to add a test
to the middle of the list, the end seems like the best place to me.

Thoughts?

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Aug 10 02:56:40 2007

This is an archived mail posted to the Subversion Dev mailing list.