hemasrishalini commented on PR #4637: URL: https://github.com/apache/polaris/pull/4637#issuecomment-4656964158
Thanks for the review and feedback! My intention was to improve coverage around the OAuth error-to-response mapping, including the default branch and validation of the returned error response. I understand the concern that the test closely mirrors the implementation and may provide limited value given the simplicity of the mapping logic. I'm happy to defer to the maintainers' judgment on whether this coverage is worthwhile to keep. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
