Looking at the playground again, it seems endpoint writers have two options. 1) implement either full CRUDer or 2) create a factories for each operation individually. Supposing every operation is needed except for the delete (as an example), would you expect the implementor to have an extraneous delete function, or to create three separate factories?
Before I realized this, I was a little more on the fence. I suppose you could argue that most operations are CRUDer only anyway, which is why support for single or mixed endpoints hasn't been implemented yet. Overall I think I'd prefer what I perceive to be both better readability and writability. I realize tests aren't perfect in the general case, but an internal server error that gets triggered depending on the presence of a method seems similar to how a 404 would behave as if the endpoint wasn't written at all. One would only need to curl the endpoint once and we get an error no matter what. Would it be better to turn it into a 404, that also logged an error internally? [ Full content available at: https://github.com/apache/trafficcontrol/pull/2886 ] This message was relayed via gitbox.apache.org for [email protected]
