[PATCH v3 06/19] test: Avoid failing skipped tests

Tom Rini trini at konsulko.com
Mon Jun 24 20:05:56 CEST 2024


On Sun, Jun 23, 2024 at 02:32:00PM -0600, Simon Glass wrote:

> When a test returns -EAGAIN this should not be considered a failure.
> Fix what seems to be a problem case, where the pytests see a failure
> when a test has merely been skipped.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> 
> (no changes since v1)
> 
>  test/test-main.c | 16 +++++++++++-----
>  1 file changed, 11 insertions(+), 5 deletions(-)
> 
> diff --git a/test/test-main.c b/test/test-main.c
> index 3fa6f6e32ec..cda1a186390 100644
> --- a/test/test-main.c
> +++ b/test/test-main.c
> @@ -448,7 +448,7 @@ static int ut_run_test(struct unit_test_state *uts, struct unit_test *test,
>  static int ut_run_test_live_flat(struct unit_test_state *uts,
>  				 struct unit_test *test)
>  {
> -	int runs;
> +	int runs, ret;
>  
>  	if ((test->flags & UT_TESTF_OTHER_FDT) && !IS_ENABLED(CONFIG_SANDBOX))
>  		return skip_test(uts);
> @@ -458,8 +458,11 @@ static int ut_run_test_live_flat(struct unit_test_state *uts,
>  	if (CONFIG_IS_ENABLED(OF_LIVE)) {
>  		if (!(test->flags & UT_TESTF_FLAT_TREE)) {
>  			uts->of_live = true;
> -			ut_assertok(ut_run_test(uts, test, test->name));
> -			runs++;
> +			ret = ut_run_test(uts, test, test->name);
> +			if (ret != -EAGAIN) {
> +				ut_assertok(ret);
> +				runs++;
> +			}
>  		}
>  	}
>  
> @@ -483,8 +486,11 @@ static int ut_run_test_live_flat(struct unit_test_state *uts,
>  	    (!runs || ut_test_run_on_flattree(test)) &&
>  	    !(gd->flags & GD_FLG_FDT_CHANGED)) {
>  		uts->of_live = false;
> -		ut_assertok(ut_run_test(uts, test, test->name));
> -		runs++;
> +		ret = ut_run_test(uts, test, test->name);
> +		if (ret != -EAGAIN) {
> +			ut_assertok(ret);
> +			runs++;
> +		}
>  	}
>  
>  	return 0;

How did you trigger this case exactly?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240624/51846f05/attachment.sig>


More information about the U-Boot mailing list