On 11/15/24 01:14, Tom Rini wrote:
> On Tue, Nov 05, 2024 at 01:06:53PM +0100, Jerome Forissier wrote:
>
>> Since net_test_wget() is testing a command and is in test/cmd it
>> should be in the 'cmd' test suite, not 'lib'.
>>
>> Reported-by: Simon Glass <[email protected]>
>> Signed-off-by: Jerome Forissier <[email protected]>
>> Reviewed-by: Simon Glass <[email protected]>
>> ---
>> test/cmd/wget.c | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> This leads to a sequencing error in tests:
> ------------------------------------ Captured stdout call
> -------------------------------------
> => ut dm dm_test_eth_act
> Test: dm_test_eth_act: eth.c
>
> Error: eth@10002000 No valid MAC address found.
>
> Error: eth@10002000 No valid MAC address found.
>
> Error: eth@10002000 No valid MAC address found.
>
> Error: eth@10002000 No valid MAC address found.
>
> Error: eth@10002000 No valid MAC address found.
>
> Error: eth@10004000 No valid MAC address found.
>
> Error: eth@10002000 No valid MAC address found.
>
> Error: eth@10004000 No valid MAC address found.
>
> Error: eth@10002000 No valid MAC address found.
>
> Error: eth@10002000 No valid MAC address found.
> test/dm/eth.c:284, dm_test_eth_act(): -ENODEV == net_loop(PING): Expected
> 0xffffffed (-19), got 0xffffffea (-22)
> Test: dm_test_eth_act: eth.c (flat tree)
> test/dm/eth.c:273, dm_test_eth_act(): addr = NULL: Expected non-null, got NULL
> Test 'dm_test_eth_act' failed 2 times
> Failures: 2
> =>
>
> Where it's fine when run by itself, but fails in CI/etc when running all
> unit tests.
How annoying ;) Any idea how to reproduce locally?
Thanks,
--
Jerome
