[Piglit] [PATCH v2] framework: Add --ignore-missing option

Martin Peres martin.peres at linux.intel.com
Fri Aug 4 11:24:40 UTC 2017


On 01/08/17 15:06, Arkadiusz Hiler wrote:
> Currently, if a test from provided testlist fails to be discovered by
> the framework, piglit blows up with an exception.
> 
> This is both good - for consistency/early errors - and bad - for
> handling some CI/automation scenarios (e.g autobisecting the tests).
> 
> So let's keep the current default, but allow some flexibility with the
> new option that reports the missing tests as 'notrun'.
> 
> v2:
>   - change switch name to 'ignore', as skip is too suggestive
>   - use DummyTest to get 'notrun' result instead of warnings
> 
> bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99649
> Cc: Dylan Baker <dylan at pnwbakers.com>
> Cc: Tomi Sarvela <tomi.p.sarvela at intel.com>
> Cc: Martin Peres <martin.peres at intel.com>
> Tested-by: Petri Latvala <petri.latvala at intel.com>
> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler at intel.com>

Sorry for the delay. Looks clean and neat to me:

Reviewed-by: Martin Peres <martin.peres at linux.intel.com>


More information about the Piglit mailing list