Re: svn commit: r19732 - trunk/subversion/tests/cmdline

On Mon, 22 May 2006 02:26:50 +0530, <lgo@tigris.org> wrote:

> Author: lgo
> Date: Sun May 21 13:56:49 2006
> New Revision: 19732
>
> Modified:
> trunk/subversion/tests/cmdline/authz_tests.py
>
> Log:
> Add new testcases for log, diff and cat, according policy in
> notes/authz_policy.txt
>
> * subversion/tests/cmdline/authz_tests.py
> (authz_log_and_tracing_test): renamed function from authz_log_test.
> include new testcases

Am not a committer, but these are things I learnt from other committers...
- the second line of the log need to be indented with only only space,
like thus:
> (authz_log_and_tracing_test): renamed function from authz_log_test.
> include new testcases

- The following line should be added:
       (test_list): Renamed to call authz_log_and_tracing_test() instead of
authz_log_test().

This is just my comment:
- You could add the following:
       (authz_log_test): Renamed to authz_log_and_tracing_test().

      and mention the newly added features under the
authz_log_and_tracing_test() section.

Regards,
Madan.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon May 22 08:26:41 2006