diff options
author | Simon Glass <sjg@chromium.org> | 2023-10-01 19:15:17 -0600 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2023-12-13 18:39:05 -0500 |
commit | 3535f607261ced5e1df361a51e301b2d212b30ec (patch) | |
tree | 9851964e40a2f11bc37de4098714cc81276f4ee5 | |
parent | 80a8a2ebb34b4c2b99e41370ab42b8ed70ef2df5 (diff) | |
download | u-boot-3535f607261ced5e1df361a51e301b2d212b30ec.zip u-boot-3535f607261ced5e1df361a51e301b2d212b30ec.tar.gz u-boot-3535f607261ced5e1df361a51e301b2d212b30ec.tar.bz2 |
test: bdinfo: Add missing asserts
Calling into sub-test functions should be done using ut_assertok() so
that the test exits immediately on failure. Add those which are
missing.
Signed-off-by: Simon Glass <sjg@chromium.org>
-rw-r--r-- | test/cmd/bdinfo.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/test/cmd/bdinfo.c b/test/cmd/bdinfo.c index e463274..4977d01 100644 --- a/test/cmd/bdinfo.c +++ b/test/cmd/bdinfo.c @@ -136,8 +136,8 @@ static int lmb_test_dump_region(struct unit_test_state *uts, static int lmb_test_dump_all(struct unit_test_state *uts, struct lmb *lmb) { ut_assert_nextline("lmb_dump_all:"); - lmb_test_dump_region(uts, &lmb->memory, "memory"); - lmb_test_dump_region(uts, &lmb->reserved, "reserved"); + ut_assertok(lmb_test_dump_region(uts, &lmb->memory, "memory")); + ut_assertok(lmb_test_dump_region(uts, &lmb->reserved, "reserved")); return 0; } @@ -191,7 +191,7 @@ static int bdinfo_test_all(struct unit_test_state *uts) ut_assertok(test_num_l(uts, "fdt_size", (ulong)gd->fdt_size)); if (IS_ENABLED(CONFIG_VIDEO)) - test_video_info(uts); + ut_assertok(test_video_info(uts)); /* The gd->multi_dtb_fit may not be available, hence, #if below. */ #if CONFIG_IS_ENABLED(MULTI_DTB_FIT) @@ -202,7 +202,7 @@ static int bdinfo_test_all(struct unit_test_state *uts) struct lmb lmb; lmb_init_and_reserve(&lmb, gd->bd, (void *)gd->fdt_blob); - lmb_test_dump_all(uts, &lmb); + ut_assertok(lmb_test_dump_all(uts, &lmb)); if (IS_ENABLED(CONFIG_OF_REAL)) ut_assert_nextline("devicetree = %s", fdtdec_get_srcname()); } @@ -227,6 +227,9 @@ static int bdinfo_test_all(struct unit_test_state *uts) ut_assertok(test_num_l(uts, "malloc base", gd_malloc_start())); } + if (IS_ENABLED(CONFIG_X86)) + ut_check_skip_to_linen(uts, " high end ="); + return 0; } |