commit 65ecfea2f3e05b2372431d922a4d029bacd08ca0
author cparsons <cparsons@google.com> Wed Aug 28 14:37:23 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Aug 28 14:39:03 2019 -0700
tree c587b597fb0cca36e4445e56c16b3782ab64f154
parent c91fa30994b52de4ddab52071c94399e50880410

Flip --incompatible_restrict_named_params to true.

RELNOTES: Make a number of parameters of Starlark builtin functions positional-only (as opposed to specifiable by keyword). See https://github.com/bazelbuild/bazel/issues/8147 for details.
PiperOrigin-RevId: 265995993

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

diff --git a/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java b/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java
index a0f9cab..69de404 100644
--- a/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java
@@ -1103,31 +1103,27 @@
 
   @Test
   public void testLegacyNamed() throws Exception {
-    new SkylarkTest()
+    new SkylarkTest("--incompatible_restrict_named_params=false")
         .update("mock", new Mock())
-        .setUp(
-            "b = mock.legacy_method(True, legacyNamed=True, named=True)")
+        .setUp("b = mock.legacy_method(True, legacyNamed=True, named=True)")
         .testLookup("b", "legacy_method(true, true, true)");
 
-    new SkylarkTest()
+    new SkylarkTest("--incompatible_restrict_named_params=false")
         .update("mock", new Mock())
-        .setUp(
-            "b = mock.legacy_method(True, True, named=True)")
+        .setUp("b = mock.legacy_method(True, True, named=True)")
         .testLookup("b", "legacy_method(true, true, true)");
 
     // Verify legacyNamed also works with proxy method objects.
-    new SkylarkTest()
+    new SkylarkTest("--incompatible_restrict_named_params=false")
         .update("mock", new Mock())
         .setUp(
             "m = mock.proxy_methods_object()",
             "b = m.legacy_method(True, legacyNamed=True, named=True)")
         .testLookup("b", "legacy_method(true, true, true)");
 
-    new SkylarkTest()
+    new SkylarkTest("--incompatible_restrict_named_params=false")
         .update("mock", new Mock())
-        .setUp(
-            "m = mock.proxy_methods_object()",
-            "b = m.legacy_method(True, True, named=True)")
+        .setUp("m = mock.proxy_methods_object()", "b = m.legacy_method(True, True, named=True)")
         .testLookup("b", "legacy_method(true, true, true)");
   }