From e9b25c496948d917d65b871d2d69c68cd44e04e5 Mon Sep 17 00:00:00 2001 From: Christian Boltz Date: Tue, 18 Sep 2018 21:36:32 +0200 Subject: [PATCH] make simple.pl / parser make check output more useful Instead of printing the (useless) numbers of no-longer-failing tests marked as TODO, let `prove` print out the actual test names. A side effect is that really unfixed TODOs and disabled tests get printed (lists about 10 tests), but that's much better than having to hunt down the no-longer-failing TODOs. This change will print out lines like (the first one is still a TODO, the second one got fixed at some time) ``` not ok 71447 - TODO: ./simple_tests//vars/vars_profile_name_13.sd: reference variables that are the profile name and attachment # TODO Unfixed testcase. ok 71448 - TODO: ./simple_tests//vars/vars_profile_name_14.sd: reference variables in rules that also have alternations # TODO Unfixed testcase. ``` before printing the test summary. --- parser/tst/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/parser/tst/Makefile b/parser/tst/Makefile index be0ddf668..77ed08d3d 100644 --- a/parser/tst/Makefile +++ b/parser/tst/Makefile @@ -6,7 +6,7 @@ PARSER_BIN=apparmor_parser PARSER=$(PARSER_DIR)/$(PARSER_BIN) # parser.conf to use in tests. Note that some test scripts have the parser options hardcoded, so passing PARSER_ARGS=... is not enough to override it. PARSER_ARGS=--config-file=./parser.conf -PROVE_ARG=-f +PROVE_ARG=-f --directives ifeq ($(VERBOSE),1) PROVE_ARG+=-v