Refactor FuncallExpression to allow for complex function terms later.
RELNOTES: None.
PiperOrigin-RevId: 164981071
diff --git a/src/test/java/com/google/devtools/build/lib/syntax/ParserTest.java b/src/test/java/com/google/devtools/build/lib/syntax/ParserTest.java
index fc5e586..c616b2a 100644
--- a/src/test/java/com/google/devtools/build/lib/syntax/ParserTest.java
+++ b/src/test/java/com/google/devtools/build/lib/syntax/ParserTest.java
@@ -176,7 +176,7 @@
public void testFuncallExpr() throws Exception {
FuncallExpression e = (FuncallExpression) parseExpression("foo(1, 2, bar=wiz)");
- Identifier ident = e.getFunction();
+ Identifier ident = (Identifier) e.getFunction();
assertThat(ident.getName()).isEqualTo("foo");
assertThat(e.getArguments()).hasSize(3);
@@ -199,8 +199,8 @@
FuncallExpression e =
(FuncallExpression) parseExpression("foo.foo(1, 2, bar=wiz)");
- Identifier ident = e.getFunction();
- assertThat(ident.getName()).isEqualTo("foo");
+ DotExpression dotExpression = (DotExpression) e.getFunction();
+ assertThat(dotExpression.getField().getName()).isEqualTo("foo");
assertThat(e.getArguments()).hasSize(3);
assertThat(e.getNumPositionalArguments()).isEqualTo(2);
@@ -222,8 +222,8 @@
FuncallExpression e =
(FuncallExpression) parseExpression("foo.replace().split(1)");
- Identifier ident = e.getFunction();
- assertThat(ident.getName()).isEqualTo("split");
+ DotExpression dotExpr = (DotExpression) e.getFunction();
+ assertThat(dotExpr.getField().getName()).isEqualTo("split");
assertThat(e.getArguments()).hasSize(1);
assertThat(e.getNumPositionalArguments()).isEqualTo(1);
@@ -244,8 +244,8 @@
public void testStringMethExpr() throws Exception {
FuncallExpression e = (FuncallExpression) parseExpression("'foo'.foo()");
- Identifier ident = e.getFunction();
- assertThat(ident.getName()).isEqualTo("foo");
+ DotExpression dotExpression = (DotExpression) e.getFunction();
+ assertThat(dotExpression.getField().getName()).isEqualTo("foo");
assertThat(e.getArguments()).isEmpty();
}
@@ -291,7 +291,8 @@
FuncallExpression e = (FuncallExpression) parseExpression(
"'FOO.CC'.lower()[1:].startswith('oo')");
- assertThat(e.getFunction().getName()).isEqualTo("startswith");
+ DotExpression dotExpression = (DotExpression) e.getFunction();
+ assertThat(dotExpression.getField().getName()).isEqualTo("startswith");
assertThat(e.getArguments()).hasSize(1);
s = (SliceExpression) parseExpression("'FOO.CC'[1:][:2]");
@@ -336,7 +337,7 @@
// Test that the actual parameters are: (1, $error$, 3):
- Identifier ident = e.getFunction();
+ Identifier ident = (Identifier) e.getFunction();
assertThat(ident.getName()).isEqualTo("f");
assertThat(e.getArguments()).hasSize(3);