On 2014-10-09 01:26:46, Steve Beattie wrote:
> With the two recent unix socket test patches to mark dgram tests as
> failing but expected to pass, I realized that there's no indication
> in the output that there are current expected failures (except for
> the single corefile test expected failure)[0]. This patch attempts
> to remedy that by emitting the x-type plus the test description for
> each test that is marked as such. I've set it to always emit these
> results; I could be talked into only doing it when VERBOSE is set, but I
> think the constant visible reminder is probably for the best.
> 
> [0] The test infrastructure does give an alert if there is a test that
>     has been marked xpass or xfail but has started behaving correctly.
> 
> Signed-off-by: Steve Beattie <[email protected]>

This seems like a good idea to me.

Acked-by: Tyler Hicks <[email protected]>

Tyler

> ---
>  tests/regression/apparmor/prologue.inc |    5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> Index: b/tests/regression/apparmor/prologue.inc
> ===================================================================
> --- a/tests/regression/apparmor/prologue.inc
> +++ b/tests/regression/apparmor/prologue.inc
> @@ -250,7 +250,10 @@ checktestfg()
>               fi
>       fi
>  
> -     if [ -n "$VERBOSE" ]; then
> +     # warn about expected failures
> +     if [ -n "${_known}" ] ; then
> +             echo "x${_pfmode}: ${_testdesc}"
> +     elif [ -n "$VERBOSE" ]; then
>               echo "ok: ${_testdesc}"
>       fi
>  }
> 
> -- 
> Steve Beattie
> <[email protected]>
> http://NxNW.org/~steve/



> -- 
> AppArmor mailing list
> [email protected]
> Modify settings or unsubscribe at: 
> https://lists.ubuntu.com/mailman/listinfo/apparmor

Attachment: signature.asc
Description: Digital signature

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to