From ebf2d10a95fff590542808ec14c7bce74ad11b37 Mon Sep 17 00:00:00 2001 From: Petri Latvala Date: Tue, 19 Feb 2019 12:18:10 +0200 Subject: runner: Exit with 0 on dry-run v2: Adjust tests accordingly Signed-off-by: Petri Latvala Cc: Arkadiusz Hiler Cc: Tomi Sarvela Reviewed-by: Arkadiusz Hiler --- runner/executor.c | 2 +- runner/runner_tests.c | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) (limited to 'runner') diff --git a/runner/executor.c b/runner/executor.c index 3f1eb010..d535c276 100644 --- a/runner/executor.c +++ b/runner/executor.c @@ -1208,7 +1208,7 @@ bool execute(struct execute_state *state, if (state->dry) { printf("Dry run, not executing. Invoke igt_resume if you want to execute.\n"); - return false; + return true; } if ((resdirfd = open(settings->results_path, O_DIRECTORY | O_RDONLY)) < 0) { diff --git a/runner/runner_tests.c b/runner/runner_tests.c index 0965612a..e46568ac 100644 --- a/runner/runner_tests.c +++ b/runner/runner_tests.c @@ -773,8 +773,9 @@ igt_main "Dry run initialization didn't create the results directory.\n"); /* Execute from just initialize_execute_state should fail */ - igt_assert(!execute(&state, &settings, &list)); - igt_assert(openat(dirfd, "0", O_DIRECTORY | O_RDONLY) < 0); + igt_assert(execute(&state, &settings, &list)); + igt_assert_f(openat(dirfd, "0", O_DIRECTORY | O_RDONLY) < 0, + "Dry run executed when it should not have.\n"); igt_assert_f((fd = openat(dirfd, "metadata.txt", O_RDONLY)) >= 0, "Dry run initialization didn't serialize settings.\n"); close(fd); -- cgit v1.2.3