From: Yingxin <yingxin.cheng@intel.com>
Date: Wed, 5 Sep 2018 15:14:09 +0000 (+0800)
Subject: tests: fix to check server_conn in MessengerTest.NameAddrTest
X-Git-Tag: v14.0.1~379^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=b3c28a16cf33cc1e5cb42cf39f0298d4cf1af4fe;p=ceph.git

tests: fix to check server_conn in MessengerTest.NameAddrTest

Signed-off-by: Yingxin <yingxin.cheng@intel.com>
---

diff --git a/src/test/msgr/test_msgr.cc b/src/test/msgr/test_msgr.cc
index facf50d13395..4ffc4d8dfb87 100644
--- a/src/test/msgr/test_msgr.cc
+++ b/src/test/msgr/test_msgr.cc
@@ -321,9 +321,9 @@ TEST_P(MessengerTest, NameAddrTest) {
   ASSERT_TRUE(conn->get_peer_addrs() == server_msgr->get_myaddrs());
   ConnectionRef server_conn = server_msgr->connect_to(
     client_msgr->get_mytype(), client_msgr->get_myaddrs());
-  // Make should server_conn is the one we already accepted from client,
-  // so it means client_msgr has the same addr when server connection has
-  ASSERT_EQ(1U, static_cast<Session*>(conn->get_priv().get())->get_count());
+  // Verify that server_conn is the one we already accepted from client,
+  // so it means the session counter in server_conn is also incremented.
+  ASSERT_EQ(1U, static_cast<Session*>(server_conn->get_priv().get())->get_count());
   server_msgr->shutdown();
   client_msgr->shutdown();
   server_msgr->wait();