From c3ba3e66d732fa8ab885f0be1659c3a93e86176a Mon Sep 17 00:00:00 2001 From: Tianning Li Date: Fri, 27 Mar 2026 16:03:01 -0400 Subject: [PATCH] fix(tests): remove stale _dd.compute_stats:1 assertion from logs integration test The _dd.compute_stats tag was moved from general log tags to trace-only payloads in commit 55659d93. The integration test mock still required this tag in the log body, causing the mock to never match, which triggered an infinite retry loop in Flusher::send() and hung the test indefinitely. Co-Authored-By: Claude Sonnet 4.6 --- bottlecap/tests/logs_integration_test.rs | 2 -- 1 file changed, 2 deletions(-) diff --git a/bottlecap/tests/logs_integration_test.rs b/bottlecap/tests/logs_integration_test.rs index 4b6367c9a..d35b05277 100644 --- a/bottlecap/tests/logs_integration_test.rs +++ b/bottlecap/tests/logs_integration_test.rs @@ -22,7 +22,6 @@ async fn test_logs() { // protobuf is using hashmap, can't set a btreemap to have sorted keys. Using multiple regexp since // Can't do look around since -> error: look-around, including look-ahead and look-behind, is not supported let regexp_message = r#"[{"message":{"message":"START RequestId: 459921b5-681c-4a96-beb0-81e0aa586026 Version: $LATEST","lambda":{"arn":"test-arn","request_id":"459921b5-681c-4a96-beb0-81e0aa586026"},"timestamp":1666361103165,"status":"info"},"hostname":"test-arn","service":"","#; - let regexp_compute_state = r#"_dd.compute_stats:1"#; let regexp_arch = format!(r#"architecture:{}"#, arch); let regexp_function_arn = r#"function_arn:test-arn"#; let regexp_extension_version = r#"dd_extension_version"#; @@ -34,7 +33,6 @@ async fn test_logs() { .header("DD-API-KEY", dd_api_key) .header("Content-Type", "application/json") .body_contains(regexp_message) - .body_contains(regexp_compute_state) .body_contains(regexp_arch) .body_contains(regexp_function_arn) .body_contains(regexp_extension_version);