From ee3fc640312457b883aae001dbe95c2dcc930ea5 Mon Sep 17 00:00:00 2001 From: Yuval Lifshitz Date: Wed, 2 Aug 2023 10:19:00 +0000 Subject: [PATCH] rgw/amqp: skip idleness tests since it needs to sleep longer than 30s current idle timeout is 30s, so, making the test sleep for 30s may not be enough. setting sleep time to be longer, and skippign the test so it won't take too long. Fixes: https://tracker.ceph.com/issues/62264 Signed-off-by: Yuval Lifshitz (cherry picked from commit 11439ecd8957cdf90b80fd52cc500372ffd24685) --- src/test/rgw/test_rgw_amqp.cc | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/src/test/rgw/test_rgw_amqp.cc b/src/test/rgw/test_rgw_amqp.cc index bf8671771d9..b9ee81faf1c 100644 --- a/src/test/rgw/test_rgw_amqp.cc +++ b/src/test/rgw/test_rgw_amqp.cc @@ -13,6 +13,7 @@ using namespace rgw; const std::chrono::milliseconds wait_time(10); const std::chrono::milliseconds long_wait_time = wait_time*50; +const std::chrono::seconds idle_time(35); class CctCleaner { @@ -457,3 +458,17 @@ TEST_F(TestAMQP, RetryFailWrite) amqp_mock::set_valid_host("localhost"); } +TEST_F(TestAMQP, IdleConnection) +{ + // this test is skipped since it takes 30seconds + GTEST_SKIP(); + const auto connection_number = amqp::get_connection_count(); + auto conn = amqp::connect("amqp://localhost", "ex1", false, false, boost::none); + EXPECT_EQ(amqp::get_connection_count(), connection_number + 1); + std::this_thread::sleep_for(idle_time); + EXPECT_EQ(amqp::get_connection_count(), connection_number); + auto rc = amqp::publish_with_confirm(conn, "topic", "message", my_callback_expect_nack); + EXPECT_EQ(rc, 0); + wait_until_drained(); +} + -- 2.39.5