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

On Sun, 24 Jun 2007, Lieven Govaerts wrote:

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

I've added corresponding bindings and tests to the JavaHL bindings in
r25665.