[Piglit] [PATCH v2 11/11] summary.py: Change skip to N/A for tests not run

[Kenneth Graunke]

On 05/17/2013 09:32 AM, Dylan Baker wrote:
> When comparing two test results the current behavior sets a status of
> skip for tests that were not run in one set of results. This is
> confusing and silly, so instead give them a status of N/A which gives a
> clear understanding of what skipped, and what simply didn't run either
> by exclusion or because it didn't exist yet.
>
> Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>
> ---
>   framework/summary.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/framework/summary.py b/framework/summary.py
> index 6780458..0702319 100644
> --- a/framework/summary.py
> +++ b/framework/summary.py
> @@ -417,7 +417,7 @@ class BuildHTMLIndex(list):
>                       self._testResult(each.name, key, each.tests[key]['result'])
>                   except KeyError:
>                       self.append({'type': 'other',
> -                                 'text': '<td class="skip">skip</td>'})
> +                                 'text': '<td class="skip">N/A</td>'})
>               self._endRow()
>
>       def _newRow(self):

Changing this sounds good, but could we perhaps call it "not run"?

I'm afraid I might get confused about whether "N/A" means "not run" or 
"this test isn't relevant because it's not supported by your driver".