commit 537a90b5b90856788412c962de61ece14e83274b
author Francois-Rene Rideau <tunes@google.com> Wed Apr 22 06:47:31 2015 +0000
committer Laurent Le Brun <laurentlb@google.com> Wed Apr 22 12:42:37 2015 +0000
tree 08b489df7f200b6fa59fee0adc857245f78f88e2
parent 443aaae842aaf5009c2972277a1089c504873877

Use BuiltinFunction for all builtins

Replace the uses of AbstractFunction, MixedModeFunction,
SkylarkFunction and SimpleSkylarkFunction by BuiltinFunction.

--
MOS_MIGRATED_REVID=91763158

src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java b/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java
index cc14802..8ca1e9e 100644
--- a/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java
@@ -558,13 +558,13 @@
 
   @Test
   public void testDictKeysTooManyArgs() throws Exception {
-    checkEvalError("Invalid number of arguments (expected 0)",
+    checkEvalError("too many (2) positional arguments in call to keys(self: dict)",
         "{'a': 1}.keys('abc')");
   }
 
   @Test
   public void testDictKeysTooManyKeyArgs() throws Exception {
-    checkEvalError("Invalid number of arguments (expected 0)",
+    checkEvalError("unexpected keyword 'arg' in call to keys(self: dict)",
         "{'a': 1}.keys(arg='abc')");
   }
 
@@ -576,8 +576,8 @@
 
   @Test
   public void testArgBothPosKey() throws Exception {
-    checkEvalError("replace(this, old, new, maxsplit = null) got multiple values "
-        + "for keyword argument 'new'",
-        "'banana'.replace('a', 'o', 3, new=4)");
+    checkEvalErrorStartsWith("arguments 'old', 'new' passed both by position and by name "
+        + "in call to replace(self: string, ",
+        "'banana'.replace('a', 'o', 3, old='a', new=4)");
   }
 }