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

jinmeiliao pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new e7987a3  GEODE-5846: can not start 1.3 gfsh with jdk9. (#2681)
e7987a3 is described below

commit e7987a335e0ae6f208df0bcbaf1df909a13e85d5
Author: jinmeiliao <[email protected]>
AuthorDate: Tue Oct 23 09:43:32 2018 -0700

    GEODE-5846: can not start 1.3 gfsh with jdk9. (#2681)
---
 .../internal/cli/commands/ConnectCommandAcceptanceTest.java           | 4 ++++
 1 file changed, 4 insertions(+)

diff --git 
a/geode-assembly/src/acceptanceTest/java/org/apache/geode/management/internal/cli/commands/ConnectCommandAcceptanceTest.java
 
b/geode-assembly/src/acceptanceTest/java/org/apache/geode/management/internal/cli/commands/ConnectCommandAcceptanceTest.java
index 29420e2..7cd23b9 100644
--- 
a/geode-assembly/src/acceptanceTest/java/org/apache/geode/management/internal/cli/commands/ConnectCommandAcceptanceTest.java
+++ 
b/geode-assembly/src/acceptanceTest/java/org/apache/geode/management/internal/cli/commands/ConnectCommandAcceptanceTest.java
@@ -18,7 +18,9 @@
 package org.apache.geode.management.internal.cli.commands;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.junit.Assume.assumeTrue;
 
+import org.apache.commons.lang.SystemUtils;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.experimental.categories.Category;
@@ -39,6 +41,8 @@ public class ConnectCommandAcceptanceTest {
 
   @Test
   public void useCurrentGfshToConnectToOlderLocator() throws Exception {
+    // this test can only be run with pre-9 jdk since it needs to run older 
version of gfsh
+    assumeTrue(!SystemUtils.isJavaVersionAtLeast(900));
     GfshScript.of("start locator").execute(gfsh130);
     GfshExecution connect = 
GfshScript.of("connect").expectFailure().execute(gfshDefault);
 

Reply via email to