commit 317a269f17e0ebb3a5d210b80860b681ffbdd923
author fzaiser <fzaiser@google.com> Wed Aug 23 16:40:30 2017 +0200
committer Damien Martin-Guillerez <dmarting@google.com> Thu Aug 24 13:53:24 2017 +0200
tree 8b9e4dc4277d2f5a73f8c8fba5edddb922bb8e16
parent 5ce914f32f5b6a3459df29060b45e5ec8b0e9ecc

Refactor: Parse return statements without an expression properly

This is an internal refactoring necessary for the Skylark linter.
It does not change any behavior.

RELNOTES: None
PiperOrigin-RevId: 166199367

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 94cdb30..288038f 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
@@ -971,12 +971,7 @@
       assertThat(bodyNoExpr).hasSize(1);
 
       ReturnStatement returnNoExpr = (ReturnStatement) bodyNoExpr.get(0);
-      Identifier none = (Identifier) returnNoExpr.getReturnExpression();
-      assertThat(none.getName()).isEqualTo("None");
-      assertLocation(
-          returnNoExpr.getLocation().getStartOffset(),
-          returnNoExpr.getLocation().getEndOffset(),
-          none.getLocation());
+      assertThat(returnNoExpr.getReturnExpression()).isNull();
     }
   }