Disable the really slow end-to-end QUICHE tests outside of google3. For AddressToken* tests, also lower timeout from 30s to 1s. We already use that timeout in similar PSK tests, so that number should work consistently. PiperOrigin-RevId: 451223063
diff --git a/quiche/quic/core/http/end_to_end_test.cc b/quiche/quic/core/http/end_to_end_test.cc index 761f592..c3cf717 100644 --- a/quiche/quic/core/http/end_to_end_test.cc +++ b/quiche/quic/core/http/end_to_end_test.cc
@@ -1508,7 +1508,8 @@ VerifyCleanConnection(true); } -TEST_P(EndToEndTest, LargePostNoPacketLoss1sRTT) { +// Marked as slow since this adds a real-clock one second of delay. +TEST_P(EndToEndTest, QUICHE_SLOW_TEST(LargePostNoPacketLoss1sRTT)) { ASSERT_TRUE(Initialize()); SetPacketSendDelay(QuicTime::Delta::FromMilliseconds(1000)); @@ -1637,7 +1638,13 @@ client_->SendCustomSynchronousRequest(headers, body)); } -TEST_P(EndToEndTest, AddressToken) { +// TODO(b/214587920): make this test not rely on timeouts. +TEST_P(EndToEndTest, QUICHE_SLOW_TEST(AddressToken)) { + client_config_.set_max_time_before_crypto_handshake( + QuicTime::Delta::FromSeconds(3)); + client_config_.set_max_idle_time_before_crypto_handshake( + QuicTime::Delta::FromSeconds(1)); + client_extra_copts_.push_back(kTRTT); ASSERT_TRUE(Initialize()); if (!version_.HasIetfQuicFrames()) { @@ -1753,7 +1760,13 @@ } // Verify that client does not reuse a source address token. -TEST_P(EndToEndTest, AddressTokenNotReusedByClient) { +// TODO(b/214587920): make this test not rely on timeouts. +TEST_P(EndToEndTest, QUICHE_SLOW_TEST(AddressTokenNotReusedByClient)) { + client_config_.set_max_time_before_crypto_handshake( + QuicTime::Delta::FromSeconds(3)); + client_config_.set_max_idle_time_before_crypto_handshake( + QuicTime::Delta::FromSeconds(1)); + ASSERT_TRUE(Initialize()); if (!version_.HasIetfQuicFrames()) { return; @@ -4254,7 +4267,8 @@ // Send a packet from the client with bad encrypted data. The server should not // tear down the connection. -TEST_P(EndToEndTest, BadEncryptedData) { +// Marked as slow since it calls absl::SleepFor(). +TEST_P(EndToEndTest, QUICHE_SLOW_TEST(BadEncryptedData)) { ASSERT_TRUE(Initialize()); // Start the connection.