Simple test for standalone Skylark binary

--
PiperOrigin-RevId: 147177381
MOS_MIGRATED_REVID=147177381

1 files changed, 7 insertions, 3 deletions

diff --git a/src/main/java/com/google/devtools/skylark/Skylark.java b/src/main/java/com/google/devtools/skylark/Skylark.java
index 24cae3a1a3..1a0f9c9deb 100644
--- a/src/main/java/com/google/devtools/skylark/Skylark.java
+++ b/src/main/java/com/google/devtools/skylark/Skylark.java
@@ -89,24 +89,28 @@ class Skylark {
   }
 
   /** Execute a Skylark file. */
-  public void execute(String path) {
+  public int execute(String path) {
     String content;
     try {
       content = new String(Files.readAllBytes(Paths.get(path)), StandardCharsets.UTF_8);
       BuildFileAST.eval(env, content);
+      return 0;
     } catch (Exception e) {
       System.err.println(e);
+      return 1;
     }
   }
 
   public static void main(String[] args) {
+    int ret = 0;
     if (args.length == 0) {
       new Skylark().readEvalPrintLoop();
     } else if (args.length == 1) {
-      new Skylark().execute(args[0]);
+      ret = new Skylark().execute(args[0]);
     } else {
       System.err.println("too many arguments");
-      System.exit(1);
+      ret = 1;
     }
+    System.exit(ret);
   }
 }