[GitHub] [trafficcontrol] moltzaum commented on issue #3394: Fix interface ptr problem

[GitBox <gi...@apache.org>]

moltzaum commented on issue #3394: Fix interface ptr problem
URL: https://github.com/apache/trafficcontrol/pull/3394#issuecomment-479935925
 
 
   @mitchell852 I ran the whole integration test suite. I did not get an error for statuses, however I got an authorization error.
   ```
   --- FAIL: TestFederations (4.81s)
   	userdeliveryservices_test.go:69: failed to set delivery service users: 401 Unauthorized[401] - Error requesting Traffic Ops https://localhost:6443/api/1.4/deliveryservice_user {"alerts":[{"level":"error","text":"Unauthorized, please log in."}]}
   	userdeliveryservices_test.go:78: get user delivery services expected 4 actual 0
   	userdeliveryservices_test.go:90: get user delivery services expected 33 actual missing
   	userdeliveryservices_test.go:90: get user delivery services expected 30 actual missing
   	userdeliveryservices_test.go:90: get user delivery services expected 31 actual missing
   	userdeliveryservices_test.go:90: get user delivery services expected 32 actual missing
   	userdeliveryservices_test.go:182: get user delivery services expected >0 actual 0
   FAIL
   exit status 1
   ```
   
   I got one other fail for user delivery services
   `--- FAIL: TestUserDeliveryServices (4.44s)`
   
   Neither of these endpoints use the CRUDer framework (hence are not affected by the fix), and I reproduced these errors on master.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services