Re: Requiring python sqlite3 module for devs

[Branko Cibej <br...@xbc.nu>]

Lieven Govaerts wrote:
> On Wed, Sep 30, 2009 at 2:49 PM, Hyrum K. Wright <hy...@hyrumwright.org> wrote:
>   
>> We currently have a couple of tests which conditionally run if the
>> sqlite3 python module is found.  As we move more working copy stuff
>> into sqlite, the number of tests that will be needed to be skipped
>> will increase, and the transparency of *why* they need to be skipped
>> will decrease.  For this reason, I'd like to propose that we require
>> the python sqlite3 module be installed when running the tests.
>>     
>
> On a related topic, I think it's wrong to put tests relying on the
> internals of subversion (white box tests) together with the acceptance
> tests (black box tests).
> Together, or at least, not clearly marked as being dependent on the
> internals of Subversion.
>   

You have a good point there; in fact we already have white-box tests, in
the C test suite. There's no real reason why that part should be
strictly C-only though, we could have tests implemented in Python there,
too. Makes for a sensible separation, IMHO.

-- Brane

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2402356