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

Unified 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 side-by-side diff with in-line comments
Download patch
« recipe_modules/context/tests/env.py ('K') | « recipe_modules/step/config.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: recipe_modules/step/tests/inject_paths.expected/with_value.json
diff --git a/recipe_modules/step/tests/inject_paths.expected/with_value.json b/recipe_modules/step/tests/inject_paths.expected/with_value.json
index 6d0471e400ac49b66b66242dec3be2e06193a1a5..cdfd99184802ec30ff8765c7f0b606d3d6c73672 100644
--- a/recipe_modules/step/tests/inject_paths.expected/with_value.json
+++ b/recipe_modules/step/tests/inject_paths.expected/with_value.json
@@ -5,8 +5,13 @@
"hi"
],
"env": {
- "FOO": "bar",
- "PATH": "foo:bar:<PATH>"
+ "FOO": "bar"
+ },
+ "env_prefixes": {
+ "PATH": [
+ "foo",
+ "bar"
+ ]
},
"name": "test step (no env)"
},
@@ -16,7 +21,13 @@
"hi"
],
"env": {
- "PATH": "foo:bar:something"
+ "PATH": "something"
+ },
+ "env_prefixes": {
+ "PATH": [
+ "foo",
+ "bar"
+ ]
},
"name": "test step (env)"
},
@@ -26,7 +37,13 @@
"hi"
],
"env": {
- "PATH": "foo:bar:something:<PATH>"
+ "PATH": "something:<PATH>"
+ },
+ "env_prefixes": {
+ "PATH": [
+ "foo",
+ "bar"
+ ]
},
"name": "test step (env, $PATH)"
},
« 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