This is an automated email from the ASF dual-hosted git repository. bmahler pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mesos.git
commit dcbee1dcdb6dd5915252530fa6b4ffad900fc80f Author: Benjamin Mahler <bmah...@apache.org> AuthorDate: Mon Jul 15 17:10:25 2019 -0400 Minor cleanups in role_tests.cpp. Review: https://reviews.apache.org/r/71078 --- src/tests/role_tests.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/tests/role_tests.cpp b/src/tests/role_tests.cpp index 7a6c8a6..d6cc31b 100644 --- a/src/tests/role_tests.cpp +++ b/src/tests/role_tests.cpp @@ -287,7 +287,7 @@ TEST_F(RoleTest, ImplicitRoleStaticReservation) // This test checks that the "/roles" endpoint returns the expected -// information when there are no active roles. +// information when there are no known roles. TEST_F_TEMP_DISABLED_ON_WINDOWS(RoleTest, EndpointEmpty) { Try<Owned<cluster::Master>> master = StartMaster(); @@ -320,9 +320,9 @@ TEST_F_TEMP_DISABLED_ON_WINDOWS(RoleTest, EndpointEmpty) // This test checks that the "/roles" endpoint returns the expected -// information when there are configured weights and explicit roles, -// but no registered frameworks. -TEST_F(RoleTest, EndpointNoFrameworks) +// information when the role whitelist is used but no frameworks +// are present. +TEST_F(RoleTest, EndpointWithWhitelistNoFrameworks) { master::Flags masterFlags = CreateMasterFlags(); masterFlags.roles = "role1,role2";