commit ceae8c50e8f92c6fbf2394ac7c5eb3b420539225
author Han-Wen Nienhuys <hanwen@google.com> Tue Sep 22 16:24:45 2015 +0000
committer Laszlo Csomor <laszlocsomor@google.com> Tue Sep 22 17:19:53 2015 +0000
tree 9b10bab2b8e06b61ff268a422ee2f27e09f10382
parent 4671896be8bf0e37c85c3b740bb3621d2a9e1cc8

Open source some skylark tests.

--
MOS_MIGRATED_REVID=103652672

src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleClassFunctions.java

diff --git a/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleClassFunctions.java b/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleClassFunctions.java
index f3ea46c..69f8e01 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleClassFunctions.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleClassFunctions.java
@@ -320,7 +320,7 @@
   };
 
   // This class is needed for testing
-  static final class RuleFunction extends BaseFunction {
+  public static final class RuleFunction extends BaseFunction {
     // Note that this means that we can reuse the same builder.
     // This is fine since we don't modify the builder from here.
     private final RuleClass.Builder builder;
@@ -367,7 +367,7 @@
     }
 
     @VisibleForTesting
-    RuleClass.Builder getBuilder() {
+    public RuleClass.Builder getBuilder() {
       return builder;
     }
   }

src/main/java/com/google/devtools/build/lib/syntax/Environment.java

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/Environment.java b/src/main/java/com/google/devtools/build/lib/syntax/Environment.java
index c283157..37769bb 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/Environment.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/Environment.java
@@ -393,7 +393,8 @@
   // TODO(bazel-team): Delete this function.
   // This function is currently used in various functions that change their behavior with respect to
   // lists depending on the Skylark-ness of the code; lists should be unified between the two modes.
-  boolean isSkylark() {
+  @VisibleForTesting
+  public boolean isSkylark() {
     return isSkylark;
   }
 
@@ -900,7 +901,7 @@
    * @param input a list of lines of code
    */
   @VisibleForTesting
-  List<Statement> parseFile(String... input) {
+  public List<Statement> parseFile(String... input) {
     return parseFileWithComments(input).statements;
   }
 

src/main/java/com/google/devtools/build/lib/syntax/Parser.java

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/Parser.java b/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
index fb95228..a385e25 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
@@ -45,9 +45,9 @@
  * Recursive descent parser for LL(2) BUILD language.
  * Loosely based on Python 2 grammar.
  * See https://docs.python.org/2/reference/grammar.html
- *
  */
-class Parser {
+@VisibleForTesting
+public class Parser {
 
   /**
    * Combines the parser result into a single value object.