[12.x] Ensure throwIfStatus / throwUnlessStatus work for all status codes

[jackbayliss]

throwIfStatus and throwUnlessStatus delegate to $this->throw() under the hood, which only throws when failed() is true ie status (4xx/5xx). This means both methods silently do nothing for 2xx and 3xx responses.

This means we can now do:

// Ensure a resource was actually created...
Http::post('https://api.example.com/users', $data)
    ->throwUnlessStatus(201);

// Catch unexpected redirects...
Http::get('https://api.example.com/resource')
    ->throwIfStatus(302);

Before this fix, both of these silently passed without throwing.

Neither passed the callback into throw..

I've added tests to ensure no regression - this should match what $this->throw was doing - can target 13.x but feels like this is expected behaviour.

[github-actions]

Thanks for submitting a PR!

Note that draft PRs are not reviewed. If you would like a review, please mark your pull request as ready for review in the GitHub user interface.

Pull requests that are abandoned in draft may be closed due to inactivity.