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.


---

Reply via email to