[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Test cleanup: use None instead of "" constistently

From: Daniel Rall <dlr_at_collab.net>
Date: 2006-11-27 20:09:17 CET

Hyrum, the patch looks good. Please commit it to trunk (noting "dlr"
as the approver).

- Dan

On Thu, 23 Nov 2006, Hyrum K. Wright wrote:

> Several files in the test suite use the run_and_verify_svn method
> extensively, but argument usage is inconsistent. The first argument can
> be either "" or None, with no functional change. Prior to applying this
> patch, the usage of None far outweighs the usage of "". This patch
> changes the python tests to use None instead of "" when appropriate for
> run_and_verify_svn.
>
> -Hyrum
>
> [[[
> Update the tests to consistently use None as the first argument when
> calling run_and_verify_svn.
>
> * subversion/tests/cmdline/revert_tests.py
> * subversion/tests/cmdline/log_tests.py
> * subversion/tests/cmdline/trans_tests.py
> * subversion/tests/cmdline/externals_tests.py
> * subversion/tests/cmdline/copy_tests.py
> * subversion/tests/cmdline/svnversion_tests.py
> * subversion/tests/cmdline/special_tests.py
> * subversion/tests/cmdline/basic_tests.py
> * subversion/tests/cmdline/utf8_tests.py
> * subversion/tests/cmdline/commit_tests.py
> * subversion/tests/cmdline/authz_tests.py
> Change calls to svntest.actions.run_and_verify_svn to use None instead
> of "" when not expecting a message.
> ]]]

  • application/pgp-signature attachment: stored
Received on Tue Nov 28 05:25:05 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.