[PATCH] regression test for issue #876

[vi...@collab.net]

[[[
    Add a regression test for issue #876
* subversion/tests/clients/cmdline/revert_tests.py
  (move_revert): A new function to test a moved and
  then reverted file.
]]]

Re: [PATCH] regression test for issue #876

[Branko Čibej <br...@xbc.nu>]

vivek@collab.net wrote:

>[[[
>    Add a regression test for issue #876
>* subversion/tests/clients/cmdline/revert_tests.py
>  (move_revert): A new function to test a moved and
>  then reverted file.
>]]]
>  
>
[snip]

>+    #after resolving this issue just comment out
>+    #the above two lines,and the test would work fine.
>  
>
[snip]

>+              move_revert,
>
This is /not/ the way to write a test that tickles a known bug. The 
thing to do is to write the test so that it expects things to work 
correctly, then to mark it as XFAIL. Once the bug is fixed, the test 
will pass with an XPASS status, which is intended to alert whoever fixed 
the bug that they have to update the test's status, too.

-- Brane


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org