u-boot.git/test/cmd/setexpr.c, branch master Unnamed repository; edit this file 'description' to name the repository. global: Avoid indirect inclusion of <env.h> from <command.h> 2025-05-29T14:29:16+00:00 Tom Rini trini@konsulko.com 2025-05-14T22:46:03+00:00 dcb7d927d181b5798c6fd4a1b1650cd821e3b6ed The include file <command.h> does not need anything from <env.h>. Furthermore, include/env.h itself includes other headers which can lead to longer indirect inclusion paths. To prepare to remove <env.h> from <command.h> fix all of the places which had relied on this indirect inclusion to instead include <env.h> directly. Reviewed-by: Mattijs Korpershoek <mkorpershoek@kernel.org> # android, bcb Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org> # spawn Signed-off-by: Tom Rini <trini@konsulko.com> 
The include file <command.h> does not need anything from <env.h>.
Furthermore, include/env.h itself includes other headers which can lead
to longer indirect inclusion paths. To prepare to remove <env.h> from
<command.h> fix all of the places which had relied on this indirect
inclusion to instead include <env.h> directly.

Reviewed-by: Mattijs Korpershoek <mkorpershoek@kernel.org> # android, bcb
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org> # spawn
Signed-off-by: Tom Rini <trini@konsulko.com>
test: Drop suites.h 2025-02-12T02:12:36+00:00 Simon Glass sjg@chromium.org 2025-02-07T18:30:56+00:00 32aba887e3d7cbcdf8fd98c107aaa79f3bd44a16 This file is empty now. Remove it and its uses. Signed-off-by: Simon Glass <sjg@chromium.org> 
This file is empty now. Remove it and its uses.

Signed-off-by: Simon Glass <sjg@chromium.org>
cmd/setexpr: support concatenation of direct strings 2025-02-07T19:35:23+00:00 Heinrich Schuchardt heinrich.schuchardt@canonical.com 2025-02-03T15:10:29+00:00 af7eca24a78b1419656d618463f1eabe297fdb2f The setexpr.s command allows to concatenate two strings. According to the description in doc/usage/cmd/setexpr.rst the parameters value1 and value2 can be either direct values or pointers to a memory location holding the values. Unfortunately `setexpr.s <value1> + <value2>` fails if any of the values is a direct value. $? is set to false. * Add support for direct values in setexpr.s. * Correct the unit test for "setexpr.s fred 0". * Add a new unit test for "setexpr.s fred '1' + '3'" giving '13'. Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com> 
The setexpr.s command allows to concatenate two strings.

According to the description in doc/usage/cmd/setexpr.rst the parameters
value1 and value2 can be either direct values or pointers to a
memory location holding the values.

Unfortunately `setexpr.s <value1> + <value2>` fails if any of the values
is a direct value. $? is set to false.

* Add support for direct values in setexpr.s.
* Correct the unit test for "setexpr.s fred 0".
* Add a new unit test for "setexpr.s fred '1' + '3'" giving '13'.

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
test: remove available memory check in setexpr_test_str_oper() 2025-02-07T19:35:23+00:00 Heinrich Schuchardt heinrich.schuchardt@canonical.com 2025-02-03T15:10:28+00:00 d4265cdcd57c1975d4e756ab1778c837acc9776c env_set() frees the previous value after allocating the new value. As the free() may merge memory chunks the available memory is not expected to stay constant. Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com> 
env_set() frees the previous value after allocating the new value.
As the free() may merge memory chunks the available memory is not
expected to stay constant.

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
test: remove available memory check in setexpr_test_str() 2025-02-07T19:35:23+00:00 Heinrich Schuchardt heinrich.schuchardt@canonical.com 2025-02-03T15:10:27+00:00 b0716426367d992db8fb9a2235c0eb9c9fafe324 env_set() frees the previous value after allocating the new value. As the free() may merge memory chunks the available memory is not expected to stay constant. Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com> 
env_set() frees the previous value after allocating the new value.
As the free() may merge memory chunks the available memory is not
expected to stay constant.

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
test: clean up setexpr_test_str() 2025-02-07T19:35:23+00:00 Heinrich Schuchardt heinrich.schuchardt@canonical.com 2025-02-03T15:10:26+00:00 0395d75d6b867a3c109e5a71c3773f8415bf5121 Assign variable buf in the sub-test where it is used. Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com> 
Assign variable buf in the sub-test where it is used.

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
test: Drop the function for running setexpr tests 2025-01-24T20:34:40+00:00 Simon Glass sjg@chromium.org 2025-01-20T21:25:46+00:00 cebdc5376f9663254387c5c59c095f9f64b3eb99 Use the new suite-runner to run these tests instead. Signed-off-by: Simon Glass <sjg@chromium.org> 
Use the new suite-runner to run these tests instead.

Signed-off-by: Simon Glass <sjg@chromium.org>
test: Drop the _test suffix on linker lists 2025-01-24T20:34:40+00:00 Simon Glass sjg@chromium.org 2025-01-20T21:25:32+00:00 4ba3ab4901c564725c37e927e2ba29595712cd9d Most test suites have a _test suffix. This is not necessary as there is also a ut_ prefix. Drop the suffix so that (with future work) the suite name can be used as the linker-list name. Remove the suffix from the pytest regex as well, moving it to the top of the file, as it is a constant. Signed-off-by: Simon Glass <sjg@chromium.org> 
Most test suites have a _test suffix. This is not necessary as there is
also a ut_ prefix.

Drop the suffix so that (with future work) the suite name can be used as
the linker-list name.

Remove the suffix from the pytest regex as well, moving it to the top of
the file, as it is a constant.

Signed-off-by: Simon Glass <sjg@chromium.org>
test: Rename test suites to match their linker-list name 2025-01-24T20:34:40+00:00 Simon Glass sjg@chromium.org 2025-01-20T21:25:29+00:00 4d31a3bd3370f8fc9d1a484129770a4de4a15d70 Some suites have a different name from that used in the linker list. That makes it hard to programmatically match the name printed when the suite runs to the linker-list name it has. Update the names so they are the same. Signed-off-by: Simon Glass <sjg@chromium.org> 
Some suites have a different name from that used in the linker list.
That makes it hard to programmatically match the name printed when the
suite runs to the linker-list name it has.

Update the names so they are the same.

Signed-off-by: Simon Glass <sjg@chromium.org>
test: cmd: Drop unnecessary console_record_reset_enable() 2024-08-27T00:51:49+00:00 Simon Glass sjg@chromium.org 2024-08-22T13:57:58+00:00 0af38d12d98070bc09001e200b15c7e0d6a80388 It is seldom necessary to call this function. Drop its use in the command tests. Add a few extra checks to the wget test so that resetting is not needed. Signed-off-by: Simon Glass <sjg@chromium.org> 
It is seldom necessary to call this function. Drop its use in the
command tests.

Add a few extra checks to the wget test so that resetting is not
needed.

Signed-off-by: Simon Glass <sjg@chromium.org>