V8: Reduce test expectations by whitelisting

V8: Reduce test expectations by whitelisting

BUG=

[Michael Achenbach, 2016-10-17 12:45:54]

The CQ bit was checked by machenbach@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-17 12:45:56]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-17 13:23:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-17 13:23:58]

Dry run: Try jobs failed on following builders:
  Build Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/31eb7f12260e7e10)

[iannucci1, 2016-11-01 20:21:50]

iannucci@google.com changed reviewers:
+ iannucci@google.com

[iannucci1, 2016-11-01 20:21:51]

https://codereview.chromium.org/2366213005/diff/40001/scripts/slave/recipes/v...
File scripts/slave/recipes/v8.py (right):

https://codereview.chromium.org/2366213005/diff/40001/scripts/slave/recipes/v...
scripts/slave/recipes/v8.py:129: api.post_process(Filter('$result'))
Hm... I would expect this to remove all of the steps from your expectation.
You're telling it:
  * Remove all the steps except 'Check'
  * THEN Remove all the steps except 'Test262'
  * THEN Remove all the steps except '$result'

Maybe you want `Filter('Check', 'Test262', '$result')`?

I'm surprised that it's not completely removing this expectation file, though.
I'll look into that.

[Michael Achenbach, 2016-11-01 20:25:20]

https://codereview.chromium.org/2366213005/diff/40001/scripts/slave/recipes/v...
File scripts/slave/recipes/v8.py (right):

https://codereview.chromium.org/2366213005/diff/40001/scripts/slave/recipes/v...
scripts/slave/recipes/v8.py:129: api.post_process(Filter('$result'))
On 2016/11/01 20:21:51, iannucci1 wrote:
> Hm... I would expect this to remove all of the steps from your expectation.
> You're telling it:
>   * Remove all the steps except 'Check'
>   * THEN Remove all the steps except 'Test262'
>   * THEN Remove all the steps except '$result'
> 
> Maybe you want `Filter('Check', 'Test262', '$result')`?
> 
> I'm surprised that it's not completely removing this expectation file, though.
> I'll look into that.

Ah, yes you're right! Will change that.

[Michael Achenbach, 2016-11-01 20:38:46]

machenbach@chromium.org changed reviewers:
+ tandrii@chromium.org

[Michael Achenbach, 2016-11-01 20:38:46]

Nice, looks as expected now.

I'll add a few more filters before landing.

[tandrii(chromium), 2016-11-02 23:08:58]

https://codereview.chromium.org/2366213005/diff/60001/scripts/slave/recipes/v...
File scripts/slave/recipes/v8.py (right):

https://codereview.chromium.org/2366213005/diff/60001/scripts/slave/recipes/v...
scripts/slave/recipes/v8.py:128: )
and that's it? wow!