moltzaum commented on issue #3450: TO: Cachegroup Bugfixes
URL: https://github.com/apache/trafficcontrol/pull/3450#issuecomment-488752787
 
 
   > can u write api tests on POST/PUT to check the response for the proper 
values for parentCachegroupName, secondaryParentCachegroupName, typeName and 
check for the existence of lastUpdated. also, check that localization_methods 
is an empty array when none are defined rather than null.
   
   Ideally, we would be able to test most endpoints the same. I think writing a 
test for just the cachegroup has limited benefit. If #2494 is resolved via the 
framework rather than requiring changes on a per-handler basis, we could 
probably test the framework instead of individual endpoints as well.
   
   The second part, the testing for default values I don't think is handled by 
any framework functions. I can write a test for those. Since I'll be in the 
test code anyway, I'll fetch a few of the names so that at least something is 
covered.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to