From 333a5c6f07cba9e6d7a5506521bae5ec89b867c1 Mon Sep 17 00:00:00 2001 From: Daniel Vetter Date: Tue, 11 Feb 2014 12:14:09 +0100 Subject: lib/drmtest: Fix up skip_henceforth checks We need to first check whether we'll even run the testcase before deciding to print a premature SKIP/FAIL. Without this and global skipped resulted in superflous output when running only one testcase. References: https://bugs.freedesktop.org/show_bug.cgi?id=74100 Signed-off-by: Daniel Vetter --- lib/drmtest.c | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) (limited to 'lib/drmtest.c') diff --git a/lib/drmtest.c b/lib/drmtest.c index f4a86005..2718de4d 100644 --- a/lib/drmtest.c +++ b/lib/drmtest.c @@ -972,6 +972,10 @@ bool __igt_run_subtest(const char *subtest_name) return false; } + if (run_single_subtest && + strcmp(subtest_name, run_single_subtest) != 0) + return false; + if (skip_subtests_henceforth) { printf("Subtest %s: %s\n", subtest_name, skip_subtests_henceforth == SKIP ? @@ -979,14 +983,7 @@ bool __igt_run_subtest(const char *subtest_name) return false; } - if (!run_single_subtest) { - return (in_subtest = subtest_name); - } else { - if (strcmp(subtest_name, run_single_subtest) == 0) - return (in_subtest = subtest_name); - - return false; - } + return (in_subtest = subtest_name); } const char *igt_subtest_name(void) -- cgit v1.2.3