[testing] Temporarily skip flaky event engine tests. (#33136)

Based on flaky tests reported by dashboard:
https://dashboards.corp.google.com/stubby_team.grpc_flaky_dashboard#1318s66d4f
pull/33058/head
Vignesh Babu 2 years ago committed by GitHub
parent 2daf767539
commit 63ecc4ba3e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 9
      test/core/end2end/end2end_tests.h
  2. 1
      test/core/end2end/tests/binary_metadata.cc
  3. 1
      test/core/end2end/tests/cancel_after_accept.cc
  4. 2
      test/core/end2end/tests/cancel_after_invoke.cc
  5. 1
      test/core/end2end/tests/cancel_with_status.cc
  6. 1
      test/core/end2end/tests/filtered_metadata.cc
  7. 1
      test/core/end2end/tests/retry_recv_initial_metadata.cc
  8. 5
      test/core/end2end/tests/simple_request.cc

@ -806,6 +806,15 @@ extern bool g_is_fuzzing_core_e2e_tests;
if (GetParam()->feature_mask & FEATURE_MASK_IS_MINSTACK) \
GTEST_SKIP() << "Skipping test for minstack"
#define SKIP_IF_USES_EVENT_ENGINE_CLIENT() \
if (grpc_core::IsEventEngineClientEnabled()) \
GTEST_SKIP() << "Skipping test to prevent it from using EventEngine client"
#define SKIP_IF_USES_EVENT_ENGINE_LISTENER() \
if (grpc_core::IsEventEngineListenerEnabled()) \
GTEST_SKIP() << "Skipping test to prevent it from using EventEngine " \
"listener"
#define SKIP_IF_FUZZING() \
if (g_is_fuzzing_core_e2e_tests) GTEST_SKIP() << "Skipping test for fuzzing"

@ -117,6 +117,7 @@ CORE_END2END_TEST(CoreEnd2endTest,
CORE_END2END_TEST(CoreEnd2endTest,
BinaryMetadataServerHttp2FallbackClientHttp2Fallback) {
SKIP_IF_USES_EVENT_ENGINE_CLIENT();
BinaryMetadata(*this, false, false);
}

@ -76,6 +76,7 @@ CORE_END2END_TEST(CoreDeadlineTest, DeadlineAfterAccept) {
}
CORE_END2END_TEST(CoreClientChannelTest, DeadlineAfterAcceptWithServiceConfig) {
SKIP_IF_USES_EVENT_ENGINE_CLIENT();
InitServer(ChannelArgs());
InitClient(ChannelArgs().Set(
GRPC_ARG_SERVICE_CONFIG,

@ -109,6 +109,7 @@ void CancelAfterInvoke3(CoreEnd2endTest& test,
}
CORE_END2END_TEST(CoreEnd2endTest, CancelAfterInvoke6) {
SKIP_IF_USES_EVENT_ENGINE_LISTENER();
CancelAfterInvoke6(*this, std::make_unique<CancelCancellationMode>());
}
@ -121,6 +122,7 @@ CORE_END2END_TEST(CoreEnd2endTest, CancelAfterInvoke4) {
}
CORE_END2END_TEST(CoreEnd2endTest, CancelAfterInvoke3) {
SKIP_IF_USES_EVENT_ENGINE_LISTENER();
CancelAfterInvoke3(*this, std::make_unique<CancelCancellationMode>());
}

@ -81,6 +81,7 @@ CORE_END2END_TEST(CoreEnd2endTest, CancelWithStatus3) {
}
CORE_END2END_TEST(CoreEnd2endTest, CancelWithStatus4) {
SKIP_IF_USES_EVENT_ENGINE_LISTENER();
auto c = NewClientCall("/foo").Timeout(Duration::Seconds(5)).Create();
CoreEnd2endTest::IncomingMetadata server_initial_metadata;
CoreEnd2endTest::IncomingStatusOnClient server_status;

@ -72,6 +72,7 @@ void TestRequestResponseWithMetadataToBeFiltered(
}
CORE_END2END_TEST(CoreEnd2endTest, ContentLengthIsFiltered) {
SKIP_IF_USES_EVENT_ENGINE_CLIENT();
TestRequestResponseWithMetadataToBeFiltered(*this, "content-length", "45");
}

@ -35,6 +35,7 @@ namespace {
// - first attempt receives initial metadata before trailing metadata,
// so no retry is done even though status was ABORTED
CORE_END2END_TEST(RetryTest, RetryRecvInitialMetadata) {
SKIP_IF_USES_EVENT_ENGINE_CLIENT();
InitServer(ChannelArgs());
InitClient(ChannelArgs().Set(
GRPC_ARG_SERVICE_CONFIG,

@ -98,7 +98,10 @@ void SimpleRequestBody(CoreEnd2endTest& test) {
expected_calls);
}
CORE_END2END_TEST(CoreEnd2endTest, SimpleRequest) { SimpleRequestBody(*this); }
CORE_END2END_TEST(CoreEnd2endTest, SimpleRequest) {
SKIP_IF_USES_EVENT_ENGINE_LISTENER();
SimpleRequestBody(*this);
}
CORE_END2END_TEST(CoreEnd2endTest, SimpleRequest10) {
for (int i = 0; i < 10; i++) {

Loading…
Cancel
Save