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)...]

With regards
Kamesh Jayachandran
-----Original Message-----
From: Prabhu Gnana Sundar Ponnarasu
Sent: Tue 2/8/2011 6:23 PM
To: Kamesh Jayachandran
Cc: dev_at_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
>
Received on 2011-02-08 15:31:00 CET