From 630889baec86ab8edfd554566034155a5af7f713 Mon Sep 17 00:00:00 2001 From: Jack Conradson Date: Fri, 1 Feb 2019 15:10:59 -0800 Subject: [PATCH] Remove extraneous test from Painless lambda tests (#38111) This test has been awaiting a fix that isn't currently relevant because incoming lambda parameters are read-only. If this ever changes a new set of tests can be added that are up-to-date. --- .../test/java/org/elasticsearch/painless/LambdaTests.java | 6 ------ 1 file changed, 6 deletions(-) diff --git a/modules/lang-painless/src/test/java/org/elasticsearch/painless/LambdaTests.java b/modules/lang-painless/src/test/java/org/elasticsearch/painless/LambdaTests.java index 1f1a6f95b36..184e97c4a47 100644 --- a/modules/lang-painless/src/test/java/org/elasticsearch/painless/LambdaTests.java +++ b/modules/lang-painless/src/test/java/org/elasticsearch/painless/LambdaTests.java @@ -144,12 +144,6 @@ public class LambdaTests extends ScriptTestCase { assertTrue(expected.getMessage().contains("is read-only")); } - @AwaitsFix(bugUrl = "def type tracking") - public void testOnlyCapturesAreReadOnly() { - assertEquals(4, exec("List l = new ArrayList(); l.add(1); l.add(1); " - + "return l.stream().mapToInt(x -> { x += 1; return x }).sum();")); - } - /** Lambda parameters shouldn't be able to mask a variable already in scope */ public void testNoParamMasking() { IllegalArgumentException expected = expectScriptThrows(IllegalArgumentException.class, () -> {