Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(486)

Unified Diff: net/quic/core/quic_utils_test.cc

Issue 2603723002: Add a new QUIC platform API for text utilities. (Closed)
Patch Set: Tests Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/quic/core/quic_utils_test.cc
diff --git a/net/quic/core/quic_utils_test.cc b/net/quic/core/quic_utils_test.cc
index a6d794e61bfd52b0e9345c9c6fdb1a224685ca79..6d0be398b07ec2ed2f57b7edd62ea2ab0b7b356d 100644
--- a/net/quic/core/quic_utils_test.cc
+++ b/net/quic/core/quic_utils_test.cc
@@ -99,27 +99,6 @@ TEST(QuicUtilsHashTest, ReferenceTest) {
reinterpret_cast<const char*>(data.data()), data.size())));
}
-TEST(QuicUtilsTest, HexDump) {
- // Verify output of the HexDump method is as expected.
- char packet[] = {
- 0x48, 0x65, 0x6c, 0x6c, 0x6f, 0x2c, 0x20, 0x51, 0x55, 0x49, 0x43, 0x21,
- 0x20, 0x54, 0x68, 0x69, 0x73, 0x20, 0x73, 0x74, 0x72, 0x69, 0x6e, 0x67,
- 0x20, 0x73, 0x68, 0x6f, 0x75, 0x6c, 0x64, 0x20, 0x62, 0x65, 0x20, 0x6c,
- 0x6f, 0x6e, 0x67, 0x20, 0x65, 0x6e, 0x6f, 0x75, 0x67, 0x68, 0x20, 0x74,
- 0x6f, 0x20, 0x73, 0x70, 0x61, 0x6e, 0x20, 0x6d, 0x75, 0x6c, 0x74, 0x69,
- 0x70, 0x6c, 0x65, 0x20, 0x6c, 0x69, 0x6e, 0x65, 0x73, 0x20, 0x6f, 0x66,
- 0x20, 0x6f, 0x75, 0x74, 0x70, 0x75, 0x74, 0x2e, 0x01, 0x02, 0x03, 0x00,
- };
- EXPECT_EQ(
- QuicUtils::HexDump(packet),
- "0x0000: 4865 6c6c 6f2c 2051 5549 4321 2054 6869 Hello,.QUIC!.Thi\n"
- "0x0010: 7320 7374 7269 6e67 2073 686f 756c 6420 s.string.should.\n"
- "0x0020: 6265 206c 6f6e 6720 656e 6f75 6768 2074 be.long.enough.t\n"
- "0x0030: 6f20 7370 616e 206d 756c 7469 706c 6520 o.span.multiple.\n"
- "0x0040: 6c69 6e65 7320 6f66 206f 7574 7075 742e lines.of.output.\n"
- "0x0050: 0102 03 ...\n");
-}
-
} // namespace
} // namespace test
} // namespace net

Powered by Google App Engine
This is Rietveld 408576698