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/11/22 17:45:39 UTC

[groovy] branch master updated: Fix failing tests on JDK8

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

sunlan 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 59343fa  Fix failing tests on JDK8
59343fa is described below

commit 59343faa52cb375a1203aab7341e0ace0a3557f9
Author: Daniel Sun <su...@apache.org>
AuthorDate: Mon Nov 23 01:45:22 2020 +0800

    Fix failing tests on JDK8
---
 .../src/spec/test/org/apache/groovy/ginq/GinqTest.groovy            | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/subprojects/groovy-ginq/src/spec/test/org/apache/groovy/ginq/GinqTest.groovy b/subprojects/groovy-ginq/src/spec/test/org/apache/groovy/ginq/GinqTest.groovy
index 81f5821..2327f59 100644
--- a/subprojects/groovy-ginq/src/spec/test/org/apache/groovy/ginq/GinqTest.groovy
+++ b/subprojects/groovy-ginq/src/spec/test/org/apache/groovy/ginq/GinqTest.groovy
@@ -22,6 +22,8 @@ import groovy.transform.CompileStatic
 import org.junit.Test
 
 import static groovy.test.GroovyAssert.assertScript
+import static groovy.test.GroovyAssert.isAtLeastJdk
+import static org.junit.Assume.assumeTrue
 
 @CompileStatic
 class GinqTest {
@@ -2721,6 +2723,8 @@ class GinqTest {
 
     @Test
     void "testGinq - lazy - 6"() {
+        assumeTrue(isAtLeastJdk('9.0'))
+
         assertScript '''
             int cnt = 0
             def result = GQ {
@@ -2744,6 +2748,8 @@ class GinqTest {
 
     @Test
     void "testGinq - lazy - 7"() {
+        assumeTrue(isAtLeastJdk('9.0'))
+
         assertScript '''
             int cnt = 0
             def result = GQ {