fixes #933 update javadoc and a test case

[stevehu]

No description provided.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.04%. Comparing base (7f1ec11) to head (eebaeaf).
Report is 119 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff            @@
##             master     #934   +/-   ##
=========================================
  Coverage     79.04%   79.04%           
  Complexity     1504     1504           
=========================================
  Files           143      143           
  Lines          4915     4915           
  Branches        932      932           
=========================================
  Hits           3885     3885           
  Misses          670      670           
  Partials        360      360           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[stevehu]

@justin-tay, One of the test cases failed on my faster desktop computer during the maven install and I am guessing it is a timing issue with other test cases. I have added ten milliseconds of sleep as a temporary solution. Do you happen to know what would cause IlegalStateException "recursive update"? If not, I think we can leave the test case as it is. Thanks a lot for the PR.

[justin-tay]

Looks good to me. Thanks!

I shall see if I can reproduce this later.