From 7a90b94ff968d2ac0e28deaa67c6fcba7126b0e9 Mon Sep 17 00:00:00 2001 From: Ryan Crabbe Date: Wed, 28 Jan 2026 13:21:02 -0800 Subject: [PATCH] test: add unit tests for get_standard_logging_metadata non-string user_api_key handling --- .../test_standard_logging_payload.py | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/tests/logging_callback_tests/test_standard_logging_payload.py b/tests/logging_callback_tests/test_standard_logging_payload.py index 3d8ffbf1f7f..53f555e7658 100644 --- a/tests/logging_callback_tests/test_standard_logging_payload.py +++ b/tests/logging_callback_tests/test_standard_logging_payload.py @@ -219,6 +219,22 @@ def test_get_standard_logging_metadata_invalid_user_api_key(): assert result["user_api_key_hash"] is None +def test_get_standard_logging_metadata_non_string_user_api_key(): + """Non-string user_api_key should not be set as user_api_key_hash.""" + metadata = {"user_api_key": 12345} + result = StandardLoggingPayloadSetup.get_standard_logging_metadata(metadata) + all_fields_present(result) + assert result["user_api_key_hash"] is None + + +def test_get_standard_logging_metadata_none_user_api_key(): + """None user_api_key should not be set as user_api_key_hash.""" + metadata = {"user_api_key": None} + result = StandardLoggingPayloadSetup.get_standard_logging_metadata(metadata) + all_fields_present(result) + assert result["user_api_key_hash"] is None + + def test_get_standard_logging_metadata_invalid_keys(): metadata = { "user_api_key_alias": "test_alias",