[Piglit] [PATCH] framework: Don't stop resume because no tests are left to be run

[Martin Peres]

On 11/07/17 20:27, Dylan Baker wrote:
> It's completely possible that if piglit resume is run with the -n flag
> (don't retry incomplete tests), that piglit resume could be invoked with
> no tests left to run. In this case it makes sense to let piglit resume
> just finalize the backend and produce a unified result.
> 
> Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
> cc: Martin Peres <martin.peres at linux.intel.com>

Thanks!

Tested-by: Hector Edmundo Ramirez Gomez 
<hector.edmundox.ramirez.gomez at intel.com>

> ---
>   framework/programs/run.py | 15 ++++++++++-----
>   1 file changed, 10 insertions(+), 5 deletions(-)
> 
> diff --git a/framework/programs/run.py b/framework/programs/run.py
> index 508bc05c4..ef268f722 100644
> --- a/framework/programs/run.py
> +++ b/framework/programs/run.py
> @@ -35,6 +35,7 @@ import six
>   
>   from framework import core, backends, exceptions, options
>   from framework import dmesg
> +from framework import exceptions
>   from framework import monitoring
>   from framework import profile
>   from framework.results import TimeAttribute
> @@ -435,11 +436,15 @@ def resume(input_):
>               p.forced_test_list = results.options['forced_test_list']
>   
>       # This is resumed, don't bother with time since it won't be accurate anyway
> -    profile.run(
> -        profiles,
> -        results.options['log_level'],
> -        backend,
> -        results.options['concurrent'])
> +    try:
> +        profile.run(
> +            profiles,
> +            results.options['log_level'],
> +            backend,
> +            results.options['concurrent'])
> +    except exceptions.PiglitUserError as e:
> +        if str(e) != 'no matching tests':
> +            raise
>   
>       backend.finalize()
>   
>