Tests: removed TODO and try_run() checks for legacy versions.

Also removed now unused spdy.t and shmem prerequisite.

Author: andrey-zelenkov

auth_request.t

@@ -25,7 +25,6 @@ select STDOUT; $| = 1;
 
 my $t = Test::Nginx->new()
 	->has(qw/http rewrite proxy cache fastcgi auth_basic auth_request/)
-	->has(qw/shmem/)
 	->plan(19);
 
 $t->write_file_expand('nginx.conf', <<'EOF');

config_dump.t

@@ -25,8 +25,6 @@ select STDOUT; $| = 1;
 
 my $t = Test::Nginx->new()->has(qw/http map/);
 
-plan(skip_all => 'no config dump') unless $t->has_version('1.9.2');
-
 $t->plan(10)->write_file_expand('nginx.conf', <<'EOF');
 
 %%TEST_GLOBALS%%

fastcgi_body.t

@@ -59,9 +59,6 @@ like(http_get_length('/', ''), qr/X-Body: _eos\x0d?$/ms, 'fastcgi empty body');
 like(http_get_length('/', 'foobar'), qr/X-Body: foobar_eos\x0d?$/ms,
 	'fastcgi body');
 
-TODO: {
-local $TODO = 'not yet' unless $t->has_version('1.9.15');
-
 like(http(<<EOF), qr/X-Body: foobar_eos\x0d?$/ms, 'fastcgi chunked');
 GET / HTTP/1.1
 Host: localhost
@@ -84,8 +81,6 @@ Transfer-Encoding: chunked
 
 EOF
 
-}
-
 ###############################################################################
 
 sub http_get_length {

fastcgi_cache.t

@@ -25,7 +25,7 @@ eval { require FCGI; };
 plan(skip_all => 'FCGI not installed') if $@;
 plan(skip_all => 'win32') if $^O eq 'MSWin32';
 
-my $t = Test::Nginx->new()->has(qw/http fastcgi cache shmem/)->plan(5)
+my $t = Test::Nginx->new()->has(qw/http fastcgi cache/)->plan(5)
 	->write_file_expand('nginx.conf', <<'EOF');
 
 %%TEST_GLOBALS%%

fastcgi_merge_params.t

@@ -25,7 +25,7 @@ eval { require FCGI; };
 plan(skip_all => 'FCGI not installed') if $@;
 plan(skip_all => 'win32') if $^O eq 'MSWin32';
 
-my $t = Test::Nginx->new()->has(qw/http fastcgi cache shmem/)->plan(9)
+my $t = Test::Nginx->new()->has(qw/http fastcgi cache/)->plan(9)
 	->write_file_expand('nginx.conf', <<'EOF');
 
 %%TEST_GLOBALS%%

fastcgi_merge_params2.t

@@ -25,7 +25,7 @@ eval { require FCGI; };
 plan(skip_all => 'FCGI not installed') if $@;
 plan(skip_all => 'win32') if $^O eq 'MSWin32';
 
-my $t = Test::Nginx->new()->has(qw/http fastcgi cache shmem/)->plan(4)
+my $t = Test::Nginx->new()->has(qw/http fastcgi cache/)->plan(4)
 	->write_file_expand('nginx.conf', <<'EOF');
 
 %%TEST_GLOBALS%%

h2.t

@@ -451,9 +451,6 @@ is($frame->{headers}->{':status'}, 200, 'padding - next stream');
 
 # padding followed by CONTINUATION
 
-TODO: {
-local $TODO = 'not yet' unless $t->has_version('1.9.11');
-
 $s = Test::Nginx::HTTP2->new();
 $sid = $s->new_stream({ padding => 42, continuation => [ 2, 4, 1, 5 ],
 	headers => [
@@ -466,8 +463,6 @@ $frames = $s->read(all => [{ sid => $sid, fin => 1 }]);
 ($frame) = grep { $_->{type} eq "HEADERS" } @$frames;
 is($frame->{headers}->{':status'}, 200, 'padding - CONTINUATION');
 
-}
-
 # internal redirect
 
 $s = Test::Nginx::HTTP2->new();
@@ -625,9 +620,6 @@ is($frame->{headers}->{'content-type'}, 'text/plain; charset=utf-8', 'charset');
 # partial request header frame received (field split),
 # the rest of frame is received after client header timeout
 
-TODO: {
-local $TODO = 'not yet' unless $t->has_version('1.9.12');
-
 $s = Test::Nginx::HTTP2->new(port(8087));
 $sid = $s->new_stream({ path => '/t2.html', split => [35],
 	split_delay => 2.1 });
@@ -637,8 +629,6 @@ $frames = $s->read(all => [{ type => 'RST_STREAM' }]);
 ok($frame, 'client header timeout');
 is($frame->{code}, 1, 'client header timeout - protocol error');
 
-}
-
 $s->h2_ping('SEE-THIS');
 $frames = $s->read(all => [{ type => 'PING' }]);
 
@@ -647,9 +637,6 @@ ok($frame, 'client header timeout - PING');
 
 # partial request body data frame received, the rest is after body timeout
 
-TODO: {
-local $TODO = 'not yet' unless $t->has_version('1.9.12');
-
 $s = Test::Nginx::HTTP2->new(port(8087));
 $sid = $s->new_stream({ path => '/proxy/t2.html', body_more => 1 });
 $s->h2_body('TEST', { split => [10], split_delay => 2.1 });
@@ -659,15 +646,12 @@ $frames = $s->read(all => [{ type => 'RST_STREAM' }]);
 ok($frame, 'client body timeout');
 is($frame->{code}, 1, 'client body timeout - protocol error');
 
-}
-
 $s->h2_ping('SEE-THIS');
 $frames = $s->read(all => [{ type => 'PING' }]);
 
 ($frame) = grep { $_->{type} eq "PING" && $_->{flags} & 0x1 } @$frames;
 ok($frame, 'client body timeout - PING');
 
-
 # proxied request with logging pristine request header field (e.g., referer)
 
 $s = Test::Nginx::HTTP2->new();

h2_headers.t

@@ -405,14 +405,8 @@ $frames = $s->read(all => [{ sid => $sid, fin => 1 }]);
 
 ($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 @@ $frames = $s->read(all => [{ sid => $sid, fin => 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 header frames limited');
 
 # 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 header split - max size');
 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 request header - RST_STREAM code');
 
 # 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 @@ $frames = $s->read(all => [{ type => 'HEADERS' }]);
 ($frame) = grep { $_->{type} eq "HEADERS" } @$frames;
 is($frame->{headers}->{'x-referer'}, 'see-this', 'after invalid header name');
 
-}
-
 # missing mandatory request header ':scheme'
 
 TODO: {

h2_limit_req.t

@@ -26,8 +26,6 @@ select STDOUT; $| = 1;
 my $t = Test::Nginx->new()->has(qw/http http_v2 proxy rewrite limit_req/)
 	->plan(7);
 
-$t->todo_alerts() unless $t->has_version('1.9.14');
-
 $t->write_file_expand('nginx.conf', <<'EOF');
 
 %%TEST_GLOBALS%%
@@ -81,9 +79,6 @@ my ($frame) = grep { $_->{type} eq "HEADERS" } @$frames;
 is(read_body_file($frame->{headers}->{'x-body-file'}), 'TEST',
 	'request body - limit req');
 
-TODO: {
-local $TODO = 'not yet' unless $t->has_version('1.9.15');
-
 $s = Test::Nginx::HTTP2->new();
 $sid = $s->new_stream({ path => '/proxy_limit_req/', body_more => 1 });
 select undef, undef, undef, 1.1;
@@ -94,8 +89,6 @@ $frames = $s->read(all => [{ sid => $sid, fin => 1 }]);
 is(read_body_file($frame->{headers}->{'x-body-file'}), 'TEST',
 	'request body - limit req - limited');
 
-}
-
 # request body delayed in limit_req - with an empty DATA frame
 # "zero size buf in output" alerts seen
 
@@ -114,7 +107,6 @@ $sid = $s->new_stream();
 my ($maxwin) = sort {$a <=> $b} $s->{streams}{$sid}, $s->{conn_window};
 
 SKIP: {
-skip 'leaves coredump', 1 unless $t->has_version('1.9.7');
 skip 'not enough window', 1 if $maxwin < 5;
 
 $s = Test::Nginx::HTTP2->new();
@@ -136,10 +128,6 @@ $s = Test::Nginx::HTTP2->new();
 SKIP: {
 skip 'not enough window', 1 if $maxwin < 4;
 
-TODO: {
-todo_skip 'use-after-free', 1 unless $ENV{TEST_NGINX_UNSAFE}
-	or $t->has_version('1.9.12');
-
 $sid = $s->new_stream({ path => '/limit_req', body => 'TEST', split => [61],
 	split_delay => 1.1 });
 $frames = $s->read(all => [{ sid => $sid, fin => 1 }]);
@@ -149,8 +137,6 @@ is($frame->{headers}->{':status'}, '200', 'discard body - limit req - limited');
 
 }
 
-}
-
 $sid = $s->new_stream({ path => '/' });
 $frames = $s->read(all => [{ sid => $sid, fin => 1 }]);
 
@@ -163,10 +149,6 @@ is($frame->{headers}->{':status'}, '200', 'discard body - limit req - next');
 SKIP: {
 skip 'not enough window', 1 if $maxwin < 4;
 
-TODO: {
-todo_skip 'use-after-free', 1 unless $ENV{TEST_NGINX_UNSAFE}
-	or $t->has_version('1.9.12');
-
 $s = Test::Nginx::HTTP2->new();
 $sid = $s->new_stream({ path => '/limit_req', body => 'TEST', split => [61],
 	abort => 1 });
@@ -178,8 +160,6 @@ pass('discard body - limit req - eof');
 
 }
 
-}
-
 ###############################################################################
 
 sub read_body_file {

h2_proxy_ssl.t

@@ -26,8 +26,6 @@ select STDOUT; $| = 1;
 my $t = Test::Nginx->new()->has(qw/http http_ssl http_v2 proxy/)
 	->has_daemon('openssl')->plan(1);
 
-$t->todo_alerts() unless $t->has_version('1.9.14');
-
 $t->write_file_expand('nginx.conf', <<'EOF');
 
 %%TEST_GLOBALS%%
@@ -83,9 +81,6 @@ $t->run();
 # request body with an empty DATA frame proxied to ssl backend
 # "zero size buf in output" alerts seen
 
-TODO: {
-local $TODO = 'not yet' unless $t->has_version('1.9.14');
-
 my $s = Test::Nginx::HTTP2->new();
 my $sid = $s->new_stream({ path => '/proxy_ssl/', body_more => 1 });
 $s->h2_body('');
@@ -94,6 +89,4 @@ my $frames = $s->read(all => [{ sid => $sid, fin => 1 }]);
 my ($frame) = grep { $_->{type} eq "HEADERS" } @$frames;
 is($frame->{headers}->{':status'}, 200, 'empty request body');
 
-}
-
 ###############################################################################