Apply suggestions from code review

Co-authored-by: Dan Wang <[email protected]>
apache · Feb 8, 2024 · 3e3d804 · 3e3d804
1 parent 449ef32
commit 3e3d804
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/runtime/test/host_port_test.cpp b/src/runtime/test/host_port_test.cpp
@@ -232,7 +232,7 @@ TEST(host_port_test, dns_resolver)
 {
     {
         host_port hp("localhost", 8080);
-        auto addr = dns_resolver::instance().resolve_address(hp);
+        const auto &addr = dns_resolver::instance().resolve_address(hp);
         ASSERT_TRUE(rpc_address("127.0.0.1", 8080) == addr ||
                     rpc_address("127.0.1.1", 8080) == addr);
     }
@@ -247,8 +247,8 @@ TEST(host_port_test, dns_resolver)
         host_port hp2("localhost", 8081);
         g_hp->set_leader(hp2);
 
-        auto addr_grp = dns_resolver::instance().resolve_address(hp_grp);
-        auto g_addr = addr_grp.group_address();
+        const auto &addr_grp = dns_resolver::instance().resolve_address(hp_grp);
+        const auto * const g_addr = addr_grp.group_address();
 
         ASSERT_EQ(g_addr->is_update_leader_automatically(), g_hp->is_update_leader_automatically());
         ASSERT_STREQ(g_addr->name(), g_hp->name());