Djam/u-boot
1
0
Fork 0
mirror of https://github.com/u-boot/u-boot.git synced 2025-04-28 00:11:32 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

test/cmd/wget.c: Fix loadaddr rewrite

Browse source 
The $loadaddr variable is a hexadecimal value, not a string, it must be
assigned using env_set_hex(). This may break follow up tests, like the
dm_test_cmd_hash_md5 in CI. To avoid any interference with other tests,
set $wgetaddr variable which is specific to this test and use it in the
test.

Fixes: 20f641987f ("test/cmd/wget.c: move net_test_wget() to the cmd test suite")
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Fabio Estevam <festevam@gmail.com>
This commit is contained in:
Marek Vasut 2025-01-27 00:57:03 +01:00 • committed by Tom Rini
parent d8a7100d65
commit 082b41df8a
1 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
test/cmd/wget.c
View file

@ -217,28 +217,26 @@ static int net_test_wget(struct unit_test_state *uts)
{ {
char *prev_ethact = env_get("ethact"); char *prev_ethact = env_get("ethact");
char *prev_ethrotate = env_get("ethrotate"); char *prev_ethrotate = env_get("ethrotate");
char *prev_loadaddr = env_get("loadaddr");
sandbox_eth_set_tx_handler(0, sb_http_handler); sandbox_eth_set_tx_handler(0, sb_http_handler);
sandbox_eth_set_priv(0, uts); sandbox_eth_set_priv(0, uts);
env_set("ethact", "eth@10002000"); env_set("ethact", "eth@10002000");
env_set("ethrotate", "no"); env_set("ethrotate", "no");
env_set("loadaddr", "0x20000"); env_set("wgetaddr", "0x20000");
ut_assertok(run_command("wget ${loadaddr} 1.1.2.2:/index.html", 0)); ut_assertok(run_command("wget ${wgetaddr} 1.1.2.2:/index.html", 0));
ut_assert_nextline_empty(); ut_assert_nextline_empty();
ut_assert_nextline("Packets received 5, Transfer Successful"); ut_assert_nextline("Packets received 5, Transfer Successful");
ut_assert_nextline("Bytes transferred = 29 (1d hex)"); ut_assert_nextline("Bytes transferred = 29 (1d hex)");
sandbox_eth_set_tx_handler(0, NULL); sandbox_eth_set_tx_handler(0, NULL);
run_command("md5sum ${loadaddr} ${filesize}", 0); run_command("md5sum ${wgetaddr} ${filesize}", 0);
ut_assert_nextline("md5 for 00020000 ... 0002001c ==> 847d5e7320a27462e90bc1ed75eb8cd8"); ut_assert_nextline("md5 for 00020000 ... 0002001c ==> 847d5e7320a27462e90bc1ed75eb8cd8");
ut_assert_console_end(); ut_assert_console_end();
env_set("ethact", prev_ethact); env_set("ethact", prev_ethact);
env_set("ethrotate", prev_ethrotate); env_set("ethrotate", prev_ethrotate);
env_set("loadaddr", prev_loadaddr);
return 0; return 0;
} }