From 3a7b411143db08cb4d8813d00988c07848738bd3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Hoguin?= Date: Mon, 22 Jan 2018 14:00:05 +0100 Subject: Don't crash in delete_resp_header if no resp headers are set --- test/handlers/resp_h.erl | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) (limited to 'test/handlers/resp_h.erl') diff --git a/test/handlers/resp_h.erl b/test/handlers/resp_h.erl index ba46213..487a724 100644 --- a/test/handlers/resp_h.erl +++ b/test/handlers/resp_h.erl @@ -94,10 +94,13 @@ do(<<"has_resp_body">>, Req0, Opts) -> {ok, cowboy_req:reply(200, #{}, Req), Opts} end; do(<<"delete_resp_header">>, Req0, Opts) -> - false = cowboy_req:has_resp_header(<<"content-type">>, Req0), - Req1 = cowboy_req:set_resp_header(<<"content-type">>, <<"text/plain">>, Req0), - true = cowboy_req:has_resp_header(<<"content-type">>, Req1), - Req = cowboy_req:delete_resp_header(<<"content-type">>, Req1), + %% We try to delete first even though it hasn't been set to + %% make sure this noop is possible. + Req1 = cowboy_req:delete_resp_header(<<"content-type">>, Req0), + false = cowboy_req:has_resp_header(<<"content-type">>, Req1), + Req2 = cowboy_req:set_resp_header(<<"content-type">>, <<"text/plain">>, Req1), + true = cowboy_req:has_resp_header(<<"content-type">>, Req2), + Req = cowboy_req:delete_resp_header(<<"content-type">>, Req2), false = cowboy_req:has_resp_header(<<"content-type">>, Req), {ok, cowboy_req:reply(200, #{}, "OK", Req), Opts}; do(<<"inform2">>, Req0, Opts) -> -- cgit v1.2.3