Skip to content

Commit

Permalink
Test trailers bug when there are data frames in queue
Browse files Browse the repository at this point in the history
  • Loading branch information
tony612 committed Dec 23, 2019
1 parent 63b17e4 commit 13cf0ba
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion test/handlers/resp_h.erl
Original file line number Diff line number Diff line change
Expand Up @@ -361,7 +361,8 @@ do(<<"stream_trailers">>, Req0, Opts) ->
Req = cowboy_req:stream_reply(200, #{
<<"trailer">> => <<"grpc-status">>
}, Req0),
cowboy_req:stream_body(<<0:800000>>, nofin, Req),
%% The size should be larger than StreamSize and ConnSize
cowboy_req:stream_body(<<0:80000000>>, nofin, Req),
cowboy_req:stream_trailers(#{
<<"grpc-status">> => <<"0">>
}, Req),
Expand Down
2 changes: 1 addition & 1 deletion test/req_SUITE.erl
Original file line number Diff line number Diff line change
Expand Up @@ -1068,7 +1068,7 @@ stream_trailers(Config) ->

stream_trailers_large(Config) ->
doc("Stream large body followed by trailer headers."),
{200, RespHeaders, <<0:800000>>, [
{200, RespHeaders, <<0:80000000>>, [
{<<"grpc-status">>, <<"0">>}
]} = do_trailers("/resp/stream_trailers/large", Config),
{_, <<"grpc-status">>} = lists:keyfind(<<"trailer">>, 1, RespHeaders),
Expand Down

0 comments on commit 13cf0ba

Please sign in to comment.