[groovy] branch master updated: GROOVY-9387: fix test case for indy

[pa...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

paulk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/master by this push:
     new 24650a5   GROOVY-9387: fix test case for indy
24650a5 is described below

commit 24650a5ea2823110566e1be9aaf7abef46dca1d1
Author: Paul King <pa...@asert.com.au>
AuthorDate: Sun Feb 9 10:50:29 2020 +1000

     GROOVY-9387: fix test case for indy
---
 src/test/groovy/bugs/Groovy9387.groovy | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/test/groovy/bugs/Groovy9387.groovy b/src/test/groovy/bugs/Groovy9387.groovy
index ef80987..93eada5 100644
--- a/src/test/groovy/bugs/Groovy9387.groovy
+++ b/src/test/groovy/bugs/Groovy9387.groovy
@@ -20,9 +20,11 @@ package groovy.bugs
 
 import groovy.test.NotYetImplemented
 import groovy.transform.CompileStatic
+import org.codehaus.groovy.control.CompilerConfiguration
 import org.junit.Test
 
 import static groovy.test.GroovyAssert.assertScript
+import static org.junit.Assume.assumeFalse
 
 @CompileStatic
 final class Groovy9387 {
@@ -60,6 +62,8 @@ final class Groovy9387 {
 
     @Test @NotYetImplemented
     void testThisSetProperty() {
+        // currently only broken for classic bytecode (TODO: remove assumeFalse line and this comment once issue is fixed - no harm in having an indy testcase at that point)
+        assumeFalse CompilerConfiguration.DEFAULT.optimizationOptions?.indy
         assertScript SUPPORT_ADAPTER + '''
             class C extends BuilderSupportAdapter {
                 String value = 'abc'