I fixed two definite issues and added test coverage to guard against regressions in the future:
RequestRateTooLargeException.getStatusCode now correctly returns TOO_MANY_REQUESTS, not NOTFOUND.
ServiceUnavailableException.getStatusCode now correctly returns SERVICE_UNAVAILABLE, not NOTFOUND.
DocumentClientExceptionTest.statusCodeIsCorrect verifies that the correct status code is returned for all DocumentClientException subtypes.
The DocumentClientExceptionTest is now split between two maven packages: commons and direct-impl. The commons variant includes DocumentClientException tests plus status code checks for all subtypes defined in the commons package. The direct-impl variant checks status codes for all subtypes defined in the direct-impl package.
I fixed two definite issues and added test coverage to guard against regressions in the future:
RequestRateTooLargeException.getStatusCodenow correctly returnsTOO_MANY_REQUESTS, notNOTFOUND.ServiceUnavailableException.getStatusCodenow correctly returnsSERVICE_UNAVAILABLE, notNOTFOUND.DocumentClientExceptionTest.statusCodeIsCorrectverifies that the correct status code is returned for allDocumentClientExceptionsubtypes.The
DocumentClientExceptionTestis now split between two maven packages: commons and direct-impl. The commons variant includesDocumentClientExceptiontests plus status code checks for all subtypes defined in the commons package. The direct-impl variant checks status codes for all subtypes defined in the direct-impl package.