dm: test: Don't fail when tests are skipped due to build
authorSimon Glass <sjg@chromium.org>
Wed, 25 Sep 2019 14:55:52 +0000 (08:55 -0600)
committerBin Meng <bmeng.cn@gmail.com>
Tue, 8 Oct 2019 05:57:38 +0000 (13:57 +0800)
At present tests that are marked as only for livetree fail when executed
on sandbox_flattree. They cannot actually be executed, but we should not
resport them as 'not found', since this causes errors. Instead, they
should be silently skipped.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
test/dm/test-main.c

index 487d8b9627539c6eb2df04d271d01ba5bd4cddd8..74f77ccc8492bbbfec3c40e2d1ab1bab071e7a8b 100644 (file)
@@ -130,7 +130,7 @@ static int dm_test_main(const char *test_name)
        const int n_ents = ll_entry_count(struct unit_test, dm_test);
        struct unit_test_state *uts = &global_dm_test_state;
        struct unit_test *test;
-       int run_count;
+       int found;
 
        uts->priv = &_global_priv_dm_test_state;
        uts->fail_count = 0;
@@ -148,7 +148,7 @@ static int dm_test_main(const char *test_name)
        if (!test_name)
                printf("Running %d driver model tests\n", n_ents);
 
-       run_count = 0;
+       found = 0;
 #ifdef CONFIG_OF_LIVE
        uts->of_root = gd->of_root;
 #endif
@@ -180,10 +180,10 @@ static int dm_test_main(const char *test_name)
                        ut_assertok(dm_do_test(uts, test, false));
                        runs++;
                }
-               run_count += runs;
+               found++;
        }
 
-       if (test_name && !run_count)
+       if (test_name && !found)
                printf("Test '%s' not found\n", test_name);
        else
                printf("Failures: %d\n", uts->fail_count);