Fix use of EXPECT_QUIC_BUG in two tests
These calls match QUIC_BUG so they should use EXPECT_QUIC_BUG.
This was found when working on improving QUIC_BUG.
gfe-relnote: n/a, test-only
PiperOrigin-RevId: 296005794
Change-Id: I5e73ab2064b478f0824d5b94be1ba696b267397a
diff --git a/quic/core/crypto/quic_crypto_client_config_test.cc b/quic/core/crypto/quic_crypto_client_config_test.cc
index 95f76c6..4b6dcd7 100644
--- a/quic/core/crypto/quic_crypto_client_config_test.cc
+++ b/quic/core/crypto/quic_crypto_client_config_test.cc
@@ -10,6 +10,7 @@
#include "net/third_party/quiche/src/quic/core/quic_server_id.h"
#include "net/third_party/quiche/src/quic/core/quic_types.h"
#include "net/third_party/quiche/src/quic/core/quic_utils.h"
+#include "net/third_party/quiche/src/quic/platform/api/quic_expect_bug.h"
#include "net/third_party/quiche/src/quic/platform/api/quic_test.h"
#include "net/third_party/quiche/src/quic/test_tools/crypto_test_utils.h"
#include "net/third_party/quiche/src/quic/test_tools/mock_random.h"
@@ -115,9 +116,9 @@
TEST_F(QuicCryptoClientConfigTest, CachedState_ServerIdConsumedBeforeSet) {
QuicCryptoClientConfig::CachedState state;
EXPECT_FALSE(state.has_server_designated_connection_id());
- EXPECT_QUIC_DEBUG_DEATH(state.GetNextServerDesignatedConnectionId(),
- "Attempting to consume a connection id "
- "that was never designated.");
+ EXPECT_QUIC_BUG(state.GetNextServerDesignatedConnectionId(),
+ "Attempting to consume a connection id "
+ "that was never designated.");
}
TEST_F(QuicCryptoClientConfigTest, CachedState_ServerNonce) {
@@ -154,9 +155,9 @@
TEST_F(QuicCryptoClientConfigTest, CachedState_ServerNonceConsumedBeforeSet) {
QuicCryptoClientConfig::CachedState state;
EXPECT_FALSE(state.has_server_nonce());
- EXPECT_QUIC_DEBUG_DEATH(state.GetNextServerNonce(),
- "Attempting to consume a server nonce "
- "that was never designated.");
+ EXPECT_QUIC_BUG(state.GetNextServerNonce(),
+ "Attempting to consume a server nonce "
+ "that was never designated.");
}
TEST_F(QuicCryptoClientConfigTest, CachedState_InitializeFrom) {
diff --git a/quic/core/quic_interval_deque_test.cc b/quic/core/quic_interval_deque_test.cc
index 8d57d5f..f7a8441 100644
--- a/quic/core/quic_interval_deque_test.cc
+++ b/quic/core/quic_interval_deque_test.cc
@@ -82,8 +82,7 @@
qid.PopFront();
EXPECT_EQ(qid.Size(), std::size_t(0));
- EXPECT_QUIC_DEBUG_DEATH(qid.PopFront(),
- "Trying to pop from an empty container.");
+ EXPECT_QUIC_BUG(qid.PopFront(), "Trying to pop from an empty container.");
}
// The goal of this test is to push data into the container at specific
