Add exception handling #20

daverbk · 2024-04-14T13:05:49Z

Pull Request

Added a GlobalExceptionHandler.java to handle exceptions and return a similar-styled error responses in form of ExceptionResponse.java

Please review and check all applicable items:

I have tested my changes thoroughly
My code follows the project's coding standards and guidelines
I have updated relevant tests
I have reviewed my changes to ensure there are no unnecessary changes
My pull request has a descriptive title and includes a detailed description of the changes

In the future we can create more custom exceptions and handle them via GlobalExceptionHandler

ps. Exception handling related tests will be added as a separate commit to this pr (done ✅)

Google API-related errors

Our custom exception

daverbk added 2 commits April 14, 2024 15:00

Add PlayListNotFoundException.java

ace6781

Add GlobalExceptionHandler.java

952eb84

daverbk added the enhancement New feature or request label Apr 14, 2024

daverbk added this to the Finish main backend endpoints development 🚀 milestone Apr 14, 2024

daverbk requested a review from leingenm April 14, 2024 13:05

daverbk self-assigned this Apr 14, 2024

daverbk linked an issue Apr 14, 2024 that may be closed by this pull request

Add proper error handling #17

Closed

Add exception handling tests

8c438fb

leingenm approved these changes Apr 15, 2024

View reviewed changes

leingenm merged commit db0480a into main Apr 15, 2024
3 checks passed

leingenm deleted the issue-17 branch April 15, 2024 11:41