commit d412c8f5d37c08ee780d6d1fdc01feb31b1a3bdc
author Laurent Le Brun <laurentlb@google.com> Tue Jun 16 11:12:54 2015 +0000
committer Kristina Chodorow <kchodorow@google.com> Tue Jun 16 14:00:08 2015 +0000
tree 2c20fdcb33a8a7b6ba661181ec548db83cbf4609
parent ee991a17299822b1ae601fd337e769e3ff513007

Fix NullPointerException in parser.

Location was not set.

--
MOS_MIGRATED_REVID=96096088

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

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 fd8478d..3658679 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
@@ -453,6 +453,27 @@
   }
 
   @Test
+  public void testForBreakContinue() throws Exception {
+    List<Statement> file = parseFileForSkylark(
+        "def foo():",
+        "  for i in [1, 2]:",
+        "    break",
+        "    continue");
+    assertThat(file).hasSize(1);
+    List<Statement> body = ((FunctionDefStatement) file.get(0)).getStatements();
+    assertThat(body).hasSize(1);
+
+    List<Statement> loop = ((ForStatement) body.get(0)).block();
+    assertThat(loop).hasSize(2);
+
+    assertThat(loop.get(0)).isEqualTo(FlowStatement.BREAK);
+    assertLocation(34, 40, loop.get(0).getLocation());
+
+    assertThat(loop.get(1)).isEqualTo(FlowStatement.CONTINUE);
+    assertLocation(44, 52, loop.get(1).getLocation());
+  }
+
+  @Test
   public void testListLiterals1() throws Exception {
     ListLiteral list = (ListLiteral) parseExpression("[0,1,2]");
     assertFalse(list.isTuple());