Skip to content

Commit 533ec79

Browse files
committed
Added assertion that response is present in MaxHttpRetriesExceededError
1 parent 285adc7 commit 533ec79

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

test/clients/http_client_test.rb

+2-1
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,8 @@ def test_retries_exceeded
162162
.with(body: @request.body.to_json, query: @request.query, headers: @expected_headers)
163163
.to_return(body: { errors: "Something very not good" }.to_json, headers: @response_headers, status: 500)
164164

165-
assert_raises(ShopifyAPI::Errors::MaxHttpRetriesExceededError) { @client.request(@request) }
165+
error = assert_raises(ShopifyAPI::Errors::MaxHttpRetriesExceededError) { @client.request(@request) }
166+
assert_instance_of(ShopifyAPI::Clients::HttpResponse, error.response)
166167
end
167168

168169
def test_throttle_error_no_retry_after_header

0 commit comments

Comments
 (0)