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

Re: [PATCH] Add size information to svn_client_info()

From: Lieven Govaerts <svnlgo_at_mobsol.be>
Date: 2007-06-24 14:20:21 CEST

Daniel Rall wrote:
> This segways into the last question I have about this change: How do
> we test it? We don't currently have a subversion/tests/libsvn_client/
> module, because testing of the command-line client has been
> traditionally carried out via Python scripts in
> subversion/tests/cmdline/ (e.g. info_tests.py). However, if we don't
> expose this function via the command-line client, we'll probably
> ending up growing API-level tests for libsvn_client. This is not the
> first time we're running into this for 1.5 -- Hyrum's commutative
> author and revision reporting APIs will also be collecting some
> information (for 'blame') that we don't display via the command-line
> UI.
>
I committed Daniel's version of this patch with one small tweak in r25521.

To be able to test this new functionality r25522 exposes the
SVN_INFO_SIZE_UNKNOWN define in the SWIG bindings and r25523 adds a
Python bindings test.
It tests the API equivalent of:
- svn info wc-file
- svn info -r BASE wc-file
- svn info remote_file

AFAIC, this should be sufficient.

Thanks for the patch Stefan,

Lieven

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Jun 24 14:16:12 2007

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