[mesos] branch master updated: Disabled RoleTest.RolesEndpointContainsConsumedQuota on Windows.

[as...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

asekretenko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git


The following commit(s) were added to refs/heads/master by this push:
     new 9ba066a  Disabled RoleTest.RolesEndpointContainsConsumedQuota on Windows.
9ba066a is described below

commit 9ba066a7d3372d51c9ff111ae613eed9b565738d
Author: Andrei Sekretenko <as...@apache.org>
AuthorDate: Tue Mar 3 14:14:11 2020 +0100

    Disabled RoleTest.RolesEndpointContainsConsumedQuota on Windows.
    
    This patch temporarily disables this test on Windows due to the need
    to use "posix" launcher in the current implementation of the test.
    
    Review: https://reviews.apache.org/r/72185
---
 src/tests/role_tests.cpp | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/tests/role_tests.cpp b/src/tests/role_tests.cpp
index fb8122b..b87c549 100644
--- a/src/tests/role_tests.cpp
+++ b/src/tests/role_tests.cpp
@@ -491,7 +491,12 @@ TEST_F_TEMP_DISABLED_ON_WINDOWS(RoleTest, RolesEndpointContainsQuota)
 //   - Outstanding offer
 //
 // TODO(bmahler): Test hierarchical accounting accuracy.
-TEST_F(RoleTest, RolesEndpointContainsConsumedQuota)
+//
+// TODO(asekretenko): This test is disabled on Windows because it needs "posix"
+// isolator to test quota consumption with multiple agents. After we have a
+// simple way to obtain such setup regardless of the platform, this test should
+// be re-enabled on Windows.
+TEST_F_TEMP_DISABLED_ON_WINDOWS(RoleTest, RolesEndpointContainsConsumedQuota)
 {
   Try<Owned<cluster::Master>> master = StartMaster();
   ASSERT_SOME(master);