changeset 1916:6ab08c255dd3

Tests: removed quic_ciphers.t TODO, CCM cipher now supported. While here, refined test comments.
author Sergey Kandaurov <pluknet@nginx.com>
date Wed, 21 Jun 2023 17:18:09 +0400
parents 15131dd931a0
children 24fea64f233f
files quic_ciphers.t
diffstat 1 files changed, 5 insertions(+), 13 deletions(-) [+]
line wrap: on
line diff
--- a/quic_ciphers.t
+++ b/quic_ciphers.t
@@ -86,25 +86,17 @@ is(get("\x13\x03"), 'TLS_CHACHA20_POLY13
 
 is(get("\x13\x02\x13\x01"), 'TLS_AES_256_GCM_SHA384', 'ciphers many');
 
-# prefer TLS_AES_128_CCM_SHA256 and fail gracefully as we are not there yet,
-# the cipher might be patched to be enabled by default in certain distributions
-
-my $s = Test::Nginx::HTTP3->new(8980, ciphers => "\x13\x04\x13\x01");
+# prefer TLS_AES_128_CCM_SHA256 with fallback to GCM,
+# the cipher is enabled by default in some distributions
 
-TODO: {
-todo_skip 'not yet', 1 unless $s;
-
-like(get("\x13\x04\x13\x01", $s), qr/TLS_AES_128_[GC]CM_SHA256/,
+like(get("\x13\x04\x13\x01"), qr/TLS_AES_128_[GC]CM_SHA256/,
 	'TLS_AES_128_CCM_SHA256');
 
-}
-
 ###############################################################################
 
 sub get {
-	my ($ciphers, $sock) = @_;
-	my $s = Test::Nginx::HTTP3->new(8980, ciphers => $ciphers,
-		socket => $sock) or return;
+	my ($ciphers) = @_;
+	my $s = Test::Nginx::HTTP3->new(8980, ciphers => $ciphers);
 	my $frames = $s->read(all => [{ sid => $s->new_stream(), fin => 1 }]);
 
 	my ($frame) = grep { $_->{type} eq "HEADERS" } @$frames;