Painless: Add public member read/write access test. (#28156)

elastic · Jan 9, 2018 · 40f4a4a · 40f4a4a
1 parent 77e4a53
commit 40f4a4a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 0 deletions.
diff --git a/modules/lang-painless/src/main/java/org/elasticsearch/painless/FeatureTest.java b/modules/lang-painless/src/main/java/org/elasticsearch/painless/FeatureTest.java
@@ -26,6 +26,7 @@
 public class FeatureTest {
     private int x;
     private int y;
+    public int z;
 
     /** empty ctor */
     public FeatureTest() {

diff --git a/modules/lang-painless/src/main/resources/org/elasticsearch/painless/org.elasticsearch.txt b/modules/lang-painless/src/main/resources/org/elasticsearch/painless/org.elasticsearch.txt
@@ -139,6 +139,7 @@ class org.elasticsearch.index.mapper.IpFieldMapper$IpFieldType$IpScriptDocValues
 # for testing.
 # currently FeatureTest exposes overloaded constructor, field load store, and overloaded static methods
 class org.elasticsearch.painless.FeatureTest only_fqn {
+  int z
   ()
   (int,int)
   int getX()

diff --git a/modules/lang-painless/src/test/java/org/elasticsearch/painless/BasicAPITests.java b/modules/lang-painless/src/test/java/org/elasticsearch/painless/BasicAPITests.java
@@ -124,4 +124,9 @@ public void testPrimitivesHaveMethods() {
         assertEquals("5", exec("int x = 5; return x.toString();"));
         assertEquals(0, exec("int x = 5; return x.compareTo(5);"));
     }
+
+    public void testPublicMemberAccess() {
+        assertEquals(5, exec("org.elasticsearch.painless.FeatureTest ft = new org.elasticsearch.painless.FeatureTest();" +
+            "ft.z = 5; return ft.z;"));
+    }
 }