You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Prabhu Gnana Sundar <pr...@collab.net> on 2011/02/08 08:41:03 UTC
[PATCH] New XFail test for issue 3781
Hi Kamesh,
I have created a new thread in order to prevent the confusion about the
test case. I have tweaked the test case as you mentioned in the previous
thread.
Here is the link for the same...
http://mail-archives.apache.org/mod_mbox/subversion-dev/201102.mbox/%3C0213965108DEAD48960CE83455E07DFF0192BE04@maa-exchmb.maa.corp.collab.net%3E
Thank you for your valuable suggestions.
I hope this test case would make things a bit more clear.
I have attached the patch and the log message with this mail. Please
share your views.
Thanks and regards
Prabhu
RE: [PATCH] New XFail test for issue 3781
Posted by Kamesh Jayachandran <ka...@collab.net>.
Thanks Prabhu committed your patch in r1068802.
With regards
Kamesh Jayachandran
-----Original Message-----
From: Prabhu Gnana Sundar Ponnarasu
Sent: Wed 2/9/2011 2:30 PM
To: Bert Huijben
Cc: Kamesh Jayachandran; dev@subversion.apache.org
Subject: Re: [PATCH] New XFail test for issue 3781
Hi Bert,
The problem was that my patch handled the expected error more
specifically and also failed to handle the svnserve case. That was wrong
on my part. I have tweaked the test case and attached a patch and log
message with this mail.
Please share your views on the same.
On Tuesday 08 February 2011 09:08 PM, Bert Huijben wrote:
> This test XFails for the wrong reason on svn://.
>
> EXPECTED STDERR (regexp):
> svn: E175013: Unable to connect to a repository at URL
> 'svn://localhost/svn-test-work/repositories/authz_tests-20/A/mu'
> svn: E175013: Access to '/svn-test-work\repositories\authz_tests-20/A/mu'
> forbidden
> svn: E175009: XML parsing failed: (403 Forbidden)
> ACTUAL STDERR:
> ..\..\..\subversion\svn\cat-cmd.c:81: (apr_err=170001)
> ..\..\..\subversion\svn\util.c:967: (apr_err=170001)
> ..\..\..\subversion\libsvn_client\cat.c:233: (apr_err=170001)
> ..\..\..\subversion\libsvn_ra_svn\client.c:1557: (apr_err=170001)
> ..\..\..\subversion\libsvn_ra_svn\client.c:242: (apr_err=170001)
> ..\..\..\subversion\svnserve\serve.c:174: (apr_err=170001)
> svn: E170001: Authorization failed
> EXCEPTION: SVNUnmatchedError
>
> See http://subversion.apache.org/buildbot/ or more specifically
> http://ci.apache.org/builders/svn-slik-w2k3-x64-ra/builds/1675/steps/Test%20
> fsfs%2Bsvn/logs/testlog
>
> Bert
>
Thanks and regards
Prabhu
Re: [PATCH] New XFail test for issue 3781
Posted by Prabhu Gnana Sundar <pr...@collab.net>.
Hi Bert,
The problem was that my patch handled the expected error more
specifically and also failed to handle the svnserve case. That was wrong
on my part. I have tweaked the test case and attached a patch and log
message with this mail.
Please share your views on the same.
On Tuesday 08 February 2011 09:08 PM, Bert Huijben wrote:
> This test XFails for the wrong reason on svn://.
>
> EXPECTED STDERR (regexp):
> svn: E175013: Unable to connect to a repository at URL
> 'svn://localhost/svn-test-work/repositories/authz_tests-20/A/mu'
> svn: E175013: Access to '/svn-test-work\repositories\authz_tests-20/A/mu'
> forbidden
> svn: E175009: XML parsing failed: (403 Forbidden)
> ACTUAL STDERR:
> ..\..\..\subversion\svn\cat-cmd.c:81: (apr_err=170001)
> ..\..\..\subversion\svn\util.c:967: (apr_err=170001)
> ..\..\..\subversion\libsvn_client\cat.c:233: (apr_err=170001)
> ..\..\..\subversion\libsvn_ra_svn\client.c:1557: (apr_err=170001)
> ..\..\..\subversion\libsvn_ra_svn\client.c:242: (apr_err=170001)
> ..\..\..\subversion\svnserve\serve.c:174: (apr_err=170001)
> svn: E170001: Authorization failed
> EXCEPTION: SVNUnmatchedError
>
> See http://subversion.apache.org/buildbot/ or more specifically
> http://ci.apache.org/builders/svn-slik-w2k3-x64-ra/builds/1675/steps/Test%20
> fsfs%2Bsvn/logs/testlog
>
> Bert
>
Thanks and regards
Prabhu
RE: [PATCH] New XFail test for issue 3781
Posted by Bert Huijben <be...@qqmail.nl>.
> -----Original Message-----
> From: Kamesh Jayachandran [mailto:kamesh@collab.net]
> Sent: dinsdag 8 februari 2011 15:30
> To: Prabhu Gnana Sundar Ponnarasu
> Cc: dev@subversion.apache.org
> Subject: RE: [PATCH] New XFail test for issue 3781
>
>
> Thanks Prabhu.
>
> I committed with the following tweaks in r1068411.
>
> 1. Added @XFail(), @Issue(3781) decorators
> 2.
>
> + # test the case-sensitivity of the repo name
> + write_authz_file(sbox, {},
> + sections = {mixed_case_repo_dir + ":/": "jrandom =
> r",
> + mixed_case_repo_dir + ":/A": "jrandom =
> r",
> + sbox.repo_dir + ":/A/mu": "jrandom =",
> + mixed_case_repo_dir + ":/A/mu":
> "jrandom = rw"})
>
> Replaced this snippet by the following snippet.
> + # test the case-sensitivity of the repo name
> + sec_mixed_case = {mixed_case_repo_dir + ":/": "jrandom = r",
> + mixed_case_repo_dir + ":/A": "jrandom = r",
> + os.path.basename(sbox.repo_dir) + ":/A/mu":
> "jrandom =",
> + mixed_case_repo_dir + ":/A/mu": "jrandom = rw"}
> + write_authz_file(sbox, {}, sec_mixed_case)
>
>
> You can reduce this to simpler one as most of the time same 'cat' and
> 'commit' is tested with one 'for' loop
> which iterates over [(rule1, section1), (rule2, section2)...]
This test XFails for the wrong reason on svn://.
EXPECTED STDERR (regexp):
svn: E175013: Unable to connect to a repository at URL
'svn://localhost/svn-test-work/repositories/authz_tests-20/A/mu'
svn: E175013: Access to '/svn-test-work\repositories\authz_tests-20/A/mu'
forbidden
svn: E175009: XML parsing failed: (403 Forbidden)
ACTUAL STDERR:
..\..\..\subversion\svn\cat-cmd.c:81: (apr_err=170001)
..\..\..\subversion\svn\util.c:967: (apr_err=170001)
..\..\..\subversion\libsvn_client\cat.c:233: (apr_err=170001)
..\..\..\subversion\libsvn_ra_svn\client.c:1557: (apr_err=170001)
..\..\..\subversion\libsvn_ra_svn\client.c:242: (apr_err=170001)
..\..\..\subversion\svnserve\serve.c:174: (apr_err=170001)
svn: E170001: Authorization failed
EXCEPTION: SVNUnmatchedError
See http://subversion.apache.org/buildbot/ or more specifically
http://ci.apache.org/builders/svn-slik-w2k3-x64-ra/builds/1675/steps/Test%20
fsfs%2Bsvn/logs/testlog
Bert
RE: [PATCH] New XFail test for issue 3781
Posted by Kamesh Jayachandran <ka...@collab.net>.
Thanks Prabhu.
I committed with the following tweaks in r1068411.
1. Added @XFail(), @Issue(3781) decorators
2.
+ # test the case-sensitivity of the repo name
+ write_authz_file(sbox, {},
+ sections = {mixed_case_repo_dir + ":/": "jrandom = r",
+ mixed_case_repo_dir + ":/A": "jrandom = r",
+ sbox.repo_dir + ":/A/mu": "jrandom =",
+ mixed_case_repo_dir + ":/A/mu": "jrandom = rw"})
Replaced this snippet by the following snippet.
+ # test the case-sensitivity of the repo name
+ sec_mixed_case = {mixed_case_repo_dir + ":/": "jrandom = r",
+ mixed_case_repo_dir + ":/A": "jrandom = r",
+ os.path.basename(sbox.repo_dir) + ":/A/mu": "jrandom =",
+ mixed_case_repo_dir + ":/A/mu": "jrandom = rw"}
+ write_authz_file(sbox, {}, sec_mixed_case)
You can reduce this to simpler one as most of the time same 'cat' and 'commit' is tested with one 'for' loop
which iterates over [(rule1, section1), (rule2, section2)...]
With regards
Kamesh Jayachandran
-----Original Message-----
From: Prabhu Gnana Sundar Ponnarasu
Sent: Tue 2/8/2011 6:23 PM
To: Kamesh Jayachandran
Cc: dev@subversion.apache.org
Subject: Re: [PATCH] New XFail test for issue 3781
Hi,
Sorry for posting an older patch. Now attached the correct patch.
Please share your views.
Thanks and regards
Prabhu
On Tuesday 08 February 2011 04:41 PM, Kamesh Jayachandran wrote:
>
> Prabhu,
>
>
> Please send the patch against the current HEAD.
>
> With regards
> Kamesh Jayachandran
>
Re: [PATCH] New XFail test for issue 3781
Posted by Prabhu Gnana Sundar <pr...@collab.net>.
Hi,
Sorry for posting an older patch. Now attached the correct patch.
Please share your views.
Thanks and regards
Prabhu
On Tuesday 08 February 2011 04:41 PM, Kamesh Jayachandran wrote:
>
> Prabhu,
>
>
> Please send the patch against the current HEAD.
>
> With regards
> Kamesh Jayachandran
>
RE: [PATCH] New XFail test for issue 3781
Posted by Kamesh Jayachandran <ka...@collab.net>.
Prabhu,
Please send the patch against the current HEAD.
With regards
Kamesh Jayachandran
-----Original Message-----
From: Prabhu Gnana Sundar Ponnarasu
Sent: Tue 2/8/2011 1:11 PM
To: Kamesh Jayachandran
Cc: dev@subversion.apache.org
Subject: [PATCH] New XFail test for issue 3781
Hi Kamesh,
I have created a new thread in order to prevent the confusion about the
test case. I have tweaked the test case as you mentioned in the previous
thread.
Here is the link for the same...
http://mail-archives.apache.org/mod_mbox/subversion-dev/201102.mbox/%3C0213965108DEAD48960CE83455E07DFF0192BE04@maa-exchmb.maa.corp.collab.net%3E
Thank you for your valuable suggestions.
I hope this test case would make things a bit more clear.
I have attached the patch and the log message with this mail. Please
share your views.
Thanks and regards
Prabhu