diff h2_headers.t @ 1020:196d33c2bb45

Tests: removed TODO and try_run() checks for legacy versions. Also removed now unused spdy.t and shmem prerequisite.
author Andrey Zelenkov <zelenkov@nginx.com>
date Wed, 31 Aug 2016 18:21:07 +0300
parents 882267679006
children e2adef0c3faa
line wrap: on
line diff
--- a/h2_headers.t
+++ b/h2_headers.t
@@ -405,14 +405,8 @@ like($s->{headers}, qr/\Q{{{{{/, 'rare c
 
 ($frame) = grep { $_->{type} eq "HEADERS" } @$frames;
 is($frame->{headers}->{'x-sent-foo'}, 'aaaaa', 'well known chars');
-
-TODO: {
-local $TODO = 'not yet' unless $t->has_version('1.9.12');
-
 unlike($s->{headers}, qr/aaaaa/, 'well known chars - huffman encoding');
 
-}
-
 # response header field with huffman encoding - complete table mod \0, CR, LF
 # first saturate with short-encoded characters (NB: implementation detail)
 
@@ -429,14 +423,8 @@ my $field = pack "C*", ((map { 97 } (1 .
 
 ($frame) = grep { $_->{type} eq "HEADERS" } @$frames;
 is($frame->{headers}->{'x-sent-foo'}, $field, 'all chars');
-
-TODO: {
-local $TODO = 'not yet' unless $t->has_version('1.9.12');
-
 unlike($s->{headers}, qr/abcde/, 'all chars - huffman encoding');
 
-}
-
 # 6.3.  Dynamic Table Size Update
 
 # remove some indexed headers from the dynamic table
@@ -657,9 +645,6 @@ cmp_ok($data[-1], '<=', 2**14, 'response
 
 # response header frame sent in parts
 
-TODO: {
-local $TODO = 'not yet' unless $t->has_version('1.9.7');
-
 $s = Test::Nginx::HTTP2->new(port(8082));
 $s->h2_settings(0, 0x5 => 2**17);
 
@@ -690,8 +675,6 @@ cmp_ok($lengths, '<=', 16384, 'response 
 is(length join('', @{@$frames[-1]->{headers}->{'x-longheader'}}), 98304,
 	'response header split - headers');
 
-}
-
 # max_field_size - header field name
 
 $s = Test::Nginx::HTTP2->new(port(8084));
@@ -951,9 +934,6 @@ is($frame->{code}, 1, 'newline in reques
 
 # invalid header name as seen with underscore should not lead to ignoring rest
 
-TODO: {
-local $TODO = 'not yet' unless $t->has_version('1.9.7');
-
 $s = Test::Nginx::HTTP2->new();
 $sid = $s->new_stream({ headers => [
 	{ name => ':method', value => 'GET', mode => 0 },
@@ -967,8 +947,6 @@ local $TODO = 'not yet' unless $t->has_v
 ($frame) = grep { $_->{type} eq "HEADERS" } @$frames;
 is($frame->{headers}->{'x-referer'}, 'see-this', 'after invalid header name');
 
-}
-
 # missing mandatory request header ':scheme'
 
 TODO: {