[SSSD] [PATCH] debug tests - allow changing cwd

Jakub Hrozek jhrozek at redhat.com
Wed Jul 25 14:36:21 UTC 2012


On Wed, Jul 25, 2012 at 10:06:57AM +0200, Pavel Březina wrote:
> On 07/23/2012 05:31 PM, Jakub Hrozek wrote:
> >On Mon, Jul 23, 2012 at 02:32:44PM +0200, Pavel Březina wrote:
> >>Debug tests does not call tests_set_cwd() to change current
> >>directory to whatever was specified in configure.
> >
> >This works fine, but before pushing, I'm wondering about adding this
> >statement to all tests so that if the unit tests are extended in the
> >future and start writing out a file, the file gets written to the test
> >directory.
> 
> The only tests that miss this call aside of debug are krb5_child,
> resolv and simple_access, The attached patch puts it in these tests
> as well.

Nack, this breaks the krb5_child-test.

This test looks for the krb5_child binary to execute. I would argue that
the krb5_child test is mainly targeted at developers, so the set_cwd()
call is not necessary

Alternatively, you can change the way krb5_child-test searches for the
krb5-child binary (-DKRB5_CHILD_DIR in Makefile.am)



More information about the sssd-devel mailing list