Review ignored HttpClientTest

[iNikem]

A lot of subclasses of HttpClientTest ignore one or more tests. This leaves us blind to the actual problems with our instrumentations, where we don't support quite common cases, e.g. redirects. We should review and fix them all.

[jaydeluca]

To add some context to this issue and to make sure I understand it - this is related to test classes that extend AbstractHttpClientTest and configure things like optionsBuilder.disableTestRedirects() which then causes a subset of tests to be ignored.

The ask is to update all the instrumentations that opt out of various tests so that they all properly instrument things like redirects, so that we can then remove these configurations.

Is that correct?

[trask]

this is my understanding as well 👍