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

Side by Side Diff: recipe_modules/json/tests/add_json_log.expected/add_json_log.json

Issue 2989423002: Add add_json_log='on_failure' to json module. (Closed)
Patch Set: Add unittests Created 3 years, 4 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 unified diff | Download patch
« no previous file with comments | « recipe_modules/json/tests/add_json_log.py ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 [
2 {
3 "cmd": [
4 "cat",
5 "{\"x\":1,\"y\":2}"
6 ],
7 "name": "no log on success",
8 "stdout": "/path/to/tmp/json"
9 },
10 {
11 "cmd": [
12 "cat",
13 "{\"x\":1,\"y\":2}"
14 ],
15 "name": "add log on failure",
16 "stdout": "/path/to/tmp/json",
17 "~followup_annotations": [
18 "step returned non-zero exit code: 1",
19 "@@@STEP_LOG_LINE@json.output[log2]@{@@@",
20 "@@@STEP_LOG_LINE@json.output[log2]@ \"x\": 1, @@@",
21 "@@@STEP_LOG_LINE@json.output[log2]@ \"y\": 2@@@",
22 "@@@STEP_LOG_LINE@json.output[log2]@}@@@",
23 "@@@STEP_LOG_END@json.output[log2]@@@",
24 "@@@STEP_FAILURE@@@"
25 ]
26 },
27 {
28 "name": "$result",
29 "recipe_result": null,
30 "status_code": 0
31 }
32 ]
OLDNEW
« no previous file with comments | « recipe_modules/json/tests/add_json_log.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698