commit 6874316e27af4074ad98b361e050b1aa13b5611b
author Laurent Le Brun <laurentlb@google.com> Wed May 13 13:18:09 2015 +0000
committer Damien Martin-Guillerez <dmarting@google.com> Fri May 15 09:43:37 2015 +0000
tree d3e3d10b56b5a72931f19cbd7a22750830ab8ec9
parent 463508baa66aa93785b7b3ccf4764cca765e32c6

Skylark: Functions don't need to be declared in order.

--
MOS_MIGRATED_REVID=93515487

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

diff --git a/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java b/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java
index 579268f..43e5ac8 100644
--- a/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java
@@ -664,6 +664,22 @@
   }
 
   @Test
+  public void testFunctionCallOrdering() throws Exception {
+    eval("def func(): return foo() * 2",
+         "def foo(): return 2",
+         "x = func()");
+    assertThat(lookup("x")).isEqualTo(4);
+  }
+
+  @Test
+  public void testFunctionCallBadOrdering() throws Exception {
+    checkEvalError("name 'foo' is not defined",
+         "def func(): return foo() * 2",
+         "x = func()",
+         "def foo(): return 2");
+  }
+
+  @Test
   public void testNoneTrueFalseInSkylark() throws Exception {
     eval("a = None",
       "b = True",