You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2020/02/12 17:43:56 UTC
[groovy] branch GROOVY_3_0_X updated: GROOVY-9387: fix test case
for indy
This is an automated email from the ASF dual-hosted git repository.
sunlan pushed a commit to branch GROOVY_3_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git
The following commit(s) were added to refs/heads/GROOVY_3_0_X by this push:
new 6547a1b GROOVY-9387: fix test case for indy
6547a1b is described below
commit 6547a1bb577f49502767573ab836cc9f19bda936
Author: Paul King <pa...@asert.com.au>
AuthorDate: Sun Feb 9 10:50:29 2020 +1000
GROOVY-9387: fix test case for indy
(cherry picked from commit 24650a5ea2823110566e1be9aaf7abef46dca1d1)
---
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'