You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ji...@apache.org on 2015/05/13 20:04:30 UTC

[2/2] mesos git commit: Fixed compiler warnings in reservation tests.

Fixed compiler warnings in reservation tests.


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/9db83273
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/9db83273
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/9db83273

Branch: refs/heads/master
Commit: 9db83273ea668c47206db3b0cb613afd7855abcb
Parents: 96e203f
Author: Jie Yu <yu...@gmail.com>
Authored: Wed May 13 11:02:38 2015 -0700
Committer: Jie Yu <yu...@gmail.com>
Committed: Wed May 13 11:02:38 2015 -0700

----------------------------------------------------------------------
 src/tests/reservation_tests.cpp | 40 ++++++++++++++++++------------------
 1 file changed, 20 insertions(+), 20 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/9db83273/src/tests/reservation_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/reservation_tests.cpp b/src/tests/reservation_tests.cpp
index 9020d04..755a375 100644
--- a/src/tests/reservation_tests.cpp
+++ b/src/tests/reservation_tests.cpp
@@ -125,7 +125,7 @@ TEST_F(ReservationTest, ReserveThenUnreserve)
   // In the first offer, expect an offer with unreserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   Offer offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(unreserved));
@@ -140,7 +140,7 @@ TEST_F(ReservationTest, ReserveThenUnreserve)
   // In the next offer, expect an offer with reserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(dynamicallyReserved));
@@ -155,7 +155,7 @@ TEST_F(ReservationTest, ReserveThenUnreserve)
   // In the next offer, expect an offer with unreserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(unreserved));
@@ -217,7 +217,7 @@ TEST_F(ReservationTest, ReserveAndLaunchThenUnreserve)
   // In the first offer, expect an offer with unreserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   Offer offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(unreserved));
@@ -247,7 +247,7 @@ TEST_F(ReservationTest, ReserveAndLaunchThenUnreserve)
   // In the next offer, expect an offer with reserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(dynamicallyReserved));
@@ -267,7 +267,7 @@ TEST_F(ReservationTest, ReserveAndLaunchThenUnreserve)
   // In the next offer, expect an offer with unreserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(unreserved));
@@ -336,7 +336,7 @@ TEST_F(ReservationTest, ReserveShareWithinRole)
   // In the first offer, expect an offer with unreserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   Offer offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(unreserved));
@@ -356,7 +356,7 @@ TEST_F(ReservationTest, ReserveShareWithinRole)
   // In the next offer, expect an offer with reserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(dynamicallyReserved));
@@ -381,7 +381,7 @@ TEST_F(ReservationTest, ReserveShareWithinRole)
   // framework1.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(dynamicallyReserved));
@@ -448,7 +448,7 @@ TEST_F(ReservationTest, DropReserveTooLarge)
   // In the first offer, expect an offer with unreserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   Offer offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(unreserved));
@@ -476,7 +476,7 @@ TEST_F(ReservationTest, DropReserveTooLarge)
   // resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(unreserved));
@@ -540,7 +540,7 @@ TEST_F(ReservationTest, DropReserveStaticReservation)
   // resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   Offer offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(staticallyReserved));
@@ -565,7 +565,7 @@ TEST_F(ReservationTest, DropReserveStaticReservation)
   // reserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1);
+  ASSERT_EQ(1u, offers.get().size());
   offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(staticallyReserved));
@@ -627,7 +627,7 @@ TEST_F(ReservationTest, SendingCheckpointResourcesMessage)
   // 'unreserved2'.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1u);
+  ASSERT_EQ(1u, offers.get().size());
   Offer offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(unreserved1 + unreserved2));
@@ -723,7 +723,7 @@ TEST_F(ReservationTest, ResourcesCheckpointing)
   // Expect an offer with the unreserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1u);
+  ASSERT_EQ(1u, offers.get().size());
   Offer offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(unreserved));
@@ -814,7 +814,7 @@ TEST_F(ReservationTest, MasterFailover)
   // In the first offer, expect an offer with unreserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1u);
+  ASSERT_EQ(1u, offers.get().size());
   Offer offer = offers.get()[0];
 
   Future<CheckpointResourcesMessage> checkpointResources =
@@ -863,7 +863,7 @@ TEST_F(ReservationTest, MasterFailover)
   // In the next offer, expect an offer with the reserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1u);
+  ASSERT_EQ(1u, offers.get().size());
   offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(reserved));
@@ -933,7 +933,7 @@ TEST_F(ReservationTest, CompatibleCheckpointedResources)
   // In the first offer, expect an offer with unreserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1u);
+  ASSERT_EQ(1u, offers.get().size());
   Offer offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(unreserved));
@@ -1047,7 +1047,7 @@ TEST_F(ReservationTest, CompatibleCheckpointedResourcesWithPersistentVolumes)
 
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1u);
+  ASSERT_EQ(1u, offers.get().size());
   Offer offer = offers.get()[0];
 
   EXPECT_TRUE(
@@ -1163,7 +1163,7 @@ TEST_F(ReservationTest, IncompatibleCheckpointedResources)
   // In the first offer, expect an offer with unreserved resources.
   AWAIT_READY(offers);
 
-  ASSERT_EQ(offers.get().size(), 1u);
+  ASSERT_EQ(1u, offers.get().size());
   Offer offer = offers.get()[0];
 
   EXPECT_TRUE(Resources(offer.resources()).contains(unreserved));