Update docs and more tests

[sonjaer]

Hey, I just made a Pull Request!

Update docs and more tests after #1041

Description

Motivation and Context

Have you tested this? If so, how?

Checklist for PR author(s)

• Changes are covered by unit test
• All tests pass
• Code coverage check passes
• Error handling is tested
• Errors are handled at the appropriate layer
• Errors that cannot be handled where they occur are propagated
• (optional) Changes are covered by system test
• Relevant documentation updated
• This PR has NO breaking change to public API
• This PR has breaking change to public API and it is documented

Checklist for PR reviewer(s)

• This PR has been incorporated in release note for the coming version
• Risky changes introduced by this PR have been all considered

[codecov]

Codecov Report

Merging #1049 (46dffe8) into master (04bf5b3) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1049      +/-   ##
============================================
+ Coverage     92.28%   92.31%   +0.03%     
- Complexity     2138     2143       +5     
============================================
  Files           200      200              
  Lines          8282     8293      +11     
  Branches        502      502              
============================================
+ Hits           7643     7656      +13     
+ Misses          523      522       -1     
+ Partials        116      115       -1