On 08/14/2010 12:55 AM, Ralf Wildenhues wrote: > * Peter Rosin wrote on Tue, Aug 10, 2010 at 11:08:03PM CEST: >> Subject: [PATCH] Keep testsuite files on unexpected pass. >> >> * lib/autotest/general.m4 (AT_INIT) <at_fn_group_postprocess>: >> Don't cleanup the group directory when a test unexpectedly passes. >> * tests/autotest.at (Cleanup): Check that an unexpected pass leaves >> the test group directory intact. > > For what it's worth, this patch looks good to me. The "Cleanup" test > group might be a bit more complex than needed, but that's not new with > your patch.
Agreed; and applied.
>
> Also I just noticed that --errexit is broken wrt. XPASS tests, but
> that's not new with your patch either. The following seems to fix
> it but I don't have testsuite exposure yet. One thing to note for
> the latter is that the "test ... and inhibited subsequent tests."
> notice is not necessarily true, in that the testsuite does run
> subsequent tests after an XPASS.
Good catch. Are you planning on cleaning up this diff into a full
patch, or should I apply it as is?
> @@ -1244,6 +1244,9 @@ _ATEOF
> # or the success was unexpected.
> if $at_debug_p || test $at_res = xpass; then
> at_fn_create_debugging_script
> + if test $at_res = xpass && $at_errexit; then
> + echo stop > "$at_stop_file"
> + fi
> else
> if test -d "$at_group_dir"; then
> find "$at_group_dir" -type d ! -perm -700 -exec chmod u+rwx \{\} \;
>
>
--
Eric Blake [email protected] +1-801-349-2682
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
