commit c32efc8375cf482e795a43ecf4f009971722e71a
author laurentlb <laurentlb@google.com> Fri Jun 23 16:03:00 2017 +0200
committer Marcel Hlopko <hlopko@google.com> Mon Jun 26 18:31:16 2017 +0200
tree 6d52f53c789b0d5437d78ebf922bbcf271ef695e
parent 1d8cd59173e9c1e2fd7fd03dd4b2a0ae8a35ef4b

Make len(depset()) fail when --incompatible_depset_is_not_iterable is set

RELNOTES: None.
PiperOrigin-RevId: 159945244

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 a8248ea..e6224fa 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
@@ -886,7 +886,8 @@
         .testIfErrorContains("not iterable", "max(depset([1, 2, 3]))")
         .testIfErrorContains("not iterable", "sorted(depset(['a', 'b']))")
         .testIfErrorContains("not iterable", "tuple(depset(['a', 'b']))")
-        .testIfErrorContains("not iterable", "[x for x in depset()]");
+        .testIfErrorContains("not iterable", "[x for x in depset()]")
+        .testIfErrorContains("not iterable", "len(depset(['a']))");
   }
 
   @Test
@@ -896,7 +897,8 @@
         .testStatement("max(depset([1, 2, 3]))", 3)
         .testStatement("str(sorted(depset(['b', 'a'])))", "[\"a\", \"b\"]")
         .testStatement("str(tuple(depset(['a', 'b'])))", "(\"a\", \"b\")")
-        .testStatement("str([x for x in depset()])", "[]");
+        .testStatement("str([x for x in depset()])", "[]")
+        .testStatement("len(depset(['a']))", 1);
   }
 
   @Test