commit a8d9355b5dbfcfd8f34a7b4a1d30c2325958f3f8
author wub <wub@google.com> Wed Apr 03 12:06:01 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Apr 03 12:06:32 2019 -0700
tree 07fbee9d922c6e53875a1d4b3d4430fc1eafe8af
parent 37ec59687e7b050dc4fd7627cff75a0c7a7d8752

gfe-relnote: (n/a) Fix some signed/unsigned integer comparisons in quic bbr test. Test only.

PiperOrigin-RevId: 241778885
Change-Id: I29a6b1269c2da72fb721bbe2c55f7bb66de219e5

quic/core/congestion_control/bbr_sender_test.cc

diff --git a/quic/core/congestion_control/bbr_sender_test.cc b/quic/core/congestion_control/bbr_sender_test.cc
index 4de237f..b3ed286 100644
--- a/quic/core/congestion_control/bbr_sender_test.cc
+++ b/quic/core/congestion_control/bbr_sender_test.cc
@@ -1321,12 +1321,12 @@
   ASSERT_FALSE(sender_->InSlowStart());
 
   const QuicConnectionStats& stats = bbr_sender_.connection()->GetStats();
-  EXPECT_EQ(1, stats.slowstart_count);
-  EXPECT_THAT(stats.slowstart_num_rtts, AllOf(Ge(5), Le(15)));
-  EXPECT_THAT(stats.slowstart_packets_sent, AllOf(Ge(100), Le(1000)));
-  EXPECT_THAT(stats.slowstart_bytes_sent, AllOf(Ge(1e5), Le(1e6)));
-  EXPECT_LE(stats.slowstart_packets_lost, 10);
-  EXPECT_LE(stats.slowstart_bytes_lost, 1e4);
+  EXPECT_EQ(1u, stats.slowstart_count);
+  EXPECT_THAT(stats.slowstart_num_rtts, AllOf(Ge(5u), Le(15u)));
+  EXPECT_THAT(stats.slowstart_packets_sent, AllOf(Ge(100u), Le(1000u)));
+  EXPECT_THAT(stats.slowstart_bytes_sent, AllOf(Ge(100000u), Le(1000000u)));
+  EXPECT_LE(stats.slowstart_packets_lost, 10u);
+  EXPECT_LE(stats.slowstart_bytes_lost, 10000u);
   EXPECT_THAT(stats.slowstart_duration,
               AllOf(Ge(QuicTime::Delta::FromMilliseconds(500)),
                     Le(QuicTime::Delta::FromMilliseconds(1500))));