[GitHub] [trafficcontrol] mitchell852 commented on issue #4191: Add Go implementation of GET, POST federations/{id}/federation_resolvers

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

mitchell852 commented on issue #4191: Add Go implementation of GET, POST federations/{id}/federation_resolvers
URL: https://github.com/apache/trafficcontrol/pull/4191#issuecomment-568525585
 
 
   @lactose - i'm seeing a couple of failing API tests:
   
   ```
   === RUN   TestCDNFederations/successfully_get_federation_federation_resolvers_for_a_federation_with_some
       --- FAIL: TestCDNFederations/successfully_get_federation_federation_resolvers_for_a_federation_with_some (0.02s)
           cdnfederations_test.go:222: expected federation of ID 19 to have associated federation resolvers, but had 0
   ```
   
   ```
   === RUN   TestFederationFederationResolvers/Fail_to_assign_federation_resolver_to_a_federation_when_federation_does_not_exist
       --- FAIL: TestFederationFederationResolvers/Fail_to_assign_federation_resolver_to_a_federation_when_federation_does_not_exist (0.01s)
           cdnfederations_test.go:186: error: expected error result 404 Not Found[404] - Error requesting Traffic Ops https://localhost:8443/api/1.4/federations/-1/federation_resolvers {"alerts":[{"text":"'-1': no such Federation","level":"error"}]}
               , want: Resource not found
   ```

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