You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bm...@apache.org on 2016/10/05 23:51:05 UTC
mesos git commit: Updated test for BadACLNoPrincipal and
BadACLDropCreateAndDestroy.
Repository: mesos
Updated Branches:
refs/heads/master 0877e89ef -> 76cc83198
Updated test for BadACLNoPrincipal and BadACLDropCreateAndDestroy.
If destroy volume failed, we should get the latest offer to make
sure that the latest offer contain the volume resource.
Review: https://reviews.apache.org/r/51946/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/76cc8319
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/76cc8319
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/76cc8319
Branch: refs/heads/master
Commit: 76cc8319855669753cc0e0836d9f9eadb992ec8a
Parents: 0877e89
Author: Guangya Liu <gy...@gmail.com>
Authored: Wed Oct 5 16:50:10 2016 -0700
Committer: Benjamin Mahler <bm...@apache.org>
Committed: Wed Oct 5 16:50:10 2016 -0700
----------------------------------------------------------------------
src/tests/persistent_volume_tests.cpp | 4 ++++
1 file changed, 4 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/76cc8319/src/tests/persistent_volume_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/persistent_volume_tests.cpp b/src/tests/persistent_volume_tests.cpp
index 2ab44d1..e10a79e 100644
--- a/src/tests/persistent_volume_tests.cpp
+++ b/src/tests/persistent_volume_tests.cpp
@@ -1619,6 +1619,8 @@ TEST_P(PersistentVolumeTest, BadACLNoPrincipal)
AWAIT_READY(offers);
EXPECT_FALSE(offers.get().empty());
+ offer = offers.get()[0];
+
// Check that the persistent volume is still contained in this offer.
// TODO(greggomann): In addition to checking that the volume is contained in
// the offer, we should also confirm that the Destroy operation failed for the
@@ -1836,6 +1838,8 @@ TEST_P(PersistentVolumeTest, BadACLDropCreateAndDestroy)
AWAIT_READY(offers);
EXPECT_FALSE(offers.get().empty());
+ offer = offers.get()[0];
+
// Check that the persistent volume is still contained in this offer.
// TODO(greggomann): In addition to checking that the volume is contained in
// the offer, we should also confirm that the Destroy operation failed for the