Re: [PATCH] New XFail test for the issue 3781

On Thursday 03 February 2011 05:23 PM, Noorul Islam K M wrote:
> Prabhu Gnana Sundar<prabhugs_at_collab.net> writes:
>
>> Hi all,
>>
>> Currently, as per the issue 3781, "checkout" reads the authz file in
>> *case insensitive* way, whereas "commit" reads the authz file in *case
>> sensitive* way.
>>
>> Here is what is observed:
>> 1. Checkout is *case insensitive* about the repository name section
>> and also the path-inside-repo section.
>> 2. Commit is *case sensitive* about the repository name section *but
>> case insensitive* about the path-inside-repo section.
>>
>> Attached an XFail testcase and the log message with this mail. Please
>> share your views.
>>
>>
>>
>>
>> Thanks and regards
>> Prabhu
>>
>> [[[
>> XFail testcase for the issue 3781
>>
>> * subversion/tests/cmdline/authz_tests.py
>> (case_insensitive_authz, test_list): New XFail test for the issue 3781
>>
>> Patch by: Prabhu Gnana Sundar<prabhugs{_AT_}collab.net>
>> Suggested by: Kamesh Jayachandran<kamesh{_AT_}collab.net>
>>
>> Index: subversion/tests/cmdline/authz_tests.py
>> ===================================================================
>> --- subversion/tests/cmdline/authz_tests.py (revision 1066732)
>> +++ subversion/tests/cmdline/authz_tests.py (working copy)
>> @@ -1061,6 +1061,67 @@
>> [], 'ls', '-R',
>> sbox.repo_url)
>>
>> +
>> +def case_insensitive_authz(sbox):
>> + "authz issue #3781, check case insensitiveness"
>> +
>> + sbox.build()
>> +
>> + # test the case-insensitivity of the path inside the repo
>> + write_authz_file(sbox, {"/": "jrandom = r", "/A": "jrandom = r", "/a/Mu": "jrandom = rw"})
>> +
> I am not sure whether this is relevant. I think for consistency lines
> should be of 79 characters.

wrapped the text for the consistency :)
Attached the patch with this mail...
Thanks Noorul for your feedback over the "indentation".

Thanks and regards
Prabhu