From fab1b81b7fec9f9561145230feee48dcc36d00a2 Mon Sep 17 00:00:00 2001
From: Alexsander Hamir <alexsanderhamirgomesbaptista@gmail.com>
Date: Sat, 13 Dec 2025 13:35:20 -0800
Subject: [PATCH] fix: add agent_id field to GCS PubSub spend_logs_payload.json
 test expectation (#17938)

- Add agent_id: null to expected JSON to match actual payload structure
- Fixes test_async_gcs_pub_sub_v1 test failure
- agent_id is an optional field in SpendLogsPayload that is always included (as null when not provided)
---
 .../gcs_pub_sub_body/spend_logs_payload.json                   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/logging_callback_tests/gcs_pub_sub_body/spend_logs_payload.json b/tests/logging_callback_tests/gcs_pub_sub_body/spend_logs_payload.json
index 8b2941672b..75c229d8b1 100644
--- a/tests/logging_callback_tests/gcs_pub_sub_body/spend_logs_payload.json
+++ b/tests/logging_callback_tests/gcs_pub_sub_body/spend_logs_payload.json
@@ -28,5 +28,6 @@
     "response": "{}",
     "proxy_server_request": "{}",
     "status": "success",
-    "mcp_namespaced_tool_name": null
+    "mcp_namespaced_tool_name": null,
+    "agent_id": null
 }
\ No newline at end of file