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

Side by Side Diff: recipe_modules/step/tests/inject_paths.expected/basic.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
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 }, 9 },
10 "name": "test step (no env)" 10 "name": "test step (no env)"
11 }, 11 },
12 { 12 {
13 "cmd": [ 13 "cmd": [
14 "echo", 14 "echo",
15 "hi" 15 "hi"
16 ], 16 ],
17 "env": { 17 "env": {
18 "PATH": "something" 18 "PATH": "something"
19 }, 19 },
20 "name": "test step (env)" 20 "name": "test step (env)"
21 }, 21 },
22 { 22 {
23 "cmd": [ 23 "cmd": [
24 "echo", 24 "echo",
25 "hi" 25 "hi"
26 ], 26 ],
27 "env": { 27 "env": {
28 "PATH": "something:%(PATH)s" 28 "PATH": "something:<PATH>"
29 }, 29 },
30 "name": "test step (env, $PATH)" 30 "name": "test step (env, $PATH)"
31 }, 31 },
32 { 32 {
33 "name": "$result", 33 "name": "$result",
34 "recipe_result": null, 34 "recipe_result": null,
35 "status_code": 0 35 "status_code": 0
36 } 36 }
37 ] 37 ]
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698