DMP-4688: Add Audio errors from DARTS API should be mapped to the correct response format in Gateway #674
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Links
Change description
Summary of Git Diff
This commit introduces significant modifications to various test files in the DARTS project, particularly focusing on the handling of exceptions in SOAP web service tests. The main change is the introduction of a new
ServiceException
class and the refactoring of existing tests to utilize this class for asserting error responses.Highlights
ServiceException
fromcom.emc.documentum.fs.rt
in multiple test files for standardized error handling.AddAudioResponse
for error assertions have been refactored to utilizeServiceException
.courtHouseNotFoundResponse.xml
dartsExceptionResponse.xml
dartsValidationExceptionAudioTooLargeResponse.xml
dartsValidationExceptionInvalidXmlDocumentResponse.xml
dartsValidationExceptionResponse.xml
problemResponse.xml
file was removed as it is no longer needed.AbstractClientProblemDecoder
was updated to improve error handling by passing the cause of the error to the exceptions.DartsEndpoint
andDartsGatewayClient
were updated to accommodate the new exception handling mechanism.These changes enhance the robustness of the error handling in the DARTS application, leading to better maintainability and clearer test assertions.
Does this PR introduce a breaking change? (check one with "x")