fix unittest

Author: xiehuc

ext/src/http/client/curl/http_client_curl.cc

@@ -433,10 +433,10 @@ bool HttpClient::MaybeSpawnBackgroundThread()
           }
 
           std::chrono::milliseconds wait_for = std::chrono::milliseconds::zero();
-          ;
-#if LIBCURL_VERSION_NUM >= 0x074200
-          // only avaliable with curl_multi_poll, because curl_multi_wait would cause CPU busy,
-          // curl_multi_wait+sleep could not wakeup quickly
+
+#if LIBCURL_VERSION_NUM >= 0x074400
+          // only available with curl_multi_poll+curl_multi_wakeup, because curl_multi_wait would
+          // cause CPU busy, curl_multi_wait+sleep could not wakeup quickly
           wait_for = self->background_thread_wait_for_;
 #endif
           if (self->is_shutdown_.load(std::memory_order_acquire))

ext/test/http/curl_http_test.cc

@@ -545,7 +545,10 @@ TEST_F(BasicCurlHttpTests, ElegantQuitQuick)
   // when use background_thread_wait_for_ should have no side effort on elegant quit
   // because ci machine may slow, so we assert it cost should less than
   // scheduled_delay_milliseconds_
-  ASSERT_TRUE(std::chrono::system_clock::now() - beg < std::chrono::milliseconds{20});
+  auto cost = std::chrono::system_clock::now() - beg;
+  ASSERT_TRUE(cost < std::chrono::milliseconds{10})
+      << "cost ms: " << std::chrono::duration_cast<std::chrono::milliseconds>(cost).count()
+      << " libcurl version: 0x" << std::hex << LIBCURL_VERSION_NUM;
   ASSERT_TRUE(handler->is_called_);
   ASSERT_TRUE(handler->got_response_);
 }