Update snippets base service and callers
Why
We want to use reason instead of http_status in accordance with our development guidelines and Rubocop rules.
Also, we want to start returning symbol reasons and have any http layer callers interpret that as needed into http status codes. This MR introduces a new class to handle that and uses it in the api layer.
What
- replace
http_statuswithreason - since this is a base class, identify inheriting classes that do not override the affected method(
Snippets::CreateService, andSnippets::UpdateService) - find callers of inheritors
- update them to use
reasoninstead ofhttp_statuswhen interacting with the service responses. - add map class to interpret reasons to http status codes
Related to #521438 (closed)
Edited by Hunter Stewart