Skip to content

Commit 3915408

Browse files
committed
Disable "disable_sni" test due to lack of robust test servers.
The current test can be run and does pass, but relies on a timeout.
1 parent cac9adf commit 3915408

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

Release/tests/functional/websockets/client/authentication_tests.cpp

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -162,15 +162,18 @@ TEST(sni_with_older_server_test)
162162
}
163163

164164
// WinRT doesn't expose option for disabling.
165-
TEST(disable_sni)
165+
// No stable server is available to reliably test this.
166+
// The configuration below relies on a timeout in the success case.
167+
TEST(disable_sni, "Ignore", "Manual")
166168
{
167169
websocket_client_config config;
170+
config.set_server_name("expired.badssl.com");
168171
config.disable_sni();
169172
websocket_client client(config);
170173

171174
try
172175
{
173-
client.connect(U("wss://swordsoftruth.com")).wait();
176+
client.connect(U("wss://badssl.com")).wait();
174177

175178
// Should never be reached.
176179
VERIFY_IS_TRUE(false);
@@ -239,3 +242,4 @@ TEST(cert_expired)
239242
}}}}
240243

241244
#endif
245+

0 commit comments

Comments
 (0)