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

Unified Diff: scripts/slave/recipes/findit/chromium/compile.expected/compile_succeeded.json

Issue 1615963005: [Findit] Refactoring compile recipe return results into report dict instead of list (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build@master
Patch Set: Fixing whitespace Created 4 years, 11 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
Index: scripts/slave/recipes/findit/chromium/compile.expected/compile_succeeded.json
diff --git a/scripts/slave/recipes/findit/chromium/compile.expected/compile_succeeded.json b/scripts/slave/recipes/findit/chromium/compile.expected/compile_succeeded.json
index 31182980bf2d3f83c21311b9b8c82aceebb07d36..11b171ea6fa7b69c3c48c6e005d3e090a951a359 100644
--- a/scripts/slave/recipes/findit/chromium/compile.expected/compile_succeeded.json
+++ b/scripts/slave/recipes/findit/chromium/compile.expected/compile_succeeded.json
@@ -404,14 +404,16 @@
"name": "report",
"~followup_annotations": [
"@@@STEP_TEXT@<br/>Culprit: r1@@@",
- "@@@STEP_LOG_LINE@result@[@@@",
- "@@@STEP_LOG_LINE@result@ [@@@",
- "@@@STEP_LOG_LINE@result@ \"r1\", @@@",
- "@@@STEP_LOG_LINE@result@ \"failed\"@@@",
- "@@@STEP_LOG_LINE@result@ ]@@@",
- "@@@STEP_LOG_LINE@result@]@@@",
+ "@@@STEP_LOG_LINE@result@{@@@",
+ "@@@STEP_LOG_LINE@result@ \"result\": {@@@",
+ "@@@STEP_LOG_LINE@result@ \"r1\": \"failed\"@@@",
+ "@@@STEP_LOG_LINE@result@ }, @@@",
+ "@@@STEP_LOG_LINE@result@ \"metadata\": {@@@",
+ "@@@STEP_LOG_LINE@result@ \"regression_range_size\": 1@@@",
+ "@@@STEP_LOG_LINE@result@ }@@@",
+ "@@@STEP_LOG_LINE@result@}@@@",
"@@@STEP_LOG_END@result@@@",
- "@@@SET_BUILD_PROPERTY@result@[[\"r1\", \"failed\"]]@@@"
+ "@@@SET_BUILD_PROPERTY@result@{\"metadata\": {\"regression_range_size\": 1}, \"result\": {\"r1\": \"failed\"}}@@@"
]
},
{

Powered by Google App Engine
This is Rietveld 408576698