diff options
author | David Benjamin <davidben@google.com> | 2016-08-10 10:45:49 -0400 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2016-09-26 23:14:19 +0100 |
commit | a905d13bbbd25c0976ba39a0f2e55033eeca26d7 (patch) | |
tree | 99004198054a7f1a78ea20354f8b138fda615a24 /test/recipes | |
parent | 384fd75ad822569a61fe43235df270b2948a8f7d (diff) |
Don't test quite so many of them.
Avoid making the CI blow up.
Reviewed-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(cherry picked from commit 5cf6d7c51f16fd78de7921dc441e24897c8b3cc6)
Diffstat (limited to 'test/recipes')
-rw-r--r-- | test/recipes/70-test_sslcbcpadding.t | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/test/recipes/70-test_sslcbcpadding.t b/test/recipes/70-test_sslcbcpadding.t index c6eca2f713..fddbeab015 100644 --- a/test/recipes/70-test_sslcbcpadding.t +++ b/test/recipes/70-test_sslcbcpadding.t @@ -34,18 +34,22 @@ my $proxy = TLSProxy::Proxy->new( (!$ENV{HARNESS_ACTIVE} || $ENV{HARNESS_VERBOSE}) ); -plan tests => 1 + 256; - my $bad_padding_offset = -1; -# Test 1: Maximally-padded records are accepted. +# TODO: We could test all 256 values, but then the log file gets too large for +# CI. See https://github.com/openssl/openssl/issues/1440. +my @test_offsets = (0, 128, 254, 255); + +plan tests => 1 + scalar(@test_offsets); + +# Test that maximally-padded records are accepted. $proxy->start() or plan skip_all => "Unable to start up Proxy for tests"; ok(TLSProxy::Message->success(), "Maximally-padded record test"); -# Tests 2 through 257: Invalid padding. -for ($bad_padding_offset = 0; $bad_padding_offset < 256; - $bad_padding_offset++) { +# Test that invalid padding is rejected. +foreach my $offset (@test_offsets) { $proxy->clear(); + $bad_padding_offset = $offset; $proxy->start() or plan skip_all => "Unable to start up Proxy for tests";; ok(TLSProxy::Message->fail(), "Invalid padding byte $bad_padding_offset"); } |