[FIXED JENKINS-34599] Integrating fix.

jenkinsci · Jul 12, 2017 · c9b8c50 · c9b8c50
1 parent 68737a3
commit c9b8c50
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/pom.xml b/pom.xml
@@ -47,7 +47,7 @@
     <dependency>
       <groupId>org.kohsuke</groupId>
       <artifactId>groovy-sandbox</artifactId>
-      <version>1.12-20170711.212038-1</version> <!-- TODO https://github.com/kohsuke/groovy-sandbox/pull/33 -->
+      <version>1.12-20170712.135157-2</version> <!-- TODO https://github.com/kohsuke/groovy-sandbox/pull/33 -->
       <exclusions>
         <exclusion>
           <groupId>org.codehaus.groovy</groupId>

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,7 +160,6 @@ 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.