zrhoffman commented on pull request #4857: URL: https://github.com/apache/trafficcontrol/pull/4857#issuecomment-672002896
Running the integration test, I get ```java Running com.comcast.cdn.traffic_control.traffic_router.core.util.AbstractResourceWatcherTest Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 6.02 sec <<< FAILURE! - in com.comcast.cdn.traffic_control.traffic_router.core.util.AbstractResourceWatcherTest testWatchers(com.comcast.cdn.traffic_control.traffic_router.core.util.AbstractResourceWatcherTest) Time elapsed: 6.019 sec <<< FAILURE! java.lang.AssertionError: expected null, but was:<"https://${toHostname}/api/2.0/federations/all";> at com.comcast.cdn.traffic_control.traffic_router.core.util.AbstractResourceWatcherTest.testWatchers(AbstractResourceWatcherTest.java:99) Running com.comcast.cdn.traffic_control.traffic_router.core.dns.ZoneManagerTest Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.001 sec - in com.comcast.cdn.traffic_control.traffic_router.core.dns.ZoneManagerTest Results : Failed tests: AbstractResourceWatcherTest.testWatchers:99 expected null, but was:<"https://${toHostname}/api/2.0/federations/all";> ``` `https://${toHostname}/api/2.0/federations/all` is `DEFAULT_FEDERATION_DATA_URL`, right? Should the test get something else? ---------------------------------------------------------------- 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]
