ocket8888 commented on PR #7268:
URL: https://github.com/apache/trafficcontrol/pull/7268#issuecomment-1373860110

   Originally I had added a TO Go client integration test case to make sure the 
response contained a `deliveryService` property, but our client drops the 
response so checking that is impossible without either a breaking change to the 
client, or adding functionality to a client of deprecated API. Neither of which 
I'm willing to do.
   
   I really wish I had realized that earlier, because the only reason I 
modified code in `lib/` was to enable that test, which is now unnecessary. But 
I'm leaving it in as it's a change that was requested in one of my PRs last 
year.


-- 
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]

Reply via email to