mbean.getMBeanServerConnection().equals(mbean.getMBeanServerConnection())
located in equals test should be removed.
It is useless and can lead to null pointer exception when connection are closed.