Skip to content

Commit

Permalink
fix tests for httpbingo upgrade (#152) (#154)
Browse files Browse the repository at this point in the history
This test started failing because httpbingo now serves a 300 response
with a location header, which gets followed so the response object ends
up being a 200 OK instead of the expected 300. This changes the test to
use 200, 400 and 404, which should never redirect.

(cherry picked from commit dbb0625)

Co-authored-by: Stefan Karpinski <stefan@karpinski.org>
  • Loading branch information
DilumAluthge and StefanKarpinski authored Oct 26, 2021
1 parent 5d00bdd commit 4ecac8d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/runtests.jl
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,7 @@ include("setup.jl")

@testset "request API" begin
@testset "basic request usage" begin
for status in (200, 300, 400)
for status in [200, 400, 404]
url = "$server/status/$status"
resp, body = request_body(url)
@test resp.url == url
Expand Down

0 comments on commit 4ecac8d

Please sign in to comment.