Repository: mesos Updated Branches: refs/heads/master 8443b1848 -> cb1fbdaa4
Fixed signed/unsigned comparison introduced in `f22af98`. Review: https://reviews.apache.org/r/48033/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/cb1fbdaa Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/cb1fbdaa Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/cb1fbdaa Branch: refs/heads/master Commit: cb1fbdaa430cda5947bc06031cb66fe32cd5857e Parents: 8443b18 Author: Benjamin Bannier <[email protected]> Authored: Mon May 30 13:04:13 2016 +0200 Committer: Bernd Mathiske <[email protected]> Committed: Mon May 30 13:04:13 2016 +0200 ---------------------------------------------------------------------- src/tests/api_tests.cpp | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/cb1fbdaa/src/tests/api_tests.cpp ---------------------------------------------------------------------- diff --git a/src/tests/api_tests.cpp b/src/tests/api_tests.cpp index 1d7a3a9..0b6bfad 100644 --- a/src/tests/api_tests.cpp +++ b/src/tests/api_tests.cpp @@ -175,7 +175,10 @@ TEST_P(MasterAPITest, GetLoggingLevel) AWAIT_READY(v1Response); ASSERT_TRUE(v1Response.get().IsInitialized()); ASSERT_EQ(v1::master::Response::GET_LOGGING_LEVEL, v1Response.get().type()); - ASSERT_EQ(v1Response.get().get_logging_level().level(), FLAGS_v); + ASSERT_LE(0, FLAGS_v); + ASSERT_EQ( + v1Response.get().get_logging_level().level(), + static_cast<uint32_t>(FLAGS_v)); } @@ -319,7 +322,10 @@ TEST_P(AgentAPITest, GetLoggingLevel) AWAIT_READY(v1Response); ASSERT_TRUE(v1Response.get().IsInitialized()); ASSERT_EQ(v1::agent::Response::GET_LOGGING_LEVEL, v1Response.get().type()); - ASSERT_EQ(v1Response.get().get_logging_level().level(), FLAGS_v); + ASSERT_LE(0, FLAGS_v); + ASSERT_EQ( + v1Response.get().get_logging_level().level(), + static_cast<uint32_t>(FLAGS_v)); } } // namespace tests {
