C client test suite fails consistently
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | zookeeper (Ubuntu) |
Low
|
Unassigned | ||
Bug Description
When the c client test suite, zktest-mt, is run, it fails with this:
tests/TestZooke
This was also reported in 3.3.1 here:
http://<email address hidden>
The C client tests are making some assumptions that are not valid. getaddrinfo may have, at one time, returned ENOENT instead of EINVAL for the host given in the test. The assertion should simply be that EINVAL | ENOENT are given, so that builds on platforms which return ENOENT for this are not broken.
Related branches
- James Page: Approve on 2012-12-14
-
Diff: 400 lines (+340/-1)7 files modified.pc/applied-patches (+1/-0)
.pc/fix-broken-c-client-unittest.patch/src/c/tests/TestZookeeperInit.cc (+301/-0)
debian/changelog (+9/-0)
debian/patches/fix-broken-c-client-unittest.patch (+20/-0)
debian/patches/series (+1/-0)
debian/rules (+7/-0)
src/c/tests/TestZookeeperInit.cc (+1/-1)
| Clint Byrum (clint-fewbar) wrote : | #1 |
| Changed in zookeeper (Ubuntu): | |
| status: | New → Triaged |
| importance: | Undecided → Low |
| Launchpad Janitor (janitor) wrote : | #2 |
This bug was fixed in the package zookeeper - 3.3.4+dfsg1-
---------------
zookeeper (3.3.4+
[ Clint Byrum ]
* d/patches/
will pass with EINVAL which is the correct error. (LP: #912935)
[ James Page ]
* d/rules: Enable execution of C client and core Java tests (LP: #913881).
-- Clint Byrum <email address hidden> Fri, 06 Jan 2012 13:58:23 -0800
| Changed in zookeeper (Ubuntu): | |
| status: | Triaged → Fix Released |
Note that this is also reported upstream here:
https:/