[ https://issues.apache.org/jira/browse/ZOOKEEPER-2656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15796653#comment-15796653 ]
ASF GitHub Bot commented on ZOOKEEPER-2656: ------------------------------------------- Github user eribeiro commented on the issue: https://github.com/apache/zookeeper/pull/140 Congratulations, @hanm. IMHO, it's a very elegant and simple solution to 3.4, 3.5, master compatibility. 😃 Let's wait for @rakeshadr feedback, but I am +1 for my corner, haha. Cheers! > Fix ServerConfigTest#testValidArguments test case failures > ---------------------------------------------------------- > > Key: ZOOKEEPER-2656 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2656 > Project: ZooKeeper > Issue Type: Test > Reporter: Rakesh R > Assignee: Michael Han > Fix For: 3.4.10, 3.5.3, 3.6.0 > > > This jira to fix ServerConfigTest#testValidArguments test case failure. > Reference: > https://builds.apache.org/job/ZooKeeper-trunk/3207/testReport/org.apache.zookeeper/ServerConfigTest/testValidArguments/ > {code} > Error Message > expected: java.lang.String</data/dir> but was: java.io.File</data/dir> > Stacktrace > junit.framework.AssertionFailedError: expected: java.lang.String</data/dir> > but was: java.io.File</data/dir> > at > org.apache.zookeeper.ServerConfigTest.testValidArguments(ServerConfigTest.java:48) > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332)