Fix bug in the parser when a block was silently skipped.

Some cosmetic changes with EnumSets.

--
MOS_MIGRATED_REVID=102742596

1 files changed, 10 insertions, 0 deletions

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 e0c074a126..b5a8015716 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
@@ -1242,6 +1242,16 @@ public class ParserTest extends EvaluationTestCase {
   }
 
   @Test
+  public void testElseWithoutIf() throws Exception {
+    setFailFast(false);
+    parseFileForSkylark(
+        "def func(a):",
+        // no if
+        "  else: return a");
+    assertContainsEvent("syntax error at 'else'");
+  }
+
+  @Test
   public void testIncludeFailureSkylark() throws Exception {
     setFailFast(false);
     parseFileForSkylark("include('//foo:bar')");