You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by kl...@apache.org on 2017/04/07 19:34:10 UTC

[11/19] geode git commit: GEM-1351: Move a test to the open side and fix the other tests - spotless

GEM-1351: Move a test to the open side and fix the other tests - spotless

* move ImportClusterConfigDistributedTest to open side
* remove @Ignore from other two tests.
* rework GfshShellConnectionRule to not create gfsh shell at class initialization time.


Project: http://git-wip-us.apache.org/repos/asf/geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/geode/commit/edd018b9
Tree: http://git-wip-us.apache.org/repos/asf/geode/tree/edd018b9
Diff: http://git-wip-us.apache.org/repos/asf/geode/diff/edd018b9

Branch: refs/heads/feature/GEODE-2632
Commit: edd018b971d93565da502a8523df22a8f0b669ca
Parents: 02ccc5c
Author: Jinmei Liao <ji...@pivotal.io>
Authored: Wed Apr 5 07:51:31 2017 -0700
Committer: Jinmei Liao <ji...@pivotal.io>
Committed: Wed Apr 5 07:51:31 2017 -0700

----------------------------------------------------------------------
 .../org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/geode/blob/edd018b9/geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java b/geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java
index 1c72bba..381b751 100644
--- a/geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java
+++ b/geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java
@@ -121,7 +121,7 @@ public class GfshShellConnectionRule extends DescribedExternalResource {
 
   public void connect(int port, PortType type, String... options) throws Exception {
     CliUtil.isGfshVM = true;
-    if(gfsh == null){
+    if (gfsh == null) {
       this.gfsh = new HeadlessGfsh(getClass().getName(), 30,
           temporaryFolder.newFolder("gfsh_files").getAbsolutePath());
     }