diff options
| author |  ulfjack <ulfjack@google.com> | 2017-11-03 23:01:12 +0100 |
|---|---|---|
| committer |  Damien Martin-Guillerez <dmarting@google.com> | 2017-11-06 20:20:20 +0100 |
| commit | e8d32b7c922f65539b74357711d5ad6b70934115 (patch) | |
| tree | 599ed722a9cbe7e0faaa3254e51bbd4e9be51424 /src/test/java/com/google/devtools/build/lib/bazel | |
| parent | 5b40660ab4b8557d6b0588044b2c732f64067e1b (diff) | |
Switch ExtraActionFactory and GenRuleBase to the Expander API
This changes the order in which expansions happen, which could change the
semantics in subtle ways.
PiperOrigin-RevId: 174518778
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/bazel')
| -rw-r--r-- | src/test/java/com/google/devtools/build/lib/bazel/rules/genrule/GenRuleCommandSubstitutionTest.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/bazel/rules/genrule/GenRuleCommandSubstitutionTest.java b/src/test/java/com/google/devtools/build/lib/bazel/rules/genrule/GenRuleCommandSubstitutionTest.java index 8b2b1d4eaa..bef6e1c4b8 100644 --- a/src/test/java/com/google/devtools/build/lib/bazel/rules/genrule/GenRuleCommandSubstitutionTest.java +++ b/src/test/java/com/google/devtools/build/lib/bazel/rules/genrule/GenRuleCommandSubstitutionTest.java @@ -93,7 +93,7 @@ public class GenRuleCommandSubstitutionTest extends BuildViewTestCase { eventCollector.clear(); genrule("$(location foo bar"); - assertExpansionFails("unterminated $(location) expression", "//test"); + assertExpansionFails("unterminated variable reference", "//test"); genrule("$(location"); assertExpansionFails("unterminated variable reference", "//test"); @@ -221,7 +221,7 @@ public class GenRuleCommandSubstitutionTest extends BuildViewTestCase { eventCollector.clear(); genrule("$(locations foo bar"); - assertExpansionFails("unterminated $(locations) expression", "//test"); + assertExpansionFails("unterminated variable reference", "//test"); genrule("$(locations"); assertExpansionFails("unterminated variable reference", "//test"); |