]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
test: exit: fix run_commandf() warnings
authorEvgeny Bachinin <EABachinin@sberdevices.ru>
Mon, 20 Mar 2023 08:23:14 +0000 (11:23 +0300)
committerTom Rini <trini@konsulko.com>
Thu, 30 Mar 2023 19:09:59 +0000 (15:09 -0400)
Fix warnings after adding printf-like attribute format for
run_commandf():
warning: too many arguments for format [-Wformat-extra-args]

Signed-off-by: Evgeny Bachinin <EABachinin@sberdevices.ru>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
test/cmd/exit.c

index ca34abef8995422df9aa0eb87e1ee880c0eaebb5..7e160f7e4bb79b9d0cfaac8427c8c7eaad7751fa 100644 (file)
@@ -60,20 +60,20 @@ static int cmd_exit_test(struct unit_test_state *uts)
 
        /* Validate that 'exit' behaves the same way as 'exit 0' */
        ut_assertok(console_record_reset_enable());
-       ut_assertok(run_commandf("setenv foo 'echo bar ; exit ; echo baz' ; run foo ; echo $?", i));
+       ut_assertok(run_commandf("setenv foo 'echo bar ; exit ; echo baz' ; run foo ; echo $?"));
        ut_assert_nextline("bar");
        ut_assert_nextline("0");
        ut_assertok(ut_check_console_end(uts));
 
        ut_assertok(console_record_reset_enable());
-       ut_assertok(run_commandf("setenv foo 'echo bar ; exit ; echo baz' ; run foo && echo quux ; echo $?", i));
+       ut_assertok(run_commandf("setenv foo 'echo bar ; exit ; echo baz' ; run foo && echo quux ; echo $?"));
        ut_assert_nextline("bar");
        ut_assert_nextline("quux");
        ut_assert_nextline("0");
        ut_assertok(ut_check_console_end(uts));
 
        ut_assertok(console_record_reset_enable());
-       ut_assertok(run_commandf("setenv foo 'echo bar ; exit ; echo baz' ; run foo || echo quux ; echo $?", i));
+       ut_assertok(run_commandf("setenv foo 'echo bar ; exit ; echo baz' ; run foo || echo quux ; echo $?"));
        ut_assert_nextline("bar");
        /* Either 'exit' returns 0, or 'echo quux' returns 0 */
        ut_assert_nextline("0");
@@ -81,39 +81,39 @@ static int cmd_exit_test(struct unit_test_state *uts)
 
        /* Validate that return value still propagates from 'run' command */
        ut_assertok(console_record_reset_enable());
-       ut_assertok(run_commandf("setenv foo 'echo bar ; true' ; run foo ; echo $?", i));
+       ut_assertok(run_commandf("setenv foo 'echo bar ; true' ; run foo ; echo $?"));
        ut_assert_nextline("bar");
        ut_assert_nextline("0");
        ut_assertok(ut_check_console_end(uts));
 
        ut_assertok(console_record_reset_enable());
-       ut_assertok(run_commandf("setenv foo 'echo bar ; true' ; run foo && echo quux ; echo $?", i));
+       ut_assertok(run_commandf("setenv foo 'echo bar ; true' ; run foo && echo quux ; echo $?"));
        ut_assert_nextline("bar");
        ut_assert_nextline("quux");
        ut_assert_nextline("0");
        ut_assertok(ut_check_console_end(uts));
 
        ut_assertok(console_record_reset_enable());
-       ut_assertok(run_commandf("setenv foo 'echo bar ; true' ; run foo || echo quux ; echo $?", i));
+       ut_assertok(run_commandf("setenv foo 'echo bar ; true' ; run foo || echo quux ; echo $?"));
        ut_assert_nextline("bar");
        /* The 'true' returns 0 */
        ut_assert_nextline("0");
        ut_assertok(ut_check_console_end(uts));
 
        ut_assertok(console_record_reset_enable());
-       ut_assertok(run_commandf("setenv foo 'echo bar ; false' ; run foo ; echo $?", i));
+       ut_assertok(run_commandf("setenv foo 'echo bar ; false' ; run foo ; echo $?"));
        ut_assert_nextline("bar");
        ut_assert_nextline("1");
        ut_assertok(ut_check_console_end(uts));
 
        ut_assertok(console_record_reset_enable());
-       ut_assertok(run_commandf("setenv foo 'echo bar ; false' ; run foo && echo quux ; echo $?", i));
+       ut_assertok(run_commandf("setenv foo 'echo bar ; false' ; run foo && echo quux ; echo $?"));
        ut_assert_nextline("bar");
        ut_assert_nextline("1");
        ut_assertok(ut_check_console_end(uts));
 
        ut_assertok(console_record_reset_enable());
-       ut_assertok(run_commandf("setenv foo 'echo bar ; false' ; run foo || echo quux ; echo $?", i));
+       ut_assertok(run_commandf("setenv foo 'echo bar ; false' ; run foo || echo quux ; echo $?"));
        ut_assert_nextline("bar");
        ut_assert_nextline("quux");
        /* The 'echo quux' returns 0 */