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

Unified Diff: recipe_modules/context/tests/env.expected/basic.json

Issue 2933473002: [context] Split "env" values and prefixes. (Closed)
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: recipe_modules/context/tests/env.expected/basic.json
diff --git a/recipe_modules/context/tests/env.expected/basic.json b/recipe_modules/context/tests/env.expected/basic.json
index 731ce1aa3c078c668b03a668be3c0d23b8db6ae7..72b04c4f0126ae0bfc01debbe5ff5cac049e1718 100644
--- a/recipe_modules/context/tests/env.expected/basic.json
+++ b/recipe_modules/context/tests/env.expected/basic.json
@@ -31,56 +31,129 @@
"cmd": [
"bash",
"-c",
- "echo $FOO"
+ "echo -n $RECIPE_ENGINE_CONTEXT_TEST"
+ ],
+ "name": "default",
+ "stdout": "/path/to/tmp/"
+ },
+ {
+ "cmd": [
+ "bash",
+ "-c",
+ "echo -n $RECIPE_ENGINE_CONTEXT_TEST"
],
"env": {
- "FOO": "bar"
+ "RECIPE_ENGINE_CONTEXT_TEST": null
},
- "name": "env step"
+ "name": "drop",
+ "stdout": "/path/to/tmp/"
},
{
"cmd": [
"bash",
"-c",
- "echo $FOO"
+ "echo -n $RECIPE_ENGINE_CONTEXT_TEST"
],
"env": {
- "FOO": "foo:<FOO>:baz"
+ "RECIPE_ENGINE_CONTEXT_TEST": "bar"
},
- "name": "env step augmented"
+ "name": "env step",
+ "stdout": "/path/to/tmp/"
},
{
"cmd": [
"bash",
"-c",
- "echo $FOO"
+ "echo -n $RECIPE_ENGINE_CONTEXT_TEST"
],
"env": {
- "FOO": "[START_DIR]/pants:[START_DIR]/shirt:foo:<FOO>:baz"
+ "RECIPE_ENGINE_CONTEXT_TEST": "foo:<RECIPE_ENGINE_CONTEXT_TEST>:bar"
},
- "name": "env step with prefix"
+ "name": "env step augmented",
+ "stdout": "/path/to/tmp/"
},
{
"cmd": [
"bash",
"-c",
- "echo $FOO"
+ "echo -n $RECIPE_ENGINE_CONTEXT_TEST"
],
"env": {
- "FOO": "[START_DIR]/shirt"
+ "RECIPE_ENGINE_CONTEXT_TEST": "foo:<RECIPE_ENGINE_CONTEXT_TEST>:bar"
+ },
+ "env_prefixes": {
+ "RECIPE_ENGINE_CONTEXT_TEST": [
+ "[START_DIR]/pants",
+ "[START_DIR]/shirt"
+ ]
},
- "name": "env with default value"
+ "name": "env step with prefix",
+ "stdout": "/path/to/tmp/"
},
{
"cmd": [
"bash",
"-c",
- "set"
+ "echo -n $RECIPE_ENGINE_CONTEXT_TEST"
+ ],
+ "env_prefixes": {
+ "RECIPE_ENGINE_CONTEXT_TEST": [
+ "[START_DIR]/shirt"
+ ]
+ },
+ "name": "env with default value",
+ "stdout": "/path/to/tmp/"
+ },
+ {
+ "cmd": [
+ "bash",
+ "-c",
+ "echo -n $RECIPE_ENGINE_CONTEXT_TEST"
],
"env": {
- "PYTHONPATH": null
+ "RECIPE_ENGINE_CONTEXT_TEST": "foo"
+ },
+ "env_prefixes": {
+ "RECIPE_ENGINE_CONTEXT_TEST": [
+ "[START_DIR]/shirt"
+ ]
+ },
+ "name": "env with override value",
+ "stdout": "/path/to/tmp/"
+ },
+ {
+ "cmd": [
+ "bash",
+ "-c",
+ "echo -n $RECIPE_ENGINE_CONTEXT_TEST"
+ ],
+ "env": {
+ "RECIPE_ENGINE_CONTEXT_TEST": null
+ },
+ "env_prefixes": {
+ "RECIPE_ENGINE_CONTEXT_TEST": [
+ "[START_DIR]/shirt"
+ ]
+ },
+ "name": "env with cleared value",
+ "stdout": "/path/to/tmp/"
+ },
+ {
+ "cmd": [
+ "bash",
+ "-c",
+ "echo -n $RECIPE_ENGINE_CONTEXT_TEST"
+ ],
+ "env": {
+ "RECIPE_ENGINE_CONTEXT_TEST": ""
+ },
+ "env_prefixes": {
+ "RECIPE_ENGINE_CONTEXT_TEST": [
+ "[START_DIR]/shirt"
+ ]
},
- "name": "drop PYTHONPATH"
+ "name": "env with empty value",
+ "stdout": "/path/to/tmp/"
},
{
"name": "$result",

Powered by Google App Engine
This is Rietveld 408576698