zrhoffman commented on code in PR #7099:
URL: https://github.com/apache/trafficcontrol/pull/7099#discussion_r1025629258
##########
traffic_ops/testing/api/utils/utils.go:
##########
@@ -119,7 +130,7 @@ type V3TestData struct {
// V3TestDataT represents the data needed for testing the v3 api endpoints.
type V3TestDataT[B any] struct {
- EndpointId func() int
+ EndpointID func() int
Review Comment:
I'm fine with you keeping the 100-file field rename in #7099. Separately,
I've removed those so I could see what the API tests diff looks like without
that part, and it looks like #7099 is missing test coverage for the `"PRIMED"`
DeliveryServiceActiveState. #7099 is also missing coverage for an invalid
DeliveryServiceActiveState being POSTed or PUT.
--
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]
