martin-g commented on a change in pull request #409:
URL: https://github.com/apache/curator/pull/409#discussion_r815353470
##########
File path:
curator-framework/src/test/java/org/apache/curator/framework/state/TestConnectionStateManager.java
##########
@@ -91,7 +92,10 @@ public void stateChanged(CuratorFramework client,
ConnectionState newState)
}
@Test
- public void testConnectionStateRecoversFromUnexpectedExpiredConnection()
throws Exception {
+ void testConnectionStateRecoversFromUnexpectedExpiredConnection() throws
Exception {
Review comment:
> 1. If you remove `@Test` above, this test doesn't actually run in any
condition.
I am not sure why do you explain this to me. I didn't touch `@Test` and the
test is still executed in `curator-framework` Maven module, where Zookeeper is
3.6.3.
The problem is when this test is executed in `curator-test-zk35` module
(Zookeeper 3.5.7) where `#queueEvent()` is not available.
https://issues.apache.org/jira/browse/ZOOKEEPER-3269 is "Fixed" only for
`3.6.0`+
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
