Ok, that is kind of weird. I'll take a look. On Fri, Feb 5, 2016 at 4:58 AM, Jordan Zimmerman <[email protected] > wrote:
> No, sorry. The last few lines of the test currently are: > > > assertEvent(TreeCacheEvent.Type.NODE_REMOVED, "/test/me", "data".getBytes()); > assertEvent(TreeCacheEvent.Type.INITIALIZED); > > This fails. But, if I switch them it works: > > assertEvent(TreeCacheEvent.Type.INITIALIZED); > > assertEvent(TreeCacheEvent.Type.NODE_REMOVED, "/test/me", "data".getBytes()); > > On Feb 5, 2016, at 2:57 AM, Scott Blum <[email protected]> wrote: > > So you end up with 2 initialized events? > > You mean this? > > assertEvent(TreeCacheEvent.Type.CONNECTION_RECONNECTED); > + assertEvent(TreeCacheEvent.Type.INITIALIZED); > assertEvent(TreeCacheEvent.Type.NODE_REMOVED, "/test/me", > "data".getBytes()); > assertEvent(TreeCacheEvent.Type.INITIALIZED); > > Seems weird if there are two, but I can help look. > > On Thu, Feb 4, 2016 at 10:48 PM, Jordan Zimmerman < > [email protected]> wrote: > >> Hey Scott, >> >> In this branch, TestTreeCache.testKilledSession() is failing at: >> >> assertEvent(TreeCacheEvent.Type.NODE_REMOVED, "/test/me", >> "data".getBytes()); >> >> However, if I change the two asserts to: >> >> assertEvent(TreeCacheEvent.Type.INITIALIZED); >> assertEvent(TreeCacheEvent.Type.NODE_REMOVED, "/test/me", >> "data".getBytes()); >> >> it works. Does that make any sense? >> >> -Jordan >> >> > On Feb 4, 2016, at 9:23 PM, Jordan Zimmerman < >> [email protected]> wrote: >> > >> > Devs, >> > >> > In trying to fix the bad log message "Failed to find watcher” (which >> turns out to be a ZK client issue), I realize that the NamespaceWatcher and >> WatcherWrapper stuff could be improved. I’m still working on getting all >> tests to pass but I’d appreciate more sets of eyes on this change. Please >> review carefully if you can. >> > >> > https://github.com/apache/curator/pull/131 >> > >> > -Jordan >> >> > >
