u-boot.git/test/cmd/setexpr.c, branch v2025.04 Unnamed repository; edit this file 'description' to name the repository. 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>
test: Rename UTF_CONSOLE_REC to UTF_CONSOLE 2024-08-27T00:51:49+00:00 Simon Glass sjg@chromium.org 2024-08-22T13:57:50+00:00 9b99762eff5eac971cd1ae561e443252afb7b985 The _REC suffix doesn't add much. Really what we want to know is whether the test uses the console, so rename this flag. Signed-off-by: Simon Glass <sjg@chromium.org> Reviewed-by: Mattijs Korpershoek <mkorpershoek@baylibre.com> 
The _REC suffix doesn't add much. Really what we want to know is whether
the test uses the console, so rename this flag.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Mattijs Korpershoek <mkorpershoek@baylibre.com>
test: Drop the blank line before test macros 2024-08-27T00:51:49+00:00 Simon Glass sjg@chromium.org 2024-08-22T13:57:49+00:00 b073d48e8dd7d178ebd237089be730d15c72ddfc Most tests don't have this. It helps to keep the test declaration clearly associated with the function it relates to, rather than the next one in the file. Remove the extra blank line and mention this in the docs. Signed-off-by: Simon Glass <sjg@chromium.org> 
Most tests don't have this. It helps to keep the test declaration
clearly associated with the function it relates to, rather than the next
one in the file. Remove the extra blank line and mention this in the
docs.

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