Refactor BadResponseException (PP-1641)

[jonathangreen]

Description

Refactor BadResponseException, so it has a reference to the response that can be used to deal with the exception.

This required changing a couple references to RemoteIntegrationException, where the response wasn't readily available. These exceptions have the same base class, so this should be handled okay.

I updated the OPDS2 + ODL importer to use the new exception, rather then doing its own status code parsing. I also updated mock OPDS2 + ODL importer to use MockHTTPClient instead of having its own mock client.

Motivation and Context

• This work was done as a pre-requisite for the work in Return problem detail when LCP fulfillment fails (PP-1641) #2033. I figured breaking this out to a separate PR would make it easier to review.

How Has This Been Tested?

• Running unit tests

Checklist

• I have updated the documentation accordingly.
• All new and existing tests passed.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.72%. Comparing base (c7489f7) to head (cba9f2d).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2034      +/-   ##
==========================================
+ Coverage   90.70%   90.72%   +0.02%     
==========================================
  Files         342      342              
  Lines       40579    40572       -7     
  Branches     8799     8796       -3     
==========================================
+ Hits        36807    36809       +2     
+ Misses       2508     2503       -5     
+ Partials     1264     1260       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[tdilauro]

Looks good. 🛠️

[tdilauro]

Minor: ^ Does the above comment make sense anymore?

[jonathangreen]

Good call, will 🪓