javadoc and fixed test