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

Side by Side Diff: recipe_modules/step/tests/inject_paths.expected/with_value.json

Issue 2913203002: [step_runner] run _merge_envs in simulation too. (Closed)
Patch Set: quotes and comments Created 3 years, 6 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/step/tests/inject_paths.expected/basic.json ('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
1 [ 1 [
2 { 2 {
3 "cmd": [ 3 "cmd": [
4 "echo", 4 "echo",
5 "hi" 5 "hi"
6 ], 6 ],
7 "env": { 7 "env": {
8 "FOO": "bar", 8 "FOO": "bar",
9 "PATH": "foo:bar:%(PATH)s" 9 "PATH": "foo:bar:<PATH>"
10 }, 10 },
11 "name": "test step (no env)" 11 "name": "test step (no env)"
12 }, 12 },
13 { 13 {
14 "cmd": [ 14 "cmd": [
15 "echo", 15 "echo",
16 "hi" 16 "hi"
17 ], 17 ],
18 "env": { 18 "env": {
19 "PATH": "foo:bar:something" 19 "PATH": "foo:bar:something"
20 }, 20 },
21 "name": "test step (env)" 21 "name": "test step (env)"
22 }, 22 },
23 { 23 {
24 "cmd": [ 24 "cmd": [
25 "echo", 25 "echo",
26 "hi" 26 "hi"
27 ], 27 ],
28 "env": { 28 "env": {
29 "PATH": "foo:bar:something:%(PATH)s" 29 "PATH": "foo:bar:something:<PATH>"
30 }, 30 },
31 "name": "test step (env, $PATH)" 31 "name": "test step (env, $PATH)"
32 }, 32 },
33 { 33 {
34 "name": "$result", 34 "name": "$result",
35 "recipe_result": null, 35 "recipe_result": null,
36 "status_code": 0 36 "status_code": 0
37 } 37 }
38 ] 38 ]
OLDNEW
« no previous file with comments | « recipe_modules/step/tests/inject_paths.expected/basic.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698