refactor: fix AssertHTTP retry and improved CFT practices

[grayside]

Fixes #1726

This is a broader change than originally intended, as expanded test coverage surfaced new bugs and that process repeated a couple times.

1. Retries are now configurable using the new WithHTTPRequestRetries
2. API know longer uses t.Fatal, instead it uses t.Error for better alignment with testify assert semantics.
3. Logging now uses GetLoggerFromT for more consistent output with other library functions
4. If response body is part of the assert, they will be checked even if the status code doesn't match.
5. Logic is simplified via wrapped errors
6. httpRetryCondition now returned two booleans: first indicates response success, second that retry is called for.
7. Test coverage is significantly expanded and uses a table testing approach to improve readability and facilitate consistently testing logic with and without retries

[apeabody]

/gcbrun