Improve http connection test

[JamesHarris-orkes]

This appears to be a test that attempts a timeout. The test asserts that the entire timeout occurred, but that isn't the only failure condition. Some valid networks can respond that the network address is unreachable in less time than the timeout. Instead of using an rfc1918 address, expecting a full time out it would be better to select an address from 127/8 (eg 127.1.2.3) that should do as the code intends, wait the full timeout for a connection failure.

conductor/http-task/src/test/java/com/netflix/conductor/tasks/http/HttpTaskTest.java

Line 298 in b71d28b

input.setUri("http://10.255.14.15");

[codingkiddo]

Hi @JamesHarris-orkes, I’d like to take this.

I’ll update the HTTP timeout test to avoid relying on an RFC1918 address and use a loopback 127/8 address as suggested, then run the relevant http-task tests locally.