u-boot.git/test/cmd, branch v2025.01

test/cmd/wget.c: move net_test_wget() to the cmd test suite

2024-11-29T13:51:57+00:00

Since net_test_wget() is testing a command and is in test/cmd it should
be in the 'cmd' test suite, not 'lib'.

Saving and restoring the values of the environment variables that the
test manipulates is necessary to avoid a regression when running the
whole ut test suite. A minimal reproducer is:

 $ ./u-boot -T -c "ut cmd net_test_wget; ut dm dm_test_eth_act" | \
     grep -E "(Test:|Failures:)"

Reported-by: Simon Glass 
Signed-off-by: Jerome Forissier

test: cmd/hash: check return value of ut_check_console_line

2024-11-19T01:10:56+00:00

ut_check_console_line() does include an assert.
Pass the result to ut_assertok().

Addresses-Coverity-ID:  514958 Error handling issues
Fixes: 7dfafcd65ef3 ("test: unit test for hash command")
Signed-off-by: Heinrich Schuchardt

lmb.c: add missing comma in lmb_dump_region()

2024-11-15T00:14:06+00:00

In the message string " %s[%d]\t[0x%llx-0x%llx], 0x%08llx bytes flags: "
a comma is missing before flags.

To avoid increasing the code size replace '0x%' by '%#'.

Printing the size with leading zeros but not the addresses does not really
make sense. Remove the leading zeros from the size output.

Signed-off-by: Heinrich Schuchardt 
[trini: Fix test/cmd/bdinfo.c for these changes]
Signed-off-by: Tom Rini

test: use %zd for size_t in mbr_test_run()

2024-11-15T00:14:06+00:00

For printing size_t we must use %zd and not %ld to avoid
a -Wformat error on 32-bit systems.

Signed-off-by: Heinrich Schuchardt

test: cmd/mbr: pass correct buffer size to init_write_buffers

2024-11-15T00:14:06+00:00

We want to completely initialize the mbr and embr buffers. This requires
passing the buffer size and not the size of a pointer to the buffer.

Addresses-Coverity-ID: 510454 Wrong sizeof argument
Signed-off-by: Heinrich Schuchardt

Merge patch series "cmd: hash: correct parameter count check"

2024-11-14T16:49:51+00:00

Heinrich Schuchardt  says:

Since commit 348ea878508d ("cmd: hash: fix param count check") the hash
command cannot be used without the optional variable name parameter if
CONFIG_HASH_VERIFY=y. 'hash sha1 $loadaddr $filesize' returns
CMD_RET_USAGE.

The minimum number of arguments is four no matter if verification is
enabled or not.

Fix the parameter check.

Provide a unit test.

Link: https://lore.kernel.org/r/20241102100836.103005-1-heinrich.schuchardt@canonical.com

test: unit test for hash command

2024-11-14T16:49:24+00:00

Provide a unit test testing the hash command.

Signed-off-by: Heinrich Schuchardt

Merge patch series "test: Tidy up the test/ directory"

2024-11-13T22:02:58+00:00

Simon Glass  says:

Some tests do not use the unit-test framework. Others are in a suite of
their own, for no obvious reason.

This series tidies this up.

Link: https://lore.kernel.org/r/20241102193715.432529-1-sjg@chromium.org

test: Update command test to use unit-test functions

2024-11-13T17:56:01+00:00

Rather than enabled DEBUG and using assert(), use the unit-test
functions now provided.

Drop a check that causes pytest to fail.

Signed-off-by: Simon Glass 
Tested-by: Tom Rini  # rpi_3, rpi_4, rpi_arm64, am64x_evm_a53, am64-sk

command: test: Move into the cmd suite

2024-11-13T17:56:01+00:00

The command test was the very first test written in U-Boot, some 12
years ago. It predates the unit-test subsystem and was never converted
over.

There is no particular need for the command test to have its own
command. It is also confusing to have it separate from the normal test
suites. At present this test is not run in CI.

Move it into the cmd suite instead, updating it to become a unit test.
One of the checks is dropped to avoid an error.

Signed-off-by: Simon Glass 
Tested-by: Tom Rini  # rpi_3, rpi_4, rpi_arm64, am64x_evm_a53, am64-sk