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

Re: [PATCH] New XFail test case for issue 3013

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Fri, 28 Jan 2011 08:38:29 +0200

Looks good, but I have a question:

Noorul Islam K M wrote on Wed, Jan 26, 2011 at 13:12:54 +0530:
>
> Attached is the python test for issue 3013. This incorporates the steps
> from the shell script attached in the tracker.
>
> Log
> [[[
>
> New XFail test for issue 3013.
>
> * subversion/tests/cmdline/update_tests.py
> (update_after_switching_to_deleted_path, test_list): New XFail test
>
> Patch by: Noorul Islam K M <noorul{_AT_}collab.net>
> ]]]
>
> Thanks and Regards
> Noorul
>

> Index: subversion/tests/cmdline/update_tests.py
> ===================================================================
> --- subversion/tests/cmdline/update_tests.py (revision 1063610)
> +++ subversion/tests/cmdline/update_tests.py (working copy)
> @@ -5347,6 +5347,34 @@
> svntest.main.run_svn(None, 'delete', os.path.join('A2', 'mu'))
> svntest.main.run_svn(None, 'update', os.path.join('A2', 'mu'))
>
> +### regression test for issue #3013
> +def update_after_switching_to_deleted_path(sbox):
> + "update after switching to deleted path"
> +
> + sbox.build()
> + wc_dir = sbox.wc_dir
> + repo_url = sbox.repo_url
> +
> + # switch to A/B
> + svntest.actions.run_and_verify_svn2(None, None, [], 0, 'switch',
> + repo_url + "/A/B", wc_dir)
> +
> + # delete A/D
> + svntest.actions.run_and_verify_svn2(None, None, [], 0, 'rm',
> + repo_url + "/A/D", '-m',
> + 'Remove A/D')
> +
> + # switch to A/D and this is known to fail
> + svntest.actions.run_and_verify_svn2(None, None, svntest.verify.AnyOutput,
> + 1, 'switch', repo_url + "/A/D", wc_dir)
> +
> + # switch to A/D_at_1 and this is known to succeed
> + svntest.actions.run_and_verify_svn2(None, None, [], 0, 'switch',
> + repo_url + "/A/D_at_1", wc_dir)
> +
> + # update should succeed
> + svntest.actions.run_and_verify_svn2(None, None, [], 0, "up", wc_dir)
> +

Should this 'update' succeed? In my testing, updating the wc root to
a revision it does not exist in fails.

> #######################################################################
> # Run the tests
>
> @@ -5412,6 +5440,7 @@
> update_with_excluded_subdir,
> XFail(update_with_file_lock_and_keywords_property_set),
> XFail(update_nonexistent_child_of_copy),
> + XFail(update_after_switching_to_deleted_path),
> ]
>
> if __name__ == '__main__':
Received on 2011-01-28 07:42:57 CET

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