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)