[JENKINS-34599] Reproduced issues in test.

jenkinsci · Jul 11, 2017 · f5d2128 · f5d2128
1 parent 29641f0
commit f5d2128
Showing 1 changed file with 21 additions and 0 deletions.
diff --git a/...test/java/org/jenkinsci/plugins/scriptsecurity/sandbox/groovy/SandboxInterceptorTest.java b/...test/java/org/jenkinsci/plugins/scriptsecurity/sandbox/groovy/SandboxInterceptorTest.java
@@ -160,6 +160,27 @@ public boolean permitsMethod(Method method, Object receiver, Object[] args) {
         assertTrue(Clazz.flag);
     }
 
+    @Ignore("TODO groovy.lang.ReadOnlyPropertyException: Cannot set readonly property: x for class: X")
+    @Issue("JENKINS-34599")
+    @Test public void finalFields() throws Exception {
+        // Control cases: non-final fields.
+        assertEvaluate(new ProxyWhitelist(), 99, "class X {int x = 99}; new X().x");
+        assertEvaluate(new ProxyWhitelist(), 99, "class X {int x; X(int x) {this.x = x}}; new X(99).x");
+        assertEvaluate(new ProxyWhitelist(), 99, "class X {int x; {this.x = 99}}; new X().x");
+        assertEvaluate(new ProxyWhitelist(), 99, "class X {static int x = 99}; X.x");
+        assertEvaluate(new ProxyWhitelist(), 99, "class X {static int x; static {x = 99}}; X.x");
+        // Control case: field set in initialization expression.
+        assertEvaluate(new ProxyWhitelist(), 99, "class X {final int x = 99}; new X().x");
+        // Test case: field set in constructor.
+        assertEvaluate(new ProxyWhitelist(), 99, "class X {final int x; X(int x) {this.x = x}}; new X(99).x");
+        // Test case: field set in instance initializer.
+        assertEvaluate(new ProxyWhitelist(), 99, "class X {final int x; {this.x = 99}}; new X().x");
+        // Control case: field set in static initialization expression.
+        assertEvaluate(new ProxyWhitelist(), 99, "class X {static final int x = 99}; X.x");
+        // Test case: field set in static instance initializer.
+        assertEvaluate(new ProxyWhitelist(), 99, "class X {static final int x; static {x = 99}}; X.x");
+    }
+
     @Test public void propertiesAndGettersAndSetters() throws Exception {
         String clazz = Clazz.class.getName();
         assertEvaluate(new StaticWhitelist("new " + clazz, "field " + clazz + " prop"), "default", "new " + clazz + "().prop");