Skip to content

Commit 32014f1

Browse files
authored
[TEST] Disable test BasicCurlHttpTests.SendGetRequestAsync (#3722)
1 parent d882c6b commit 32014f1

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

ext/test/http/curl_http_test.cc

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -217,6 +217,9 @@ class BasicCurlHttpTests : public ::testing::Test, public HTTP_SERVER_NS::HttpRe
217217
}
218218
};
219219

220+
class DISABLED_BasicCurlHttpTests : public BasicCurlHttpTests
221+
{};
222+
220223
TEST_F(BasicCurlHttpTests, DoNothing) {}
221224

222225
TEST_F(BasicCurlHttpTests, HttpRequest)
@@ -495,7 +498,8 @@ TEST_F(BasicCurlHttpTests, GetBaseUri)
495498
"http://127.0.0.1:31339/");
496499
}
497500

498-
TEST_F(BasicCurlHttpTests, SendGetRequestAsync)
501+
// DISABLED, see https://github.com/open-telemetry/opentelemetry-cpp/issues/3535
502+
TEST_F(DISABLED_BasicCurlHttpTests, SendGetRequestAsync)
499503
{
500504
curl::HttpClient http_client;
501505

@@ -528,6 +532,8 @@ TEST_F(BasicCurlHttpTests, SendGetRequestAsync)
528532
ASSERT_FALSE(sessions[i]->IsSessionActive());
529533

530534
ASSERT_TRUE(handlers[i]->is_called_.load(std::memory_order_acquire));
535+
536+
// TODO: Spurious test failures here.
531537
ASSERT_TRUE(handlers[i]->got_response_.load(std::memory_order_acquire));
532538
}
533539

0 commit comments

Comments
 (0)