selftests/bpf: Test_progs option for listing test names
authorJesper Dangaard Brouer <brouer@redhat.com>
Wed, 1 Jul 2020 21:44:17 +0000 (23:44 +0200)
committerAlexei Starovoitov <ast@kernel.org>
Wed, 1 Jul 2020 22:22:13 +0000 (15:22 -0700)
The program test_progs have some very useful ability to specify a list of
test name substrings for selecting which tests to run.

This patch add the ability to list the selected test names without running
them. This is practical for seeing which tests gets selected with given
select arguments (which can also contain a exclude list via --name-blacklist).

This output can also be used by shell-scripts in a for-loop:

 for N in $(./test_progs --list -t xdp); do \
   ./test_progs -t $N 2>&1 > result_test_${N}.log & \
 done ; wait

This features can also be used for looking up a test number and returning
a testname. If the selection was empty then a shell EXIT_FAILURE is
returned.  This is useful for scripting. e.g. like this:

 n=1;
 while [ $(./test_progs --list -n $n) ] ; do \
   ./test_progs -n $n ; n=$(( n+1 )); \
 done

Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Acked-by: Andrii Nakryiko <andriin@fb.com>
Link: https://lore.kernel.org/bpf/159363985751.930467.9610992940793316982.stgit@firesoul
tools/testing/selftests/bpf/test_progs.c
tools/testing/selftests/bpf/test_progs.h

index a5dba14b2025a1d2f5f94f11d16c92c943fb5cb4..ef05d2f0e7bbdc168fd44a235d31483161cb01da 100644 (file)
@@ -367,6 +367,7 @@ enum ARG_KEYS {
        ARG_VERIFIER_STATS = 's',
        ARG_VERBOSE = 'v',
        ARG_GET_TEST_CNT = 'c',
+       ARG_LIST_TEST_NAMES = 'l',
 };
 
 static const struct argp_option opts[] = {
@@ -382,6 +383,8 @@ static const struct argp_option opts[] = {
          "Verbose output (use -vv or -vvv for progressively verbose output)" },
        { "count", ARG_GET_TEST_CNT, NULL, 0,
          "Get number of selected top-level tests " },
+       { "list", ARG_LIST_TEST_NAMES, NULL, 0,
+         "List test names that would run (without running them) " },
        {},
 };
 
@@ -517,6 +520,9 @@ static error_t parse_arg(int key, char *arg, struct argp_state *state)
        case ARG_GET_TEST_CNT:
                env->get_test_cnt = true;
                break;
+       case ARG_LIST_TEST_NAMES:
+               env->list_test_names = true;
+               break;
        case ARGP_KEY_ARG:
                argp_usage(state);
                break;
@@ -665,6 +671,12 @@ int main(int argc, char **argv)
                        continue;
                }
 
+               if (env.list_test_names) {
+                       fprintf(env.stdout, "%s\n", test->test_name);
+                       env.succ_cnt++;
+                       continue;
+               }
+
                test->run_test();
                /* ensure last sub-test is finalized properly */
                if (test->subtest_name)
@@ -694,6 +706,9 @@ int main(int argc, char **argv)
                goto out;
        }
 
+       if (env.list_test_names)
+               goto out;
+
        fprintf(stdout, "Summary: %d/%d PASSED, %d SKIPPED, %d FAILED\n",
                env.succ_cnt, env.sub_succ_cnt, env.skip_cnt, env.fail_cnt);
 
index 0030584619c33b2c8506567e8cafa1cb6d374f26..ec31f382e7fd2423007cc5a533bbfcfd0f671dab 100644 (file)
@@ -67,6 +67,7 @@ struct test_env {
 
        bool jit_enabled;
        bool get_test_cnt;
+       bool list_test_names;
 
        struct prog_test_def *test;
        FILE *stdout;