diff options
author | Petri Latvala <petri.latvala@intel.com> | 2019-01-22 12:47:23 +0200 |
---|---|---|
committer | Petri Latvala <petri.latvala@intel.com> | 2019-01-22 14:26:21 +0200 |
commit | 96f3a1b876e0dd24706b85fb872f12031a436e84 (patch) | |
tree | eb4ef11df3f31ba7dc40605f46cdd24a6fa84dd6 /runner/runner_tests.c | |
parent | 34e312999df75d6ac68ca7714b9137df56acc3f8 (diff) |
runner: Implement --dry-run
Actually implement --dry-run to not execute tests. With dry-run
active, attempting to execute will figure out the list of things to
execute, serialize them along with settings, and stop. This will be
useful for CI that wants to post-mortem on failed test rounds to
generate a list of tests that should have been executed and produce
json result files (full of 'notrun') for proper statistics.
Signed-off-by: Petri Latvala <petri.latvala@intel.com>
Cc: Andi Shyti <andi.shyti@intel.com>
Cc: Martin Peres <martin.peres@linux.intel.com>
Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Cc: Tomi Sarvela <tomi.p.sarvela@intel.com>
Reviewed-by: Martin Peres <martin.peres@linux.intel.com>
Diffstat (limited to 'runner/runner_tests.c')
-rw-r--r-- | runner/runner_tests.c | 70 |
1 files changed, 70 insertions, 0 deletions
diff --git a/runner/runner_tests.c b/runner/runner_tests.c index 6213b8e7..0965612a 100644 --- a/runner/runner_tests.c +++ b/runner/runner_tests.c @@ -745,6 +745,76 @@ igt_main igt_subtest_group { char dirname[] = "tmpdirXXXXXX"; struct job_list list; + int dirfd = -1, subdirfd = -1, fd = -1; + + igt_fixture { + init_job_list(&list); + igt_require(mkdtemp(dirname) != NULL); + rmdir(dirname); + } + + igt_subtest("dry-run-option") { + struct execute_state state; + const char *argv[] = { "runner", + "--dry-run", + testdatadir, + dirname, + }; + + igt_assert(parse_options(ARRAY_SIZE(argv), (char**)argv, &settings)); + igt_assert(create_job_list(&list, &settings)); + + igt_assert(initialize_execute_state(&state, &settings, &list)); + igt_assert_eq(state.next, 0); + igt_assert(state.dry); + igt_assert_eq(list.size, 5); + + igt_assert_f((dirfd = open(dirname, O_DIRECTORY | O_RDONLY)) >= 0, + "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_f((fd = openat(dirfd, "metadata.txt", O_RDONLY)) >= 0, + "Dry run initialization didn't serialize settings.\n"); + close(fd); + igt_assert_f((fd = openat(dirfd, "joblist.txt", O_RDONLY)) >= 0, + "Dry run initialization didn't serialize the job list.\n"); + close(fd); + igt_assert_f((fd = openat(dirfd, "uname.txt", O_RDONLY)) < 0, + "Dry run initialization created uname.txt.\n"); + + igt_assert(initialize_execute_state_from_resume(dirfd, &state, &settings, &list)); + igt_assert_eq(state.next, 0); + igt_assert(!state.dry); + igt_assert_eq(list.size, 5); + /* initialize_execute_state_from_resume() closes the dirfd */ + igt_assert_f((dirfd = open(dirname, O_DIRECTORY | O_RDONLY)) >= 0, + "Dry run resume somehow deleted the results directory.\n"); + + /* Execute from resume should work */ + igt_assert(execute(&state, &settings, &list)); + igt_assert_f((fd = openat(dirfd, "uname.txt", O_RDONLY)) >= 0, + "Dry run resume didn't create uname.txt.\n"); + close(fd); + igt_assert_f((subdirfd = openat(dirfd, "0", O_DIRECTORY | O_RDONLY)) >= 0, + "Dry run resume didn't create result directory.\n"); + igt_assert_f((fd = openat(subdirfd, "journal.txt", O_RDONLY)) >= 0, + "Dry run resume didn't create a journal.\n"); + } + + igt_fixture { + close(fd); + close(dirfd); + close(subdirfd); + clear_directory(dirname); + free_job_list(&list); + } + } + + igt_subtest_group { + char dirname[] = "tmpdirXXXXXX"; + struct job_list list; int dirfd = -1, fd = -1; igt_fixture { |