Update

Author: yhirose

test/test.cc

@@ -3174,7 +3174,7 @@ TEST(BenchmarkTest, SimpleGetPerformance) {
   Client cli("localhost", PORT);
 
   const int NUM_REQUESTS = 50;
-  const int MAX_AVERAGE_MS = 15;
+  const int MAX_AVERAGE_MS = 5;
 
   auto warmup = cli.Get("/benchmark");
   ASSERT_TRUE(warmup);
@@ -3196,53 +3196,6 @@ TEST(BenchmarkTest, SimpleGetPerformance) {
   EXPECT_LE(avg_ms, MAX_AVERAGE_MS) << "Standalone test too slow: " << avg_ms << "ms (Issue #1777)";
 }
 
-TEST(BenchmarkTest, MultiClientConnectionTest) {
-  // Test connection overhead with multiple clients (Issue #1777)
-  // This helps identify DNS resolution delays and socket creation issues
-  
-  Server svr;
-  
-  svr.Get("/benchmark", [&](const Request & /*req*/, Response &res) {
-    res.set_content("OK", "text/plain");
-  });
-
-  auto listen_thread = std::thread([&svr]() { svr.listen("localhost", PORT); });
-  auto se = detail::scope_exit([&] {
-    svr.stop();
-    listen_thread.join();
-    ASSERT_FALSE(svr.is_running());
-  });
-
-  svr.wait_until_ready();
-  
-  const int NUM_CLIENTS = 10;
-  const int REQUESTS_PER_CLIENT = 5;
-  
-  auto start_total = std::chrono::high_resolution_clock::now();
-  
-  // Test multiple clients each making several requests
-  for (int client_id = 0; client_id < NUM_CLIENTS; ++client_id) {
-    Client cli("localhost", PORT);
-    
-    for (int req = 0; req < REQUESTS_PER_CLIENT; ++req) {
-      auto res = cli.Get("/benchmark");
-      ASSERT_TRUE(res) << "Client " << client_id << " request " << req << " failed";
-      EXPECT_EQ(StatusCode::OK_200, res->status);
-    }
-  }
-  
-  auto end_total = std::chrono::high_resolution_clock::now();
-  auto total_ms = std::chrono::duration_cast<std::chrono::milliseconds>(end_total - start_total).count();
-  auto total_requests = NUM_CLIENTS * REQUESTS_PER_CLIENT;
-  double avg_ms = static_cast<double>(total_ms) / total_requests;
-  
-  std::cout << "Multi-client: " << NUM_CLIENTS << " clients x " << REQUESTS_PER_CLIENT 
-            << " requests (" << total_requests << " total) in " << total_ms 
-            << "ms (avg: " << avg_ms << "ms)" << std::endl;
-  
-  EXPECT_LE(avg_ms, 20.0) << "Multi-client test too slow: " << avg_ms << "ms (Issue #1777)";
-}
-
 TEST_F(ServerTest, GetEmptyFile) {
   auto res = cli_.Get("/empty_file");
   ASSERT_TRUE(res);