[apparmor] [PATCH] parser: Move failing test to TODO
Tyler Hicks
tyhicks at canonical.com
Thu Feb 11 22:00:17 UTC 2016
On 2016-02-11 13:20:58, Steve Beattie wrote:
> On Thu, Feb 11, 2016 at 02:54:53PM -0600, Tyler Hicks wrote:
> > This test causes `make check` to fail but it is known bug so mark it as
> > a TODO test.
> >
> > Signed-off-by: Tyler Hicks <tyhicks at canonical.com>
> > ---
> > parser/tst/simple_tests/file/ok_bare_1.sd | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/parser/tst/simple_tests/file/ok_bare_1.sd b/parser/tst/simple_tests/file/ok_bare_1.sd
> > index 5097f38..b8d6d06 100644
> > --- a/parser/tst/simple_tests/file/ok_bare_1.sd
> > +++ b/parser/tst/simple_tests/file/ok_bare_1.sd
> > @@ -1,6 +1,7 @@
> > #
> > #=Description bare file rule
> > -#=EXRESULT PASS
> > +#=EXRESULT FAIL
> > +#=TODO https://launchpad.net/bugs/1215637
> > #
>
> Please keep the expected result as-is (i.e. '#=EXRESULT PASS');
> otherwise, perl's prove script will tell you that the todo test has
> passed[0]. Marking the test as todo just says to not fail the overall
> testrun based on the failure of that test (because fixing it is still
> on the todo list).
Bah... that's what I meant to do but screwed up when trying to quickly
send this patch out. Thanks for catching that.
> With the expected result set correctly,
> Acked-by: Steve Beattie <steve at nxnw.org>.
Thanks! I'll run another `make check` before committing the updated
test.
Tyler
> Thanks.
>
> [0] In fact, we have some currently marked todo tests that pass their
> tests:
>
> TODO passed: 71337, 71348-71349, 71383, 71466
>
> which are:
>
> ok 71337 - TODO: ./simple_tests//vars/vars_profile_name_14.sd: reference variables in rules that also have alternations # TODO Unfixed testcase.
> ok 71348 - TODO: ./simple_tests//vars/vars_profile_name_25.sd: reference variables is null # TODO Unfixed testcase.
> ok 71349 - TODO: ./simple_tests//vars/vars_profile_name_26.sd: reference variables is null # TODO Unfixed testcase.
> ok 71383 - TODO: ./simple_tests//xtrans/minimize-x-conflict.sd: test for conflict resolution in minimization phase of dfa gen # TODO Unfixed testcase.
> ok 71466 - TODO: ./simple_tests//xtrans/x-conflict.sd: test for conflict resolution in minimization phase of dfa gen # TODO Unfixed testcase.
>
> (They ought to be confirmed to have been addressed before removing
> the TODO annotation.)
>
> --
> Steve Beattie
> <sbeattie at ubuntu.com>
> http://NxNW.org/~steve/
> --
> AppArmor mailing list
> AppArmor at lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20160211/d3faae59/attachment.pgp>
More information about the AppArmor
mailing list