This is an automated email from the ASF dual-hosted git repository. bennoe pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mesos.git
commit ebcd3feee03f3a6666d7f4015ed15f71701b8252 Author: Benno Evers <[email protected]> AuthorDate: Tue Jun 4 19:42:07 2019 +0200 Fixed Mesos unit tests after Address API change. Adjusted some Mesos unit tests after the renaming of `Address::hostname()` -> `Address::lookup_hostname()`. Review: https://reviews.apache.org/r/70796 --- src/tests/master_tests.cpp | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/src/tests/master_tests.cpp b/src/tests/master_tests.cpp index 097f1b7..791c9a0 100644 --- a/src/tests/master_tests.cpp +++ b/src/tests/master_tests.cpp @@ -169,14 +169,14 @@ TEST_F(MasterTest, TaskRunning) // Ensure the hostname and url are set correctly. EXPECT_EQ( - slave.get()->pid.address.hostname().get(), + slave.get()->pid.address.lookup_hostname().get(), offers.get()[0].hostname()); mesos::URL url; url.set_scheme("http"); url.mutable_address()->set_ip(stringify(slave.get()->pid.address.ip)); url.mutable_address()->set_hostname( - slave.get()->pid.address.hostname().get()); + slave.get()->pid.address.lookup_hostname().get()); url.mutable_address()->set_port(slave.get()->pid.address.port); url.set_path("/" + slave.get()->pid.id); @@ -1447,7 +1447,7 @@ TEST_F(HostnameTest, LookupEnabled) ASSERT_SOME(master); EXPECT_EQ( - master.get()->pid.address.hostname().get(), + master.get()->pid.address.lookup_hostname().get(), master.get()->getMasterInfo().hostname()); } @@ -3642,8 +3642,10 @@ TEST_F(MasterZooKeeperTest, MasterInfoAddress) // Protect from failures on those hosts where // hostname cannot be resolved. - if (master.get()->pid.address.hostname().isSome()) { - ASSERT_EQ(master.get()->pid.address.hostname().get(), address.hostname()); + if (master.get()->pid.address.lookup_hostname().isSome()) { + ASSERT_EQ( + master.get()->pid.address.lookup_hostname().get(), + address.hostname()); } driver.stop();
