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

test: cmd/hash: check return value of ut_check_console_line

2024-11-19T01:10:56Z

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:06Z

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:06Z

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:06Z

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:51Z

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:24Z

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:58Z

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:01Z

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:01Z

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

command_ut: test: Move test into lib

2024-11-13T17:56:01Z

This test doesn't belong at the top level. Move it into the lib/ directory, since that is where compression is implemented. Rename it to just 'command', since it is obviously a unit test and the _ut suffix does not add much except to make it different from the names of other test files. Signed-off-by: Simon Glass Tested-by: Tom Rini # rpi_3, rpi_4, rpi_arm64, am64x_evm_a53, am64-sk