Skip to content

Commit ad17391

Browse files
committed
fix(ci): Use >= instead of > for 80% threshold on macOS
The test was failing with exactly 40/50 (80.0%) success rate: - EXPECT_GT(40, 40) fails (strict greater than) - EXPECT_GE(40, 40) passes (greater than or equal) macOS consistently achieves exactly 80% under concurrent load, so the threshold should be inclusive (>=) rather than exclusive (>).
1 parent 9ebfad7 commit ad17391

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tests/server/health_endpoint_test.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -282,7 +282,7 @@ TEST_F(HealthEndpointTest, ConcurrentHealthChecks) {
282282
// Most requests should succeed (allowing some failures due to timing)
283283
#ifdef __APPLE__
284284
// macOS network stack is more sensitive to concurrent connections, allow lower threshold
285-
EXPECT_GT(success_count, num_requests * 0.8) << "At least 80% of concurrent requests should succeed on macOS";
285+
EXPECT_GE(success_count, num_requests * 0.8) << "At least 80% of concurrent requests should succeed on macOS";
286286
#else
287287
EXPECT_GT(success_count, num_requests * 0.9) << "At least 90% of concurrent requests should succeed";
288288
#endif

0 commit comments

Comments
 (0)