Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/546 Thank you for the quick turnaround @eolivelli . Mockito unit test is cool, exactly what I meant to. I think it's okay to validate the existence of JMX property in the existing test (similar checks were already there), but would you please remove checks for the *value* of the property? I really don't want the hammer test failing in case of an incorrect JMX value.
---