Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(927)

Unified Diff: scripts/slave/recipes/v8.py

Issue 2366213005: V8: Reduce test expectations by whitelisting
Patch Set: New post process Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux_swarming_collect_failure.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipes/v8.py
diff --git a/scripts/slave/recipes/v8.py b/scripts/slave/recipes/v8.py
index 4c969316865efb605aedbb250ae82586c59eca5a..a564f9b54798e86e5cc8f37c1f0c241ec67e1a7e 100644
--- a/scripts/slave/recipes/v8.py
+++ b/scripts/slave/recipes/v8.py
@@ -2,6 +2,8 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
+from recipe_engine.post_process import Filter
+
DEPS = [
'archive',
'chromium',
@@ -121,7 +123,10 @@ def GenTests(api):
'V8 Linux',
'swarming_collect_failure',
) +
- api.step_data('Check', retcode=1)
+ api.step_data('Check', retcode=1) +
+ api.post_process(Filter('Check')) +
+ api.post_process(Filter('Test262')) +
+ api.post_process(Filter('$result'))
iannucci1 2016/11/01 20:21:51 Hm... I would expect this to remove all of the ste
Michael Achenbach 2016/11/01 20:25:20 Ah, yes you're right! Will change that.
)
# Simulate a tryjob triggered by the CQ for setting up different swarming
« no previous file with comments | « no previous file | scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux_swarming_collect_failure.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698