test/py: memtest: Fix test for non-trivial parameters
authorAndrew Goodbody <andrew.goodbody@linaro.org>
Fri, 28 Mar 2025 16:30:37 +0000 (16:30 +0000)
committerTom Rini <trini@konsulko.com>
Thu, 10 Apr 2025 17:57:59 +0000 (11:57 -0600)
commit3c6a3e99b398558b6bfb1362a44fbf0e07e2539e
treef88c73f753d9e79faaea5d9994b887d24beef5af
parent60218f07f357aeae34b524f0cdf2e512bda44645
test/py: memtest: Fix test for non-trivial parameters

When using non-trivial values for parameters for this test it
will cause a spurious failure as the test passes a decimal value
to the mtest command which will interpret it as hexadecimal and
result in failure as below.

test/py/tests/test_memtest.py:66: in test_memtest_ddr
    assert expected_response in response
E   AssertionError: assert 'Tested 16 iteration(s) with 0 errors.' in 'Refusing to do empty test\r\nmtest - simple RAM read/write test\r\n\r\nUsage:\r\nmtest [start [end [pattern [iterations]]]]'
----------------------------- Captured stdout call -----------------------------
U-Boot> mtest 134217728 0x8001000 90 0x10
Refusing to do empty test
mtest - simple RAM read/write test
Usage:
mtest [start [end [pattern [iterations]]]]

The fix is to ensure that all the parameters to the mtest command are
passed as hexadecimal values.

Fixes: 22efc1cf276c ("test/py: memtest: Add tests for mtest command")
Signed-off-by: Andrew Goodbody <andrew.goodbody@linaro.org>
Reviewed-by: Love Kumar <love.kumar@amd.com>
test/py/tests/test_memtest.py