Mercurial > hg > nginx-tests

diff h3_limit_req.t @ 1888:cc13f7b098db

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Tests: avoid premature stream reset in h3_limit_req.t. STREAM and RESET_STREAM frames could be batched, which prevents the stream from being processed and changes the status code. The fix is to wait for the stream acknowledgment. Here we just look at the largest acknowledged, this should be enough for simple cases.
author Sergey Kandaurov <pluknet@nginx.com>
date Tue, 04 Apr 2023 00:33:54 +0400
parents 12d950e1165c
children 8b74936ff2ac
line wrap: on
line diff
--- a/h3_limit_req.t
+++ b/h3_limit_req.t
@@ -138,6 +138,8 @@ is($frame->{headers}->{':status'}, 200, 
 
 $s = Test::Nginx::HTTP3->new();
 $sid = $s->new_stream({ path => '/', body_more => 1 });
+wait_ack($s);
+
 $s->reset_stream($sid, 0x010c);
 $frames = $s->read(all => [{ type => 'DECODER_C' }]);
 
@@ -150,6 +152,17 @@ like($t->read_file('test.log'), qr/499/,
 
 ###############################################################################
 
+sub wait_ack {
+	my ($s) = @_;
+	my $last = $s->{pn}[0][3];
+
+	for (1 .. 5) {
+		my $frames = $s->read(all => [ {type => 'ACK' }]);
+		my ($frame) = grep { $_->{type} eq "ACK" } @$frames;
+		last unless $frame->{largest} < $last;
+	}
+}
+
 sub read_body_file {
 	my ($path) = @_;
 	return unless $path;