On Fri, Nov 15, 2024 at 11:53:59AM +0100, Jerome Forissier wrote: > > > 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?
Running all of the tests (or at least all of the "ut" unit tests) in sandbox should do it. -- Tom
signature.asc
Description: PGP signature
