Skip to content

Commit

Permalink
Fix BIOTest.InvokeConnectCallback test. (#1433)
Browse files Browse the repository at this point in the history
Previously, the test tried to connect to localhost on port 8080, which
is a frequently used alternative HTTP port, and it failed if there was
anything listening on that port.

Signed-off-by: Piotr Sikora <piotr@aviatrix.com>
Co-authored-by: Justin W Smith <103147162+justsmth@users.noreply.github.com>
  • Loading branch information
PiotrSikora and justsmth authored Feb 8, 2024
1 parent af5fe84 commit b614bbf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion crypto/bio/bio_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1052,7 +1052,7 @@ TEST(BIOTest, InvokeConnectCallback) {
ASSERT_NE(bio, nullptr);

ASSERT_TRUE(BIO_set_conn_hostname(bio, "localhost"));
ASSERT_TRUE(BIO_set_conn_port(bio, "8080"));
ASSERT_TRUE(BIO_set_conn_port(bio, "5325"));
ASSERT_TRUE(BIO_callback_ctrl(bio, BIO_CTRL_SET_CALLBACK, callback));

ASSERT_EQ(BIO_do_connect(bio), 0);
Expand Down

0 comments on commit b614bbf

Please sign in to comment.