diff options
-rw-r--r-- | runner/runner_tests.c | 29 |
1 files changed, 19 insertions, 10 deletions
diff --git a/runner/runner_tests.c b/runner/runner_tests.c index 39d4a078..9bbc4252 100644 --- a/runner/runner_tests.c +++ b/runner/runner_tests.c @@ -20,6 +20,15 @@ static const char testdatadir[] = TESTDATA_DIRECTORY; +/* + * The total sum of subtests in the tests in runner/testdata/. Note + * that test binaries without subtests should still be counted as one + * for this macro. + */ +#define NUM_TESTDATA_SUBTESTS 5 +/* The total number of test binaries in runner/testdata/ */ +#define NUM_TESTDATA_BINARIES 3 + static void igt_assert_eqstr(const char *one, const char *two) { if (one == NULL && two == NULL) @@ -650,11 +659,11 @@ igt_main } } - job_list_filter_test("nofilters", "-n", "placeholderargs", 5, 3); + job_list_filter_test("nofilters", "-n", "placeholderargs", NUM_TESTDATA_SUBTESTS, NUM_TESTDATA_BINARIES); job_list_filter_test("binary-include", "-t", "successtest", 2, 1); - job_list_filter_test("binary-exclude", "-x", "successtest", 3, 2); + job_list_filter_test("binary-exclude", "-x", "successtest", NUM_TESTDATA_SUBTESTS - 2, NUM_TESTDATA_BINARIES - 1); job_list_filter_test("subtest-include", "-t", "first-subtest", 1, 1); - job_list_filter_test("subtest-exclude", "-x", "second-subtest", 4, 3); + job_list_filter_test("subtest-exclude", "-x", "second-subtest", NUM_TESTDATA_SUBTESTS - 1, NUM_TESTDATA_BINARIES); job_list_filter_test("piglit-names", "-t", "igt@successtest", 2, 1); job_list_filter_test("piglit-names-subtest", "-t", "igt@successtest@first", 1, 1); @@ -869,7 +878,7 @@ igt_main 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_eq(list->size, NUM_TESTDATA_SUBTESTS); igt_assert_f((dirfd = open(dirname, O_DIRECTORY | O_RDONLY)) >= 0, "Dry run initialization didn't create the results directory.\n"); @@ -890,7 +899,7 @@ igt_main 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); + igt_assert_eq(list->size, NUM_TESTDATA_SUBTESTS); /* 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"); @@ -940,7 +949,7 @@ igt_main igt_assert(initialize_execute_state(&state, settings, list)); igt_assert_eq(state.next, 0); - igt_assert_eq(list->size, 5); + igt_assert_eq(list->size, NUM_TESTDATA_SUBTESTS); igt_assert_f((dirfd = open(dirname, O_DIRECTORY | O_RDONLY)) >= 0, "Execute state initialization didn't create the results directory.\n"); igt_assert_f((fd = openat(dirfd, "metadata.txt", O_RDONLY)) >= 0, @@ -1096,7 +1105,7 @@ igt_main igt_assert(parse_options(ARRAY_SIZE(argv), (char**)argv, settings)); igt_assert(create_job_list(list, settings)); - igt_assert(list->size == 3); + igt_assert(list->size == NUM_TESTDATA_BINARIES); if (!strcmp(list->entries[0].binary, "no-subtests")) { struct job_list_entry tmp = list->entries[0]; @@ -1120,7 +1129,7 @@ igt_main igt_assert(initialize_execute_state_from_resume(dirfd, &state, settings, list)); igt_assert_eq(state.next, 1); - igt_assert_eq(list->size, 3); + igt_assert_eq(list->size, NUM_TESTDATA_BINARIES); } igt_fixture { @@ -1154,12 +1163,12 @@ igt_main struct execute_state state; const char *argv[] = { "runner", multiple ? "--multiple-mode" : "--sync", - "-t", "-subtest", + "-t", "successtest.*-subtest", testdatadir, dirname, }; char testdirname[16]; - size_t expected_tests = multiple ? 2 : 3; + size_t expected_tests = multiple ? 1 : 2; size_t i; igt_assert(parse_options(ARRAY_SIZE(argv), (char**)argv, settings)); |