Disable some of the tests in crypto_server_test.cc
They were never run in the first place since they were missing INSTANTIATE_TEST_SUITE_P, and once I added that all of them failed.
This fixes Chromium build since Chromium doesn't have GTEST_ALLOW_UNINSTANTIATED_PARAMETERIZED_TEST.
gfe-relnote: n/a (test only)
PiperOrigin-RevId: 290314666
Change-Id: I16365660bc7b3c0377777455dc0ad8e08ac32996
diff --git a/quic/core/crypto/crypto_server_test.cc b/quic/core/crypto/crypto_server_test.cc
index bddc7f3..4cf618b 100644
--- a/quic/core/crypto/crypto_server_test.cc
+++ b/quic/core/crypto/crypto_server_test.cc
@@ -1049,14 +1049,21 @@
EXPECT_EQ(0, memcmp(digest, scid_str.c_str(), scid.size()));
}
+// Those tests were declared incorrectly and thus never ran in first place.
+// TODO(b/147891553): figure out if we should fix or delete those.
+#if 0
+
class CryptoServerTestNoConfig : public CryptoServerTest {
public:
void SetUp() override {
// Deliberately don't add a config so that we can test this situation.
}
};
-GTEST_ALLOW_UNINSTANTIATED_PARAMETERIZED_TEST(
- CryptoServerTestNoConfig); // TODO(b/139702016) go/are-your-tests-running
+
+INSTANTIATE_TEST_SUITE_P(CryptoServerTestsNoConfig,
+ CryptoServerTestNoConfig,
+ ::testing::ValuesIn(GetTestParams()),
+ ::testing::PrintToStringParamName());
TEST_P(CryptoServerTestNoConfig, DontCrash) {
CryptoHandshakeMessage msg = crypto_test_utils::CreateCHLO(
@@ -1078,8 +1085,11 @@
CryptoServerTest::SetUp();
}
};
-GTEST_ALLOW_UNINSTANTIATED_PARAMETERIZED_TEST(
- CryptoServerTestOldVersion); // TODO(b/139702016) go/are-your-tests-running
+
+INSTANTIATE_TEST_SUITE_P(CryptoServerTestsOldVersion,
+ CryptoServerTestOldVersion,
+ ::testing::ValuesIn(GetTestParams()),
+ ::testing::PrintToStringParamName());
TEST_P(CryptoServerTestOldVersion, ServerIgnoresXlct) {
CryptoHandshakeMessage msg =
@@ -1124,5 +1134,7 @@
EXPECT_EQ(kSHLO, out_.tag());
}
+#endif // 0
+
} // namespace test
} // namespace quic