From 4d08610ea19a3b508804a2e6fe684ae5df11b261 Mon Sep 17 00:00:00 2001 From: geeksilva97 Date: Wed, 10 Jan 2024 16:36:39 -0300 Subject: [PATCH] add remaining tests --- test/handlers/resp_h.erl | 20 ++++++++++++++++++++ test/req_SUITE.erl | 4 ++++ 2 files changed, 24 insertions(+) diff --git a/test/handlers/resp_h.erl b/test/handlers/resp_h.erl index 9faf09be..fc22ade4 100644 --- a/test/handlers/resp_h.erl +++ b/test/handlers/resp_h.erl @@ -203,6 +203,9 @@ do(<<"reply4">>, Req0, Opts) -> <<"error">> -> ct_helper:ignore(erlang, iolist_size, 1), cowboy_req:reply(200, #{}, ok, Req0); + <<"set_cookie">> -> + ct_helper:ignore(cowboy_req, reply, 4), + cowboy_req:reply(200, #{<<"set-cookie">> => <<"name=paco loco">>}, ok, Req0); <<"204body">> -> ct_helper:ignore(cowboy_req, do_reply_ensure_no_body, 4), cowboy_req:reply(204, #{}, <<"OK">>, Req0); @@ -224,6 +227,9 @@ do(<<"stream_reply2">>, Req0, Opts) -> Req = cowboy_req:stream_reply(ok, Req0), stream_body(Req), {ok, Req, Opts}; + <<"set_cookie">> -> + ct_helper:ignore(cowboy_req, reply, 4), + cowboy_req:reply(200, #{<<"set-cookie">> => <<"name=paco loco">>}, ok, Req0); <<"204">> -> Req = cowboy_req:stream_reply(204, Req0), {ok, Req, Opts}; @@ -254,6 +260,9 @@ do(<<"stream_reply3">>, Req0, Opts) -> <<"error">> -> ct_helper:ignore(cowboy_req, stream_reply, 3), cowboy_req:stream_reply(200, ok, Req0); + <<"set_cookie">> -> + ct_helper:ignore(cowboy_req, stream_reply, 3), + cowboy_req:stream_reply(200, #{<<"set-cookie">> => <<"name=cormano">>}, Req0); Status -> cowboy_req:stream_reply(binary_to_integer(Status), #{<<"content-type">> => <<"text/plain">>}, Req0) @@ -409,6 +418,17 @@ do(<<"stream_trailers">>, Req0, Opts) -> <<"grpc-status">> => <<"0">> }, Req), {ok, Req, Opts}; + <<"set_cookie">> -> + ct_helper:ignore(cowboy_req, stream_trailers, 2), + Req = cowboy_req:stream_reply(200, #{ + <<"trailer">> => <<"grpc-status">> + }, Req0), + %% The size should be larger than StreamSize and ConnSize + cowboy_req:stream_body(<<0:80000000>>, nofin, Req), + cowboy_req:stream_trailers(#{ + <<"set-cookie">> => <<"name=cormano">> + }, Req), + {ok, Req, Opts}; _ -> Req = cowboy_req:stream_reply(200, #{ <<"trailer">> => <<"grpc-status">> diff --git a/test/req_SUITE.erl b/test/req_SUITE.erl index dada6b98..3d8ea00d 100644 --- a/test/req_SUITE.erl +++ b/test/req_SUITE.erl @@ -931,6 +931,7 @@ reply4(Config) -> {201, _, <<"OK">>} = do_get("/resp/reply4/201", Config), {404, _, <<"OK">>} = do_get("/resp/reply4/404", Config), {500, _, _} = do_get("/resp/reply4/error", Config), + {500, _, _} = do_get("/resp/reply4/set_cookie", Config), ok. stream_reply2(Config) -> @@ -980,6 +981,7 @@ stream_reply3(Config) -> {404, Headers3, Body} = do_get("/resp/stream_reply3/404", Config), true = lists:keymember(<<"content-type">>, 1, Headers3), {500, _, _} = do_get("/resp/stream_reply3/error", Config), + {500, _, _} = do_get("/resp/stream_reply3/set_cookie", Config), ok. stream_body_fin0(Config) -> @@ -1136,6 +1138,8 @@ stream_trailers(Config) -> {<<"grpc-status">>, <<"0">>} ]} = do_trailers("/resp/stream_trailers", Config), {_, <<"grpc-status">>} = lists:keyfind(<<"trailer">>, 1, RespHeaders), + + {500, _, _} = do_trailers("resp/stream_trailers/set_cookie", Config), ok. stream_trailers_large(Config) ->