Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 2dadd06

Browse filesBrowse files
committed
Fix test counting in SSL tests
The branch that does not support tls-server-end-point runs more tests, so we need to structure the test counting dynamically. Reviewed-by: Michael Paquier <michael@paquier.xyz>
1 parent 06ca148 commit 2dadd06
Copy full SHA for 2dadd06

File tree

Expand file treeCollapse file tree

1 file changed

+6
-5
lines changed
Filter options
Expand file treeCollapse file tree

1 file changed

+6
-5
lines changed

‎src/test/ssl/t/002_scram.pl

Copy file name to clipboardExpand all lines: src/test/ssl/t/002_scram.pl
+6-5Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,13 @@
88
use ServerSetup;
99
use File::Copy;
1010

11-
if ($ENV{with_openssl} eq 'yes')
12-
{
13-
plan tests => 6;
14-
}
15-
else
11+
if ($ENV{with_openssl} ne 'yes')
1612
{
1713
plan skip_all => 'SSL not supported by this build';
1814
}
1915

16+
my $number_of_tests = 6;
17+
2018
# This is the hostname used to connect to the server.
2119
my $SERVERHOSTADDR = '127.0.0.1';
2220

@@ -70,8 +68,11 @@
7068
"scram_channel_binding=tls-server-end-point",
7169
qr/channel binding type "tls-server-end-point" is not supported by this build/,
7270
"SCRAM authentication with tls-server-end-point as channel binding");
71+
$number_of_tests++;
7372
}
7473
test_connect_fails($common_connstr,
7574
"scram_channel_binding=not-exists",
7675
qr/unsupported SCRAM channel-binding type/,
7776
"SCRAM authentication with invalid channel binding");
77+
78+
done_testing($number_of_tests);

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.