You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by David Glasser <gl...@davidglasser.net> on 2007/10/16 22:12:29 UTC
Re: svn commit: r27224 - trunk/subversion/tests/cmdline
On 10/16/07, lgo@tigris.org <lg...@tigris.org> wrote:
> Author: lgo
> Date: Tue Oct 16 14:06:11 2007
> New Revision: 27224
>
> Log:
> Add a test to indicate that switching to scheme://A/B/E at this time
> incorrectly requires read access on /A/B.
>
> * subversion/tests/cmdline/authz_tests.py
> (authz_switch_to_directory): new test.
> (test_list): run the new test, XFailing for ra_svn and ra_{neon/serf}.
>
>
> Modified:
> trunk/subversion/tests/cmdline/authz_tests.py
>
> Modified: trunk/subversion/tests/cmdline/authz_tests.py
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/tests/cmdline/authz_tests.py?pathrev=27224&r1=27223&r2=27224
> ==============================================================================
> --- trunk/subversion/tests/cmdline/authz_tests.py (original)
> +++ trunk/subversion/tests/cmdline/authz_tests.py Tue Oct 16 14:06:11 2007
> @@ -862,6 +862,25 @@
> 'checkout',
> D_url, D_path)
>
> +def authz_switch_to_directory(sbox):
> + "switched to directory, no read access on parents"
> +
> + sbox.build()
> +
> + write_authz_file(sbox, {"/": "*=rw", "/A/B": "*=", "/A/B/E": "jrandom = rw"})
> +
> + write_restrictive_svnserve_conf(sbox.repo_dir)
> +
> + wc_dir = sbox.wc_dir
> + mu_path = os.path.join(wc_dir, 'A', 'mu')
> + F_path = os.path.join(wc_dir, 'A', 'B', 'F')
> + G_path = os.path.join(wc_dir, 'A', 'D', 'G')
> +
> + # Switch /A/B/E to /A/B/F.
> + svntest.main.run_svn(None, 'switch', sbox.repo_url + "/A/B/E", G_path,
> + '--username', svntest.main.wc_author,
> + '--password', svntest.main.wc_passwd)
> +
> ########################################################################
> # Run the tests
>
> @@ -883,6 +902,8 @@
> Skip(authz_locking, svntest.main.is_ra_type_file),
> SkipUnless(authz_svnserve_anon_access_read,
> svntest.main.is_ra_type_svn),
> + XFail(Skip(authz_switch_to_directory,
> + svntest.main.is_ra_type_file)),
Did you really mean to write XFail(Skip( ... ?
--dave
> ]
>
> if __name__ == '__main__':
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: svn-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: svn-help@subversion.tigris.org
>
>
--
David Glasser | glasser@davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: svn commit: r27224 - trunk/subversion/tests/cmdline
Posted by sv...@mobsol.be.
Quoting David Glasser <gl...@davidglasser.net>:
> On 10/16/07, lgo@tigris.org <lg...@tigris.org> wrote:
> > Author: lgo
> > Date: Tue Oct 16 14:06:11 2007
> > New Revision: 27224
> >
> > Log:
> > Add a test to indicate that switching to scheme://A/B/E at this time
> > incorrectly requires read access on /A/B.
> >
> > * subversion/tests/cmdline/authz_tests.py
> > (authz_switch_to_directory): new test.
> > (test_list): run the new test, XFailing for ra_svn and ra_{neon/serf}.
> >
[..]
> > @@ -883,6 +902,8 @@
> > Skip(authz_locking, svntest.main.is_ra_type_file),
> > SkipUnless(authz_svnserve_anon_access_read,
> > svntest.main.is_ra_type_svn),
> > + XFail(Skip(authz_switch_to_directory,
> > + svntest.main.is_ra_type_file)),
>
> Did you really mean to write XFail(Skip( ... ?
Yes I did. It skips the test for ra_local, fails in all other cases. Do you see
any issue with that?
Lieven
----------------------------------------------------------------
This message was sent using IMP, the Internet Messaging Program.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org