You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by jo...@apache.org on 2016/03/21 16:38:15 UTC

mesos git commit: Fixed a typo in the HierarchicalAllocatorTest.CoarseGrained test.

Repository: mesos
Updated Branches:
  refs/heads/master 1d39e0d39 -> d0e745dc7


Fixed a typo in the HierarchicalAllocatorTest.CoarseGrained test.

Review: https://reviews.apache.org/r/44332/


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

Branch: refs/heads/master
Commit: d0e745dc790b46de2f099e035fcd7a1701bc6f0b
Parents: 1d39e0d
Author: Alexander Rukletsov <ru...@gmail.com>
Authored: Mon Mar 21 16:10:45 2016 +0100
Committer: Joris Van Remoortere <jo...@gmail.com>
Committed: Mon Mar 21 16:38:01 2016 +0100

----------------------------------------------------------------------
 src/tests/hierarchical_allocator_tests.cpp | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/d0e745dc/src/tests/hierarchical_allocator_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/hierarchical_allocator_tests.cpp b/src/tests/hierarchical_allocator_tests.cpp
index 459e025..bec99e3 100644
--- a/src/tests/hierarchical_allocator_tests.cpp
+++ b/src/tests/hierarchical_allocator_tests.cpp
@@ -831,18 +831,17 @@ TEST_F(HierarchicalAllocatorTest, CoarseGrained)
   AWAIT_READY(allocation);
   frameworkAllocations[allocation.get().frameworkId] = allocation.get();
 
-  // NOTE: `slave1` and `slave2` have the same resources, we don't
-  // care which framework received which slave.. only that they each
-  // received one.
+  // NOTE: `slave1` and `slave2` have the same resources, we don't care
+  // which framework received which slave, only that they each received one.
   ASSERT_TRUE(frameworkAllocations.contains(framework1.id()));
   ASSERT_EQ(1u, frameworkAllocations[framework1.id()].resources.size());
   EXPECT_EQ(slave1.resources(),
             Resources::sum(frameworkAllocations[framework1.id()].resources));
 
   ASSERT_TRUE(frameworkAllocations.contains(framework2.id()));
-  ASSERT_EQ(1u, frameworkAllocations[framework1.id()].resources.size());
+  ASSERT_EQ(1u, frameworkAllocations[framework2.id()].resources.size());
   EXPECT_EQ(slave2.resources(),
-            Resources::sum(frameworkAllocations[framework1.id()].resources));
+            Resources::sum(frameworkAllocations[framework2.id()].resources));
 }