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

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

Issue 2933473002: [context] Split "env" values and prefixes. (Closed)
Patch Set: plumb pathsep through so we don't assume OS 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 "PATH": "foo:bar:<PATH>" 9 },
10 "env_prefixes": {
11 "PATH": [
12 "foo",
13 "bar"
14 ]
10 }, 15 },
11 "name": "test step (no env)" 16 "name": "test step (no env)"
12 }, 17 },
13 { 18 {
14 "cmd": [ 19 "cmd": [
15 "echo", 20 "echo",
16 "hi" 21 "hi"
17 ], 22 ],
18 "env": { 23 "env": {
19 "PATH": "foo:bar:something" 24 "PATH": "something"
25 },
26 "env_prefixes": {
27 "PATH": [
28 "foo",
29 "bar"
30 ]
20 }, 31 },
21 "name": "test step (env)" 32 "name": "test step (env)"
22 }, 33 },
23 { 34 {
24 "cmd": [ 35 "cmd": [
25 "echo", 36 "echo",
26 "hi" 37 "hi"
27 ], 38 ],
28 "env": { 39 "env": {
29 "PATH": "foo:bar:something:<PATH>" 40 "PATH": "something:<PATH>"
41 },
42 "env_prefixes": {
43 "PATH": [
44 "foo",
45 "bar"
46 ]
30 }, 47 },
31 "name": "test step (env, $PATH)" 48 "name": "test step (env, $PATH)"
32 }, 49 },
33 { 50 {
34 "name": "$result", 51 "name": "$result",
35 "recipe_result": null, 52 "recipe_result": null,
36 "status_code": 0 53 "status_code": 0
37 } 54 }
38 ] 55 ]
OLDNEW
« recipe_modules/context/tests/env.py ('K') | « recipe_modules/step/config.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698