You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zookeeper.apache.org by ha...@apache.org on 2016/12/23 05:58:27 UTC

zookeeper git commit: ZOOKEEPER-2656: Fix ServerConfigTest#testValidArguments test case fai…

Repository: zookeeper
Updated Branches:
  refs/heads/master f588e1982 -> 085a7ab4e


ZOOKEEPER-2656: Fix ServerConfigTest#testValidArguments test case fai\u2026

\u2026lures

Author: Rakesh Radhakrishnan <ra...@apache.org>

Reviewers: Michael Han <ha...@apache.org>

Closes #134 from rakeshadr/ZK-2656


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

Branch: refs/heads/master
Commit: 085a7ab4efe04e566e952af1123e98eda609bd1b
Parents: f588e19
Author: Rakesh Radhakrishnan <ra...@apache.org>
Authored: Thu Dec 22 21:58:23 2016 -0800
Committer: Michael Han <ha...@apache.org>
Committed: Thu Dec 22 21:58:23 2016 -0800

----------------------------------------------------------------------
 src/java/test/org/apache/zookeeper/ServerConfigTest.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zookeeper/blob/085a7ab4/src/java/test/org/apache/zookeeper/ServerConfigTest.java
----------------------------------------------------------------------
diff --git a/src/java/test/org/apache/zookeeper/ServerConfigTest.java b/src/java/test/org/apache/zookeeper/ServerConfigTest.java
index d4855fd..e2d0da8 100644
--- a/src/java/test/org/apache/zookeeper/ServerConfigTest.java
+++ b/src/java/test/org/apache/zookeeper/ServerConfigTest.java
@@ -24,6 +24,8 @@ import org.junit.Test;
 
 import static org.junit.Assert.assertEquals;
 
+import java.io.File;
+
 public class ServerConfigTest {
 
     private ServerConfig serverConfig;
@@ -45,7 +47,7 @@ public class ServerConfigTest {
         serverConfig.parse(args);
 
         assertEquals(2181, serverConfig.getClientPortAddress().getPort());
-        assertEquals("/data/dir", serverConfig.getDataDir());
+        assertEquals(new File("/data/dir"), serverConfig.getDataDir());
         assertEquals(60000, serverConfig.getTickTime());
         assertEquals(10000, serverConfig.getMaxClientCnxns());
     }