commit 142051ab3e96977d8da6a23838cb15b7522b8ccb
author dschinazi <dschinazi@google.com> Wed Sep 18 18:17:29 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Fri Sep 20 05:26:49 2019 -0700
tree 0814df2fdf43905d10f6292cdb36fec8bb614eaf
parent 08a6c955af009f7cd047c543316a6debb9db63f1

Fix QUIC core test display

Tests names will now display a pretty name instead of the test index.

For example new string is:
QuicStreamTests/QuicStreamTest.PendingStreamStaticness/Q048
as opposed to old:
QuicStreamTests/QuicStreamTest.PendingStreamStaticness/33

gfe-relnote: n/a, test-only
PiperOrigin-RevId: 269934211
Change-Id: I44552e9730c57e00a3829f5a15ba10d8af2dc207

quic/core/quic_received_packet_manager_test.cc

diff --git a/quic/core/quic_received_packet_manager_test.cc b/quic/core/quic_received_packet_manager_test.cc
index db5e260..9ce50dd 100644
--- a/quic/core/quic_received_packet_manager_test.cc
+++ b/quic/core/quic_received_packet_manager_test.cc
@@ -45,14 +45,14 @@
 struct TestParams {
   explicit TestParams(QuicTransportVersion version) : version(version) {}
 
-  friend std::ostream& operator<<(std::ostream& os, const TestParams& p) {
-    os << "{ version: " << QuicVersionToString(p.version) << " }";
-    return os;
-  }
-
   QuicTransportVersion version;
 };
 
+// Used by ::testing::PrintToStringParamName().
+std::string PrintToString(const TestParams& p) {
+  return QuicVersionToString(p.version);
+}
+
 std::vector<TestParams> GetTestParams() {
   std::vector<TestParams> params;
   QuicTransportVersionVector all_supported_versions =
@@ -110,7 +110,8 @@
 
 INSTANTIATE_TEST_SUITE_P(QuicReceivedPacketManagerTest,
                          QuicReceivedPacketManagerTest,
-                         ::testing::ValuesIn(GetTestParams()));
+                         ::testing::ValuesIn(GetTestParams()),
+                         ::testing::PrintToStringParamName());
 
 TEST_P(QuicReceivedPacketManagerTest, DontWaitForPacketsBefore) {
   QuicPacketHeader header;