Fix test counting in SSL tests

petere · petere · commit 2dadd061b3a8 · 2018-03-07T00:53:25.000-05:00
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 &lt;michael@paquier.xyz&gt;
diff --git a/src/test/ssl/t/002_scram.pl b/src/test/ssl/t/002_scram.pl
@@ -8,15 +8,13 @@
 use ServerSetup;
 use File::Copy;
 
-if ($ENV{with_openssl} eq 'yes')
-{
-	plan tests => 6;
-}
-else
+if ($ENV{with_openssl} ne 'yes')
 {
 	plan skip_all => 'SSL not supported by this build';
 }
 
+my $number_of_tests = 6;
+
 # This is the hostname used to connect to the server.
 my $SERVERHOSTADDR = '127.0.0.1';
 
@@ -70,8 +68,11 @@
 					"scram_channel_binding=tls-server-end-point",
 					qr/channel binding type "tls-server-end-point" is not supported by this build/,
 					"SCRAM authentication with tls-server-end-point as channel binding");
+	$number_of_tests++;
 }
 test_connect_fails($common_connstr,
 	"scram_channel_binding=not-exists",
 	qr/unsupported SCRAM channel-binding type/,
 	"SCRAM authentication with invalid channel binding");
+
+done_testing($number_of_tests);
